From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH 2/4] dma-buf: lock the reservation object during (un)map_dma_buf v2 Date: Tue, 3 Jul 2018 15:11:15 +0200 Message-ID: <20180703131115.GE3891@phenom.ffwll.local> References: <20180622141103.1787-1-christian.koenig@amd.com> <20180622141103.1787-3-christian.koenig@amd.com> <20180625082231.GM2958@phenom.ffwll.local> <20180625091217.GO2958@phenom.ffwll.local> <2ebff18b-414d-c971-1b7f-f6a21aacf196@gmail.com> <20180703125235.GB3891@phenom.ffwll.local> <35187534-cc89-3e31-5428-2700c3f8a90b@amd.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: <35187534-cc89-3e31-5428-2700c3f8a90b@amd.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Christian =?iso-8859-1?Q?K=F6nig?= Cc: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, linaro-mm-sig@lists.linaro.org, sumit.semwal@linaro.org, linux-media@vger.kernel.org List-Id: dri-devel@lists.freedesktop.org T24gVHVlLCBKdWwgMDMsIDIwMTggYXQgMDM6MDI6MTFQTSArMDIwMCwgQ2hyaXN0aWFuIEvDtm5p ZyB3cm90ZToKPiBBbSAwMy4wNy4yMDE4IHVtIDE0OjUyIHNjaHJpZWIgRGFuaWVsIFZldHRlcjoK PiA+IE9uIFR1ZSwgSnVsIDAzLCAyMDE4IGF0IDAxOjQ2OjQ0UE0gKzAyMDAsIENocmlzdGlhbiBL w7ZuaWcgd3JvdGU6Cj4gPiA+IEFtIDI1LjA2LjIwMTggdW0gMTE6MTIgc2NocmllYiBEYW5pZWwg VmV0dGVyOgo+ID4gPiA+IE9uIE1vbiwgSnVuIDI1LCAyMDE4IGF0IDEwOjIyOjMxQU0gKzAyMDAs IERhbmllbCBWZXR0ZXIgd3JvdGU6Cj4gPiA+ID4gPiBPbiBGcmksIEp1biAyMiwgMjAxOCBhdCAw NDoxMTowMVBNICswMjAwLCBDaHJpc3RpYW4gS8O2bmlnIHdyb3RlOgo+ID4gPiA+ID4gPiBGaXJz dCBzdGVwIHRvd2FyZHMgdW5waW5uZWQgRE1BIGJ1ZiBvcGVyYXRpb24uCj4gPiA+ID4gPiA+IAo+ ID4gPiA+ID4gPiBJJ3ZlIGNoZWNrZWQgdGhlIERSTSBkcml2ZXJzIHRvIHBvdGVudGlhbCBsb2Nr aW5nIG9mIHRoZSByZXNlcnZhdGlvbgo+ID4gPiA+ID4gPiBvYmplY3QsIGJ1dCBlc3NlbnRpYWxs eSB3ZSBuZWVkIHRvIGF1ZGl0IGFsbCBpbXBsZW1lbnRhdGlvbnMgb2YgdGhlCj4gPiA+ID4gPiA+ IGRtYV9idWYgX29wcyBmb3IgdGhpcyB0byB3b3JrLgo+ID4gPiA+ID4gPiAKPiA+ID4gPiA+ID4g djI6IHJlb3JkZXJlZAo+ID4gPiA+ID4gPiAKPiA+ID4gPiA+ID4gU2lnbmVkLW9mZi1ieTogQ2hy aXN0aWFuIEvDtm5pZyA8Y2hyaXN0aWFuLmtvZW5pZ0BhbWQuY29tPgo+ID4gPiA+ID4gUmV2aWV3 ZWQtYnk6IERhbmllbCBWZXR0ZXIgPGRhbmllbC52ZXR0ZXJAZmZ3bGwuY2g+Cj4gPiA+ID4gT2sg SSBkaWQgcmV2aWV3IGRyaXZlcnMgYSBiaXQsIGJ1dCBhcHBhcmVudGx5IG5vdCB3ZWxsIGVub3Vn aCBieSBmYXIuIGk5MTUKPiA+ID4gPiBDSSBpcyB1bmhhcHB5Ogo+ID4gPiA+IAo+ID4gPiA+IGh0 dHBzOi8vaW50ZWwtZ2Z4LWNpLjAxLm9yZy90cmVlL2RybS10aXAvUGF0Y2h3b3JrXzk0MDAvZmkt d2hsLXUvaWd0QGdlbV9tbWFwX2d0dEBiYXNpYy1zbWFsbC1iby10aWxlZHguaHRtbAo+ID4gPiA+ IAo+ID4gPiA+IFNvIHllYWggaW5zZXJ0aW5nIHRoYXQgbG9jayBpbiB0aGVyZSBpc24ndCB0aGUg bW9zdCB0cml2aWFsIHRoaW5nIDotLwo+ID4gPiA+IAo+ID4gPiA+IEkga2luZGEgYXNzdW1lIHRo YXQgb3RoZXIgZHJpdmVycyB3aWxsIGhhdmUgc2ltaWxhciBpc3N1ZXMsIGUuZy4gb21hcGRybSdz Cj4gPiA+ID4gdXNlIG9mIGRldi0+c3RydWN0X211dGV4IGFsc28gdmVyeSBtdWNoIGxvb2tzIGxp a2UgaXQnbGwgcmVzdWx0IGluIGEgbmV3Cj4gPiA+ID4gbG9ja2luZyBpbnZlcnNpb24uCj4gPiA+ IEFoLCBjcmFwLiBBbHJlYWR5IGZlYXJlZCB0aGF0IHRoaXMgd291bGRuJ3QgYmUgZWFzeSwgYnV0 IHllYWggdGhhdCBpdCBpcyBhcwo+ID4gPiBiYWQgYXMgdGhpcyBpcyByYXRoZXIgZGlzYXBwb2lu dGluZy4KPiA+ID4gCj4gPiA+IFRoYW5rcyBmb3IgdGhlIGluZm8sIGdvaW5nIHRvIGtlZXAgdGhp bmtpbmcgYWJvdXQgaG93IHRvIHNvbHZlIHRob3NlIGlzc3Vlcy4KPiA+IFNpZGUgbm90ZTogV2Ug d2FudCB0byBtYWtlIHN1cmUgdGhhdCBkcml2ZXJzIGRvbid0IGdldCB0aGUgcmVzZXJ2YXRpb25f b2JqCj4gPiBsb2NraW5nIGhpZXJhcmNoeSB3cm9uZyBpbiBvdGhlciBwbGFjZXMgKHVzaW5nIGRl di0+c3RydWN0X211dGV4IGlzIGtpbmRhCj4gPiBhIHByZS1leGlzdGluZyBtaXMtdXNlIHRoYXQg d2UgY2FuJ3Qgd2lzaCBhd2F5IHJldHJvYWN0aXZlbHkKPiA+IHVuZm9ydHVuYXRlbHkpLiBPbmUg cmVhbGx5IGltcG9ydGFudCB0aGluZyBpcyB0aGF0IHNocmlua2VyIHZzIHJlc3Zfb2JqCj4gPiBt dXN0IHdvcmsgd2l0aCB0cnlsb2NrcyBpbiB0aGUgc2hyaW5rZXIsIHNvIHRoYXQgeW91IGNhbiBh bGxvY2F0ZSBtZW1vcnkKPiA+IHdoaWxlIGhvbGRpbmcgcmVzZXJ2YXRpb24gb2JqZWN0cy4KPiA+ IAo+ID4gT25lIG5lYXQgdHJpY2sgdG8gdGVhY2ggbG9ja2RlcCBhYm91dCB0aGlzIHdvdWxkIGJl IHRvIGhhdmUgYSBkdW1teQo+ID4gCj4gPiBpZiAoSVNfRU5BQkxFRChDT05GSUdfUFJPVkVfTE9D S0lORykpIHsKPiA+IAl3d19tdXRleF9sb2NrKGRtYV9idWYtPnJlc3Zfb2JqKTsKPiA+IAlmc19y ZWNsYWltX2FjcXVpcmUoR0ZQX0tFUk5FTCk7Cj4gPiAJZnNfcmVjbGFpbV9yZWxlYXNlKEdGUF9L RVJORUwpOwo+ID4gCXd3X211dGV4X3VubG9jayhkbWFfYnVmLT5yZXN2X29iaik7Cj4gPiB9Cj4g PiAKPiA+IGluIGRtYV9idWZfaW5pdCgpLiBXZSdyZSB1c2luZyB0aGUgZnNfcmVjbGFpbV9hY3F1 aXJlL3JlbGVhc2UgY2hlY2sgdmVyeQo+ID4gc3VjY2Vzc2Z1bGx5IHRvIGltcHJvdmUgb3VyIGln dCB0ZXN0IGNvdmVyYWdlIGZvciBpOTE1LmtvIGluIG90aGVyIGFyZWFzLgo+ID4gCj4gPiBUb3Rh bGx5IHVucmVsYXRlZCB0byBkZXYtPnN0cnVjdF9tdXRleCwgYnV0IHRob3VnaHRzPyBXZWxsIGZv cgo+ID4gZGV2LT5zdHJ1Y3RfbXV0ZXggd2UgY291bGQgYXQgbGVhc3QgZGVjaWRlIG9uIG9uZSB0 cnVlIHdheSB0byBuZXN0Cj4gPiByZXN2X29iaiB2cy4gZGV2LT5zdHJ1Y3RfbXV0ZXggYXMgbWF5 YmUgYW4gaW50ZXJpbSBzdGVwLCBidXQgbm90IHN1cmUgaG93Cj4gPiBtdWNoIHRoYXQgd291bGQg aGVscC4KPiAKPiBJIGRvbid0IHRoaW5rIHRoYXQgd291bGQgaGVscC4gQXMgZmFyIGFzIEkgY2Fu IHNlZSB3ZSBvbmx5IGhhdmUgdHdvIGNob2ljZXM6Cj4gCj4gMS4gRWl0aGVyIGhhdmUgYSBiaWcg cGF0Y2ggd2hpY2ggZml4ZXMgYWxsIERNQS1idWYgaW1wbGVtZW50YXRpb25zIHRvIGFsbG93Cj4g dGhlIHJlc2VydmF0aW9uIGxvY2sgdG8gYmUgaGVsZCBkdXJpbmcgbWFwL3VubWFwICh1bnJlYWxp c3RpYykuCj4gCj4gMi4gQWRkIGEgZmxhZyB0byBhdCBsZWFzdCBpbiB0aGUgbWlkIHRlcm0gdGVs bCB0aGUgRE1BLWJ1ZiBoZWxwZXIgZnVuY3Rpb25zCj4gd2hhdCB0byBkby4gRS5nLiBjcmVhdGUg dGhlIG1hcHBpbmcgd2l0aG91dCB0aGUgcmVzZXJ2YXRpb24gbG9jayBoZWxkLgo+IAo+IAo+IEhv dyBhYm91dCBtb3ZpbmcgdGhlIFNHTCBjYWNoaW5nIGZyb20gdGhlIERSTSBsYXllciBpbnRvIHRo ZSBETUEtYnVmIGxheWVyCj4gYW5kIGFkZCBhIGZsYWcgaWYgdGhlIGV4cG9ydGVyIHdhbnRzL25l ZWRzIHRoaXMgY2FjaGluZz8KPiAKPiBUaGVuIG9ubHkgdGhlIGltcGxlbWVudGF0aW9ucyB3aGlj aCBjYW4gZGVhbCB3aXRoIGR5bmFtaWMgaW52YWxpZGF0aW9uCj4gZGlzYWJsZSBTR0wgY2FjaGlu ZyBhbmQgd2l0aCBpdCBlbmFibGUgY3JlYXRpbmcgdGhlIHNnbCB3aXRoIHRoZSByZXNlcnZhdGlv bgo+IG9iamVjdCBsb2NrZWQuCj4gCj4gVGhpcyB3YXkgd2UgY2FuIGtpbGwgdHdvIGJpcmRzIHdp dGggb25lIHN0b25lIGJ5IGJvdGggYXZvaWRpbmcgdGhlIFNHTAo+IGNhY2hpbmcgaW4gdGhlIERS TSBsYXllciBhcyB3ZWxsIGFzIGhhdmluZyBhIHNhbmUgaGFuZGxpbmcgZm9yIHRoZSBsb2NraW5n Lgo+IAo+IFRob3VnaHRzPwoKSSBkb24ndCBzZWUgaG93IHRoZSBTR0wgc3R1ZmYgZmFjdG9ycyBp bnRvIG5laXRoZXIgdGhlIGRldi0+c3RydWN0X211dGV4Cm5vciBpbnRvIHRoZSBuZWVkIHRvIGRv IGFsbG9jYXRpb25zIHdoaWxlIGhvbGRpbmcgcmVzdl9vYmouIE5laXRoZXIKY2hhbmdlcyBieSBt b3ZpbmcgdGhhdCBwaWVjZSBhcm91bmQuIEF0IGxlYXN0IGFzIGZhciBhcyBJIGNhbiBzZWUgaXQg U0dMCmNhY2hpbmcgaXMgZnVsbHkgb3J0aG9nb25hbCB0byBhbnkga2luZCBvZiBsb2NraW5nIGZ1 bi4KCldoeSBkbyB5b3Ugc2VlIGEgY29ubmVjdGlvbiBoZXJlPwotRGFuaWVsCi0tIApEYW5pZWwg VmV0dGVyClNvZnR3YXJlIEVuZ2luZWVyLCBJbnRlbCBDb3Jwb3JhdGlvbgpodHRwOi8vYmxvZy5m ZndsbC5jaApfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpJ bnRlbC1nZnggbWFpbGluZyBsaXN0CkludGVsLWdmeEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0 cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9pbnRlbC1nZngK