From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751286AbdFECjv (ORCPT ); Sun, 4 Jun 2017 22:39:51 -0400 Received: from mga11.intel.com ([192.55.52.93]:32638 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751204AbdFECjn (ORCPT ); Sun, 4 Jun 2017 22:39:43 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.39,298,1493708400"; d="scan'208";a="976748520" From: "Chen, Xiaoguang" To: Gerd Hoffmann , Alex Williamson CC: "Tian, Kevin" , "intel-gfx@lists.freedesktop.org" , "linux-kernel@vger.kernel.org" , "zhenyuw@linux.intel.com" , "chris@chris-wilson.co.uk" , "Lv, Zhiyuan" , "intel-gvt-dev@lists.freedesktop.org" , "Wang, Zhi A" Subject: RE: [PATCH v6 6/6] drm/i915/gvt: Adding interface so user space can get the dma-buf Thread-Topic: [PATCH v6 6/6] drm/i915/gvt: Adding interface so user space can get the dma-buf Thread-Index: AQHS1sTjDVt2qF3L0keVbZ8anq02C6IP0HMAgAEcHXD//4I1gIAA06kg///rUYCAAAcdAIAEXL5Q Date: Mon, 5 Jun 2017 02:39:32 +0000 Message-ID: References: <1495874332-2851-1-git-send-email-xiaoguang.chen@intel.com> <1495874332-2851-7-git-send-email-xiaoguang.chen@intel.com> <20170601120820.3358f7dd@w520.home> <20170601213459.461ecf36@w520.home> <20170602085831.41454a1f@t450s.home> <1496417038.16895.7.camel@redhat.com> In-Reply-To: <1496417038.16895.7.camel@redhat.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 10.0.102.7 dlp-reaction: no-action x-originating-ip: [10.239.127.40] Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id v552e0Yl003724 Hi, >-----Original Message----- >From: intel-gvt-dev [mailto:intel-gvt-dev-bounces@lists.freedesktop.org] On >Behalf Of Gerd Hoffmann >Sent: Friday, June 02, 2017 11:24 PM >To: Alex Williamson ; Chen, Xiaoguang > >Cc: Tian, Kevin ; intel-gfx@lists.freedesktop.org; linux- >kernel@vger.kernel.org; zhenyuw@linux.intel.com; chris@chris-wilson.co.uk; Lv, >Zhiyuan ; intel-gvt-dev@lists.freedesktop.org; Wang, Zhi >A >Subject: Re: [PATCH v6 6/6] drm/i915/gvt: Adding interface so user space can get >the dma-buf > > Hi, > >> > When i915's dma-buf's release() callback is called it will try to >> > free the gem object associated with the dma-buf if its ref count is >> > 0. But in our case the ref count is 1 so no free callback is called >> > so we can not release allocations there. > >Why the ref count is one? The gem object is created by us while creating the dma-buf(the ref count of the gem object is initialized to 1). Later when user import the dma-buf the ref count of the gem object associate with the dma-buf will increased. When user finished using the dma-buf it will decrease the ref count. But the ref count of the gem object will become 1 when all the user finished using the dma-buf because we create the gem object(the test also showing this result). Typically user only export a dma-buf(no gem object yet) then when user import the dma-buf then a gem object will be created. But in our case we do not implement the dma-buf from scratch but calling the i915_gem_prime_export() where a gem object is an input parameter. Chenxg >Who holds a reference and why? >Maybe it should be the other way around, i.e. the dmabuf holds a reference on >the vgpu instance backing it, i.e. you can't delete the vgpu while dma-bufs exist? > >> We cannot simply say that the user isn't allowed to release them in >> that order. > >Yep, not going to fly. Can happen even unintentionally because we can pass >around dmabufs to other processes. Example: qemu passes dmabuf to spice- >client, then qemu crashes. mgmt fd is closed before dmabuf fd then. The kernel >must be able to handle that. > >cheers, > Gerd >_______________________________________________ >intel-gvt-dev mailing list >intel-gvt-dev@lists.freedesktop.org >https://lists.freedesktop.org/mailman/listinfo/intel-gvt-dev From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Chen, Xiaoguang" Subject: Re: [PATCH v6 6/6] drm/i915/gvt: Adding interface so user space can get the dma-buf Date: Mon, 5 Jun 2017 02:39:32 +0000 Message-ID: References: <1495874332-2851-1-git-send-email-xiaoguang.chen@intel.com> <1495874332-2851-7-git-send-email-xiaoguang.chen@intel.com> <20170601120820.3358f7dd@w520.home> <20170601213459.461ecf36@w520.home> <20170602085831.41454a1f@t450s.home> <1496417038.16895.7.camel@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <1496417038.16895.7.camel@redhat.com> Content-Language: en-US List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Gerd Hoffmann , Alex Williamson Cc: "intel-gfx@lists.freedesktop.org" , "linux-kernel@vger.kernel.org" , "Lv, Zhiyuan" , "intel-gvt-dev@lists.freedesktop.org" List-Id: intel-gfx@lists.freedesktop.org SGksIA0KDQo+LS0tLS1PcmlnaW5hbCBNZXNzYWdlLS0tLS0NCj5Gcm9tOiBpbnRlbC1ndnQtZGV2 IFttYWlsdG86aW50ZWwtZ3Z0LWRldi1ib3VuY2VzQGxpc3RzLmZyZWVkZXNrdG9wLm9yZ10gT24N Cj5CZWhhbGYgT2YgR2VyZCBIb2ZmbWFubg0KPlNlbnQ6IEZyaWRheSwgSnVuZSAwMiwgMjAxNyAx MToyNCBQTQ0KPlRvOiBBbGV4IFdpbGxpYW1zb24gPGFsZXgud2lsbGlhbXNvbkByZWRoYXQuY29t PjsgQ2hlbiwgWGlhb2d1YW5nDQo+PHhpYW9ndWFuZy5jaGVuQGludGVsLmNvbT4NCj5DYzogVGlh biwgS2V2aW4gPGtldmluLnRpYW5AaW50ZWwuY29tPjsgaW50ZWwtZ2Z4QGxpc3RzLmZyZWVkZXNr dG9wLm9yZzsgbGludXgtDQo+a2VybmVsQHZnZXIua2VybmVsLm9yZzsgemhlbnl1d0BsaW51eC5p bnRlbC5jb207IGNocmlzQGNocmlzLXdpbHNvbi5jby51azsgTHYsDQo+WmhpeXVhbiA8emhpeXVh bi5sdkBpbnRlbC5jb20+OyBpbnRlbC1ndnQtZGV2QGxpc3RzLmZyZWVkZXNrdG9wLm9yZzsgV2Fu ZywgWmhpDQo+QSA8emhpLmEud2FuZ0BpbnRlbC5jb20+DQo+U3ViamVjdDogUmU6IFtQQVRDSCB2 NiA2LzZdIGRybS9pOTE1L2d2dDogQWRkaW5nIGludGVyZmFjZSBzbyB1c2VyIHNwYWNlIGNhbiBn ZXQNCj50aGUgZG1hLWJ1Zg0KPg0KPiAgSGksDQo+DQo+PiA+IFdoZW4gaTkxNSdzIGRtYS1idWYn cyByZWxlYXNlKCkgY2FsbGJhY2sgaXMgY2FsbGVkIGl0IHdpbGwgdHJ5IHRvDQo+PiA+IGZyZWUg dGhlIGdlbSBvYmplY3QgYXNzb2NpYXRlZCB3aXRoIHRoZSBkbWEtYnVmIGlmIGl0cyByZWYgY291 bnQgaXMNCj4+ID4gMC4gQnV0IGluIG91ciBjYXNlIHRoZSByZWYgY291bnQgaXMgMSBzbyBubyBm cmVlIGNhbGxiYWNrIGlzIGNhbGxlZA0KPj4gPiBzbyB3ZSBjYW4gbm90IHJlbGVhc2UgYWxsb2Nh dGlvbnMgdGhlcmUuDQo+DQo+V2h5IHRoZSByZWYgY291bnQgaXMgb25lPyAgDQpUaGUgZ2VtIG9i amVjdCBpcyBjcmVhdGVkIGJ5IHVzIHdoaWxlIGNyZWF0aW5nIHRoZSBkbWEtYnVmKHRoZSByZWYg Y291bnQgb2YgdGhlIGdlbSBvYmplY3QgaXMgaW5pdGlhbGl6ZWQgdG8gMSkuDQpMYXRlciB3aGVu IHVzZXIgaW1wb3J0IHRoZSBkbWEtYnVmIHRoZSByZWYgY291bnQgb2YgdGhlIGdlbSBvYmplY3Qg YXNzb2NpYXRlIHdpdGggdGhlIGRtYS1idWYgd2lsbCBpbmNyZWFzZWQuDQpXaGVuIHVzZXIgZmlu aXNoZWQgdXNpbmcgdGhlIGRtYS1idWYgaXQgd2lsbCBkZWNyZWFzZSB0aGUgcmVmIGNvdW50Lg0K QnV0IHRoZSByZWYgY291bnQgb2YgdGhlIGdlbSBvYmplY3Qgd2lsbCBiZWNvbWUgMSB3aGVuIGFs bCB0aGUgdXNlciBmaW5pc2hlZCB1c2luZyB0aGUgZG1hLWJ1ZiBiZWNhdXNlIHdlIGNyZWF0ZSB0 aGUgZ2VtIG9iamVjdCh0aGUgdGVzdCBhbHNvIHNob3dpbmcgdGhpcyByZXN1bHQpLg0KDQpUeXBp Y2FsbHkgdXNlciBvbmx5IGV4cG9ydCBhIGRtYS1idWYobm8gZ2VtIG9iamVjdCB5ZXQpIHRoZW4g d2hlbiB1c2VyIGltcG9ydCB0aGUgZG1hLWJ1ZiB0aGVuIGEgZ2VtIG9iamVjdCB3aWxsIGJlIGNy ZWF0ZWQuDQpCdXQgaW4gb3VyIGNhc2Ugd2UgZG8gbm90IGltcGxlbWVudCB0aGUgZG1hLWJ1ZiBm cm9tIHNjcmF0Y2ggYnV0IGNhbGxpbmcgdGhlIGk5MTVfZ2VtX3ByaW1lX2V4cG9ydCgpIHdoZXJl IGEgZ2VtIG9iamVjdCBpcyBhbiBpbnB1dCBwYXJhbWV0ZXIuDQoNCkNoZW54Zw0KDQoNCj5XaG8g aG9sZHMgYSByZWZlcmVuY2UgYW5kIHdoeT8NCj5NYXliZSBpdCBzaG91bGQgYmUgdGhlIG90aGVy IHdheSBhcm91bmQsIGkuZS4gdGhlIGRtYWJ1ZiBob2xkcyBhIHJlZmVyZW5jZSBvbg0KPnRoZSB2 Z3B1IGluc3RhbmNlIGJhY2tpbmcgaXQsIGkuZS4geW91IGNhbid0IGRlbGV0ZSB0aGUgdmdwdSB3 aGlsZSBkbWEtYnVmcyBleGlzdD8NCj4NCj4+IFdlIGNhbm5vdCBzaW1wbHkgc2F5IHRoYXQgdGhl IHVzZXIgaXNuJ3QgYWxsb3dlZCB0byByZWxlYXNlIHRoZW0gaW4NCj4+IHRoYXQgb3JkZXIuDQo+ DQo+WWVwLCBub3QgZ29pbmcgdG8gZmx5LiAgQ2FuIGhhcHBlbiBldmVuIHVuaW50ZW50aW9uYWxs eSBiZWNhdXNlIHdlIGNhbiBwYXNzDQo+YXJvdW5kIGRtYWJ1ZnMgdG8gb3RoZXIgcHJvY2Vzc2Vz LiAgRXhhbXBsZTogcWVtdSBwYXNzZXMgZG1hYnVmIHRvIHNwaWNlLQ0KPmNsaWVudCwgdGhlbiBx ZW11IGNyYXNoZXMuICBtZ210IGZkIGlzIGNsb3NlZCBiZWZvcmUgZG1hYnVmIGZkIHRoZW4uICBU aGUga2VybmVsDQo+bXVzdCBiZSBhYmxlIHRvIGhhbmRsZSB0aGF0Lg0KPg0KPmNoZWVycywNCj4g IEdlcmQNCj5fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXw0K PmludGVsLWd2dC1kZXYgbWFpbGluZyBsaXN0DQo+aW50ZWwtZ3Z0LWRldkBsaXN0cy5mcmVlZGVz a3RvcC5vcmcNCj5odHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZv L2ludGVsLWd2dC1kZXYNCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fCkludGVsLWdmeCBtYWlsaW5nIGxpc3QKSW50ZWwtZ2Z4QGxpc3RzLmZyZWVkZXNrdG9w Lm9yZwpodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2ludGVs LWdmeAo=