From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753859AbeDTHWc (ORCPT ); Fri, 20 Apr 2018 03:22:32 -0400 Received: from mail-wr0-f195.google.com ([209.85.128.195]:35339 "EHLO mail-wr0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750985AbeDTHWb (ORCPT ); Fri, 20 Apr 2018 03:22:31 -0400 X-Google-Smtp-Source: AIpwx48CC6ZUO/5Kn4deKgNPa80iZ+5FZbYyI3JlUHqMjEfJEXrYj4QIUf7vPcHpWJEl4947romADA== Date: Fri, 20 Apr 2018 09:22:27 +0200 From: Daniel Vetter To: Roger Pau =?iso-8859-1?Q?Monn=E9?= Cc: Oleksandr Andrushchenko , "jgross@suse.com" , Artem Mygaiev , Dongwon Kim , "Oleksandr_Andrushchenko@epam.com" , "airlied@linux.ie" , "linux-kernel@vger.kernel.org" , "dri-devel@lists.freedesktop.org" , Paul Durrant , "Potrola, MateuszX" , "daniel.vetter@intel.com" , "xen-devel@lists.xenproject.org" , "boris.ostrovsky@oracle.com" Subject: Re: [Xen-devel] [PATCH 0/1] drm/xen-zcopy: Add Xen zero-copy helper DRM driver Message-ID: <20180420072227.GH31310@phenom.ffwll.local> Mail-Followup-To: Roger Pau =?iso-8859-1?Q?Monn=E9?= , Oleksandr Andrushchenko , "jgross@suse.com" , Artem Mygaiev , Dongwon Kim , "Oleksandr_Andrushchenko@epam.com" , "airlied@linux.ie" , "linux-kernel@vger.kernel.org" , "dri-devel@lists.freedesktop.org" , Paul Durrant , "Potrola, MateuszX" , "daniel.vetter@intel.com" , "xen-devel@lists.xenproject.org" , "boris.ostrovsky@oracle.com" References: <20180416192905.GA18096@downor-Z87X-UD5H> <20180417075928.GT31310@phenom.ffwll.local> <20180417205744.GA15930@downor-Z87X-UD5H> <41487acb-a67a-8933-d0c3-702c19b0938e@gmail.com> <20180418073508.ptvntwedczpvl7bx@MacBook-Pro-de-Roger.local> <20180418101058.hyqk3gr3b2ibxswu@MacBook-Pro-de-Roger.local> <7d6710a76b9a42299139d7914358ed52@AMSPEX02CL03.citrite.net> <46489b33-e6fc-b874-6cd4-dbb94c002ef8@gmail.com> <20180418105526.a4qtlhofrn3gubsl@MacBook-Pro-de-Roger.local> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20180418105526.a4qtlhofrn3gubsl@MacBook-Pro-de-Roger.local> X-Operating-System: Linux phenom 4.15.0-1-amd64 User-Agent: Mutt/1.9.4 (2018-02-28) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Apr 18, 2018 at 11:55:26AM +0100, Roger Pau Monné wrote: > On Wed, Apr 18, 2018 at 01:39:35PM +0300, Oleksandr Andrushchenko wrote: > > On 04/18/2018 01:18 PM, Paul Durrant wrote: > > > > -----Original Message----- > > > > From: Xen-devel [mailto:xen-devel-bounces@lists.xenproject.org] On Behalf > > > > Of Roger Pau Monné > > > > Sent: 18 April 2018 11:11 > > > > To: Oleksandr Andrushchenko > > > > Cc: jgross@suse.com; Artem Mygaiev ; > > > > Dongwon Kim ; airlied@linux.ie; > > > > Oleksandr_Andrushchenko@epam.com; linux-kernel@vger.kernel.org; dri- > > > > devel@lists.freedesktop.org; Potrola, MateuszX > > > > ; xen-devel@lists.xenproject.org; > > > > daniel.vetter@intel.com; boris.ostrovsky@oracle.com; Matt Roper > > > > > > > > Subject: Re: [Xen-devel] [PATCH 0/1] drm/xen-zcopy: Add Xen zero-copy > > > > helper DRM driver > > > > > > > > On Wed, Apr 18, 2018 at 11:01:12AM +0300, Oleksandr Andrushchenko > > > > wrote: > > > > > On 04/18/2018 10:35 AM, Roger Pau Monné wrote: > > > > After speaking with Oleksandr on IRC, I think the main usage of the > > > > gntdev extension is to: > > > > > > > > 1. Create a dma-buf from a set of grant references. > > > > 2. Share dma-buf and get a list of grant references. > > > > > > > > I think this set of operations could be broken into: > > > > > > > > 1.1 Map grant references into user-space using the gntdev. > > > > 1.2 Create a dma-buf out of a set of user-space virtual addresses. > > > > > > > > 2.1 Map a dma-buf into user-space. > > > > 2.2 Get grefs out of the user-space addresses where the dma-buf is > > > > mapped. > > > > > > > > So it seems like what's actually missing is a way to: > > > > > > > > - Create a dma-buf from a list of user-space virtual addresses. > > > > - Allow to map a dma-buf into user-space, so it can then be used with > > > > the gntdev. > > > > > > > > I think this is generic enough that it could be implemented by a > > > > device not tied to Xen. AFAICT the hyper_dma guys also wanted > > > > something similar to this. > > Ok, so just to summarize, xen-zcopy/hyper-dmabuf as they are now, > > are no go from your POV? > > My opinion is that there seems to be a more generic way to implement > this, and thus I would prefer that one. > > > Instead, we have to make all that fancy stuff > > with VAs <-> device-X and have that device-X driver live out of drivers/xen > > as it is not a Xen specific driver? > > That would be my preference if feasible, simply because it can be > reused by other use-cases that need to create dma-bufs in user-space. > > In any case I just knew about dma-bufs this morning, there might be > things that I'm missing. See my other reply, I really don't want a generic userspace memory -> dma-buf thing, it doesn't work. Having a xen-specific grant references <-> dma-buf conversion thing (and I'm honestly still not sure whether the dma-buf -> grant renferences is a good idea) seems much more reasonable to me. Imo simplest way forward would be to start out with a grant refs -> dma-buf ioctl, and see how far that gets us. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [Xen-devel] [PATCH 0/1] drm/xen-zcopy: Add Xen zero-copy helper DRM driver Date: Fri, 20 Apr 2018 09:22:27 +0200 Message-ID: <20180420072227.GH31310@phenom.ffwll.local> References: <20180416192905.GA18096@downor-Z87X-UD5H> <20180417075928.GT31310@phenom.ffwll.local> <20180417205744.GA15930@downor-Z87X-UD5H> <41487acb-a67a-8933-d0c3-702c19b0938e@gmail.com> <20180418073508.ptvntwedczpvl7bx@MacBook-Pro-de-Roger.local> <20180418101058.hyqk3gr3b2ibxswu@MacBook-Pro-de-Roger.local> <7d6710a76b9a42299139d7914358ed52@AMSPEX02CL03.citrite.net> <46489b33-e6fc-b874-6cd4-dbb94c002ef8@gmail.com> <20180418105526.a4qtlhofrn3gubsl@MacBook-Pro-de-Roger.local> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mail-wr0-x243.google.com (mail-wr0-x243.google.com [IPv6:2a00:1450:400c:c0c::243]) by gabe.freedesktop.org (Postfix) with ESMTPS id B11F46E101 for ; Fri, 20 Apr 2018 07:22:31 +0000 (UTC) Received: by mail-wr0-x243.google.com with SMTP id h3-v6so20191653wrh.5 for ; Fri, 20 Apr 2018 00:22:31 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20180418105526.a4qtlhofrn3gubsl@MacBook-Pro-de-Roger.local> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Roger Pau =?iso-8859-1?Q?Monn=E9?= Cc: "jgross@suse.com" , Artem Mygaiev , Dongwon Kim , "Oleksandr_Andrushchenko@epam.com" , Oleksandr Andrushchenko , "linux-kernel@vger.kernel.org" , "dri-devel@lists.freedesktop.org" , "airlied@linux.ie" , Paul Durrant , "Potrola, MateuszX" , "xen-devel@lists.xenproject.org" , "daniel.vetter@intel.com" , "boris.ostrovsky@oracle.com" List-Id: dri-devel@lists.freedesktop.org T24gV2VkLCBBcHIgMTgsIDIwMTggYXQgMTE6NTU6MjZBTSArMDEwMCwgUm9nZXIgUGF1IE1vbm7D qSB3cm90ZToKPiBPbiBXZWQsIEFwciAxOCwgMjAxOCBhdCAwMTozOTozNVBNICswMzAwLCBPbGVr c2FuZHIgQW5kcnVzaGNoZW5rbyB3cm90ZToKPiA+IE9uIDA0LzE4LzIwMTggMDE6MTggUE0sIFBh dWwgRHVycmFudCB3cm90ZToKPiA+ID4gPiAtLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQo+ID4g PiA+IEZyb206IFhlbi1kZXZlbCBbbWFpbHRvOnhlbi1kZXZlbC1ib3VuY2VzQGxpc3RzLnhlbnBy b2plY3Qub3JnXSBPbiBCZWhhbGYKPiA+ID4gPiBPZiBSb2dlciBQYXUgTW9ubsOpCj4gPiA+ID4g U2VudDogMTggQXByaWwgMjAxOCAxMToxMQo+ID4gPiA+IFRvOiBPbGVrc2FuZHIgQW5kcnVzaGNo ZW5rbyA8YW5kcjIwMDBAZ21haWwuY29tPgo+ID4gPiA+IENjOiBqZ3Jvc3NAc3VzZS5jb207IEFy dGVtIE15Z2FpZXYgPEFydGVtX015Z2FpZXZAZXBhbS5jb20+Owo+ID4gPiA+IERvbmd3b24gS2lt IDxkb25nd29uLmtpbUBpbnRlbC5jb20+OyBhaXJsaWVkQGxpbnV4LmllOwo+ID4gPiA+IE9sZWtz YW5kcl9BbmRydXNoY2hlbmtvQGVwYW0uY29tOyBsaW51eC1rZXJuZWxAdmdlci5rZXJuZWwub3Jn OyBkcmktCj4gPiA+ID4gZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnOyBQb3Ryb2xhLCBNYXRl dXN6WAo+ID4gPiA+IDxtYXRldXN6eC5wb3Ryb2xhQGludGVsLmNvbT47IHhlbi1kZXZlbEBsaXN0 cy54ZW5wcm9qZWN0Lm9yZzsKPiA+ID4gPiBkYW5pZWwudmV0dGVyQGludGVsLmNvbTsgYm9yaXMu b3N0cm92c2t5QG9yYWNsZS5jb207IE1hdHQgUm9wZXIKPiA+ID4gPiA8bWF0dGhldy5kLnJvcGVy QGludGVsLmNvbT4KPiA+ID4gPiBTdWJqZWN0OiBSZTogW1hlbi1kZXZlbF0gW1BBVENIIDAvMV0g ZHJtL3hlbi16Y29weTogQWRkIFhlbiB6ZXJvLWNvcHkKPiA+ID4gPiBoZWxwZXIgRFJNIGRyaXZl cgo+ID4gPiA+IAo+ID4gPiA+IE9uIFdlZCwgQXByIDE4LCAyMDE4IGF0IDExOjAxOjEyQU0gKzAz MDAsIE9sZWtzYW5kciBBbmRydXNoY2hlbmtvCj4gPiA+ID4gd3JvdGU6Cj4gPiA+ID4gPiBPbiAw NC8xOC8yMDE4IDEwOjM1IEFNLCBSb2dlciBQYXUgTW9ubsOpIHdyb3RlOgo+ID4gPiA+IEFmdGVy IHNwZWFraW5nIHdpdGggT2xla3NhbmRyIG9uIElSQywgSSB0aGluayB0aGUgbWFpbiB1c2FnZSBv ZiB0aGUKPiA+ID4gPiBnbnRkZXYgZXh0ZW5zaW9uIGlzIHRvOgo+ID4gPiA+IAo+ID4gPiA+IDEu IENyZWF0ZSBhIGRtYS1idWYgZnJvbSBhIHNldCBvZiBncmFudCByZWZlcmVuY2VzLgo+ID4gPiA+ IDIuIFNoYXJlIGRtYS1idWYgYW5kIGdldCBhIGxpc3Qgb2YgZ3JhbnQgcmVmZXJlbmNlcy4KPiA+ ID4gPiAKPiA+ID4gPiBJIHRoaW5rIHRoaXMgc2V0IG9mIG9wZXJhdGlvbnMgY291bGQgYmUgYnJv a2VuIGludG86Cj4gPiA+ID4gCj4gPiA+ID4gMS4xIE1hcCBncmFudCByZWZlcmVuY2VzIGludG8g dXNlci1zcGFjZSB1c2luZyB0aGUgZ250ZGV2Lgo+ID4gPiA+IDEuMiBDcmVhdGUgYSBkbWEtYnVm IG91dCBvZiBhIHNldCBvZiB1c2VyLXNwYWNlIHZpcnR1YWwgYWRkcmVzc2VzLgo+ID4gPiA+IAo+ ID4gPiA+IDIuMSBNYXAgYSBkbWEtYnVmIGludG8gdXNlci1zcGFjZS4KPiA+ID4gPiAyLjIgR2V0 IGdyZWZzIG91dCBvZiB0aGUgdXNlci1zcGFjZSBhZGRyZXNzZXMgd2hlcmUgdGhlIGRtYS1idWYg aXMKPiA+ID4gPiAgICAgIG1hcHBlZC4KPiA+ID4gPiAKPiA+ID4gPiBTbyBpdCBzZWVtcyBsaWtl IHdoYXQncyBhY3R1YWxseSBtaXNzaW5nIGlzIGEgd2F5IHRvOgo+ID4gPiA+IAo+ID4gPiA+ICAg LSBDcmVhdGUgYSBkbWEtYnVmIGZyb20gYSBsaXN0IG9mIHVzZXItc3BhY2UgdmlydHVhbCBhZGRy ZXNzZXMuCj4gPiA+ID4gICAtIEFsbG93IHRvIG1hcCBhIGRtYS1idWYgaW50byB1c2VyLXNwYWNl LCBzbyBpdCBjYW4gdGhlbiBiZSB1c2VkIHdpdGgKPiA+ID4gPiAgICAgdGhlIGdudGRldi4KPiA+ ID4gPiAKPiA+ID4gPiBJIHRoaW5rIHRoaXMgaXMgZ2VuZXJpYyBlbm91Z2ggdGhhdCBpdCBjb3Vs ZCBiZSBpbXBsZW1lbnRlZCBieSBhCj4gPiA+ID4gZGV2aWNlIG5vdCB0aWVkIHRvIFhlbi4gQUZB SUNUIHRoZSBoeXBlcl9kbWEgZ3V5cyBhbHNvIHdhbnRlZAo+ID4gPiA+IHNvbWV0aGluZyBzaW1p bGFyIHRvIHRoaXMuCj4gPiBPaywgc28ganVzdCB0byBzdW1tYXJpemUsIHhlbi16Y29weS9oeXBl ci1kbWFidWYgYXMgdGhleSBhcmUgbm93LAo+ID4gYXJlIG5vIGdvIGZyb20geW91ciBQT1Y/Cj4g Cj4gTXkgb3BpbmlvbiBpcyB0aGF0IHRoZXJlIHNlZW1zIHRvIGJlIGEgbW9yZSBnZW5lcmljIHdh eSB0byBpbXBsZW1lbnQKPiB0aGlzLCBhbmQgdGh1cyBJIHdvdWxkIHByZWZlciB0aGF0IG9uZS4K PiAKPiA+IEluc3RlYWQsIHdlIGhhdmUgdG8gbWFrZSBhbGwgdGhhdCBmYW5jeSBzdHVmZgo+ID4g d2l0aCBWQXMgPC0+IGRldmljZS1YIGFuZCBoYXZlIHRoYXQgZGV2aWNlLVggZHJpdmVyIGxpdmUg b3V0IG9mIGRyaXZlcnMveGVuCj4gPiBhcyBpdCBpcyBub3QgYSBYZW4gc3BlY2lmaWMgZHJpdmVy Pwo+IAo+IFRoYXQgd291bGQgYmUgbXkgcHJlZmVyZW5jZSBpZiBmZWFzaWJsZSwgc2ltcGx5IGJl Y2F1c2UgaXQgY2FuIGJlCj4gcmV1c2VkIGJ5IG90aGVyIHVzZS1jYXNlcyB0aGF0IG5lZWQgdG8g Y3JlYXRlIGRtYS1idWZzIGluIHVzZXItc3BhY2UuCj4gCj4gSW4gYW55IGNhc2UgSSBqdXN0IGtu ZXcgYWJvdXQgZG1hLWJ1ZnMgdGhpcyBtb3JuaW5nLCB0aGVyZSBtaWdodCBiZQo+IHRoaW5ncyB0 aGF0IEknbSBtaXNzaW5nLgoKU2VlIG15IG90aGVyIHJlcGx5LCBJIHJlYWxseSBkb24ndCB3YW50 IGEgZ2VuZXJpYyB1c2Vyc3BhY2UgbWVtb3J5IC0+CmRtYS1idWYgdGhpbmcsIGl0IGRvZXNuJ3Qg d29yay4gSGF2aW5nIGEgeGVuLXNwZWNpZmljIGdyYW50IHJlZmVyZW5jZXMgPC0+CmRtYS1idWYg Y29udmVyc2lvbiB0aGluZyAoYW5kIEknbSBob25lc3RseSBzdGlsbCBub3Qgc3VyZSB3aGV0aGVy IHRoZQpkbWEtYnVmIC0+IGdyYW50IHJlbmZlcmVuY2VzIGlzIGEgZ29vZCBpZGVhKSBzZWVtcyBt dWNoIG1vcmUgcmVhc29uYWJsZSB0bwptZS4KCkltbyBzaW1wbGVzdCB3YXkgZm9yd2FyZCB3b3Vs ZCBiZSB0byBzdGFydCBvdXQgd2l0aCBhIGdyYW50IHJlZnMgLT4KZG1hLWJ1ZiBpb2N0bCwgYW5k IHNlZSBob3cgZmFyIHRoYXQgZ2V0cyB1cy4KLURhbmllbAotLSAKRGFuaWVsIFZldHRlcgpTb2Z0 d2FyZSBFbmdpbmVlciwgSW50ZWwgQ29ycG9yYXRpb24KaHR0cDovL2Jsb2cuZmZ3bGwuY2gKX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVsIG1h aWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMu ZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg==