From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?Q?Christian_K=c3=b6nig?= Subject: Re: [PATCH 3/5] dma-buf: lock the reservation object during (un)map_dma_buf Date: Mon, 18 Jun 2018 14:43:11 +0200 Message-ID: <832d4f04-ced2-ce9f-723d-d611e8241e07@gmail.com> References: <20180601120020.11520-1-christian.koenig@amd.com> <20180601120020.11520-3-christian.koenig@amd.com> <20180618082224.GW3438@phenom.ffwll.local> Reply-To: christian.koenig-5C7GfCeVMHo@public.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20180618082224.GW3438-dv86pmgwkMBes7Z6vYuT8azUEOm+Xw19@public.gmane.org> Content-Language: en-US List-Id: Discussion list for AMD gfx List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: amd-gfx-bounces-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org Sender: "amd-gfx" To: Daniel Vetter Cc: linaro-mm-sig-cunTk1MwBs8s++Sfvej+rw@public.gmane.org, amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org, dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org, linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org QW0gMTguMDYuMjAxOCB1bSAxMDoyMiBzY2hyaWViIERhbmllbCBWZXR0ZXI6Cj4gT24gRnJpLCBK dW4gMDEsIDIwMTggYXQgMDI6MDA6MThQTSArMDIwMCwgQ2hyaXN0aWFuIEvDtm5pZyB3cm90ZToK Pj4gRmlyc3Qgc3RlcCB0b3dhcmRzIHVucGlubmVkIERNQSBidWYgb3BlcmF0aW9uLgo+Pgo+PiBJ J3ZlIGNoZWNrZWQgdGhlIERSTSBkcml2ZXJzIHRvIHBvdGVudGlhbCBsb2NraW5nIG9mIHRoZSBy ZXNlcnZhdGlvbgo+PiBvYmplY3QsIGJ1dCBlc3NlbnRpYWxseSB3ZSBuZWVkIHRvIGF1ZGl0IGFs bCBpbXBsZW1lbnRhdGlvbnMgb2YgdGhlCj4+IGRtYV9idWYgX29wcyBmb3IgdGhpcyB0byB3b3Jr Lgo+Pgo+PiBTaWduZWQtb2ZmLWJ5OiBDaHJpc3RpYW4gS8O2bmlnIDxjaHJpc3RpYW4ua29lbmln QGFtZC5jb20+Cj4gQWdyZWVkIGluIHByaW5jaXBsZSwgYnV0IEkgZXhwZWN0IGEgZmlyZXdvcmtz IHNob3cgd2l0aCBqdXN0IHRoaXMgcGF0Y2gKPiBhcHBsaWVkLiBJdCdzIG5vdCBqdXN0IHRoYXQg d2UgbmVlZCB0byBhdWRpdCBhbGwgdGhlIGltcGxlbWVudGF0aW9ucyBvZgo+IGRtYS1idWYtb3Bz LCB3ZSBhbHNvIG5lZWQgdG8gYXVkaXQgYWxsIHRoZSBjYWxsZXJzLgoKQWgsIHllYWggb2YgY291 cnNlIGEgZ29vZCBwb2ludC4KCj4gTm8gaWRlYSB5ZXQgaG93IHRvIGdvIGFib3V0IG1lcmdpbmcg dGhpcywgYnV0IGZvciBhIHN0YXJ0IG1pZ2h0IGJlIGdvb2QgdG8KPiB0aHJvdyB0aGlzIGF0IHRo ZSBpbnRlbC1nZnggQ0kgKGp1c3QgQ2M6IHRoZSBpbnRlbC1nZnggbWFpbGluZyBsaXN0cywgYnV0 Cj4gbWFrZSBzdXJlIHlvdXIgc2VyaWVzIGFwcGxpZXMgd2l0aG91dCBhbWQtc3RhZ2luZy1uZXh0 IHN0dWZmIHdoaWNoIGlzbid0Cj4gaW4gZHJtLmdpdCB5ZXQpLgoKT2ssIGdvaW5nIHRvIGluY29y cG9yYXRlIGFsbCB5b3VyIG90aGVyIGNvbW1lbnRzIGFzIHdlbGwgYW5kIHRoZW4gc2VuZCAKdGhl IG5leHQgcm91bmQgb2YgdGhpcyB3aXRoIENDaW5nIGludGVsLWdmeCBhcyB3ZWxsLgoKVGhhbmtz IGZvciB0aGUgcmV2aWV3LApDaHJpc3RpYW4uCgo+IC1EYW5pZWwKPgo+PiAtLS0KPj4gICBkcml2 ZXJzL2RtYS1idWYvZG1hLWJ1Zi5jIHwgNCArKysrCj4+ICAgaW5jbHVkZS9saW51eC9kbWEtYnVm LmggICB8IDQgKysrKwo+PiAgIDIgZmlsZXMgY2hhbmdlZCwgOCBpbnNlcnRpb25zKCspCj4+Cj4+ IGRpZmYgLS1naXQgYS9kcml2ZXJzL2RtYS1idWYvZG1hLWJ1Zi5jIGIvZHJpdmVycy9kbWEtYnVm L2RtYS1idWYuYwo+PiBpbmRleCBlNGM2NTdkOWZhZDcuLjRmMDcwOGNiNThhNyAxMDA2NDQKPj4g LS0tIGEvZHJpdmVycy9kbWEtYnVmL2RtYS1idWYuYwo+PiArKysgYi9kcml2ZXJzL2RtYS1idWYv ZG1hLWJ1Zi5jCj4+IEBAIC02MzEsNyArNjMxLDkgQEAgc3RydWN0IHNnX3RhYmxlICpkbWFfYnVm X21hcF9hdHRhY2htZW50KHN0cnVjdCBkbWFfYnVmX2F0dGFjaG1lbnQgKmF0dGFjaCwKPj4gICAJ aWYgKFdBUk5fT04oIWF0dGFjaCB8fCAhYXR0YWNoLT5kbWFidWYpKQo+PiAgIAkJcmV0dXJuIEVS Ul9QVFIoLUVJTlZBTCk7Cj4+ICAgCj4+ICsJcmVzZXJ2YXRpb25fb2JqZWN0X2xvY2soYXR0YWNo LT5kbWFidWYtPnJlc3YsIE5VTEwpOwo+PiAgIAlzZ190YWJsZSA9IGF0dGFjaC0+ZG1hYnVmLT5v cHMtPm1hcF9kbWFfYnVmKGF0dGFjaCwgZGlyZWN0aW9uKTsKPj4gKwlyZXNlcnZhdGlvbl9vYmpl Y3RfdW5sb2NrKGF0dGFjaC0+ZG1hYnVmLT5yZXN2KTsKPj4gICAJaWYgKCFzZ190YWJsZSkKPj4g ICAJCXNnX3RhYmxlID0gRVJSX1BUUigtRU5PTUVNKTsKPj4gICAKPj4gQEAgLTY1OCw4ICs2NjAs MTAgQEAgdm9pZCBkbWFfYnVmX3VubWFwX2F0dGFjaG1lbnQoc3RydWN0IGRtYV9idWZfYXR0YWNo bWVudCAqYXR0YWNoLAo+PiAgIAlpZiAoV0FSTl9PTighYXR0YWNoIHx8ICFhdHRhY2gtPmRtYWJ1 ZiB8fCAhc2dfdGFibGUpKQo+PiAgIAkJcmV0dXJuOwo+PiAgIAo+PiArCXJlc2VydmF0aW9uX29i amVjdF9sb2NrKGF0dGFjaC0+ZG1hYnVmLT5yZXN2LCBOVUxMKTsKPj4gICAJYXR0YWNoLT5kbWFi dWYtPm9wcy0+dW5tYXBfZG1hX2J1ZihhdHRhY2gsIHNnX3RhYmxlLAo+PiAgIAkJCQkJCWRpcmVj dGlvbik7Cj4+ICsJcmVzZXJ2YXRpb25fb2JqZWN0X3VubG9jayhhdHRhY2gtPmRtYWJ1Zi0+cmVz dik7Cj4+ICAgfQo+PiAgIEVYUE9SVF9TWU1CT0xfR1BMKGRtYV9idWZfdW5tYXBfYXR0YWNobWVu dCk7Cj4+ICAgCj4+IGRpZmYgLS1naXQgYS9pbmNsdWRlL2xpbnV4L2RtYS1idWYuaCBiL2luY2x1 ZGUvbGludXgvZG1hLWJ1Zi5oCj4+IGluZGV4IGQxN2NhZGQ3NjgwMi4uZDJiYTdhMDI3YTc4IDEw MDY0NAo+PiAtLS0gYS9pbmNsdWRlL2xpbnV4L2RtYS1idWYuaAo+PiArKysgYi9pbmNsdWRlL2xp bnV4L2RtYS1idWYuaAo+PiBAQCAtMTE4LDYgKzExOCw4IEBAIHN0cnVjdCBkbWFfYnVmX29wcyB7 Cj4+ICAgCSAqIGFueSBvdGhlciBraW5kIG9mIHNoYXJpbmcgdGhhdCB0aGUgZXhwb3J0ZXIgbWln aHQgd2lzaCB0byBtYWtlCj4+ICAgCSAqIGF2YWlsYWJsZSB0byBidWZmZXItdXNlcnMuCj4+ICAg CSAqCj4+ICsJICogVGhpcyBpcyBjYWxsZWQgd2l0aCB0aGUgZG1hYnVmLT5yZXN2IG9iamVjdCBs b2NrZWQuCj4+ICsJICoKPj4gICAJICogUmV0dXJuczoKPj4gICAJICoKPj4gICAJICogQSAmc2df dGFibGUgc2NhdHRlciBsaXN0IG9mIG9yIHRoZSBiYWNraW5nIHN0b3JhZ2Ugb2YgdGhlIERNQSBi dWZmZXIsCj4+IEBAIC0xMzgsNiArMTQwLDggQEAgc3RydWN0IGRtYV9idWZfb3BzIHsKPj4gICAJ ICogSXQgc2hvdWxkIGFsc28gdW5waW4gdGhlIGJhY2tpbmcgc3RvcmFnZSBpZiB0aGlzIGlzIHRo ZSBsYXN0IG1hcHBpbmcKPj4gICAJICogb2YgdGhlIERNQSBidWZmZXIsIGl0IHRoZSBleHBvcnRl ciBzdXBwb3J0cyBiYWNraW5nIHN0b3JhZ2UKPj4gICAJICogbWlncmF0aW9uLgo+PiArCSAqCj4+ ICsJICogVGhpcyBpcyBjYWxsZWQgd2l0aCB0aGUgZG1hYnVmLT5yZXN2IG9iamVjdCBsb2NrZWQu Cj4+ICAgCSAqLwo+PiAgIAl2b2lkICgqdW5tYXBfZG1hX2J1Zikoc3RydWN0IGRtYV9idWZfYXR0 YWNobWVudCAqLAo+PiAgIAkJCSAgICAgIHN0cnVjdCBzZ190YWJsZSAqLAo+PiAtLSAKPj4gMi4x NC4xCj4+Cj4+IF9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f Cj4+IGRyaS1kZXZlbCBtYWlsaW5nIGxpc3QKPj4gZHJpLWRldmVsQGxpc3RzLmZyZWVkZXNrdG9w Lm9yZwo+PiBodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2Ry aS1kZXZlbAoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18K YW1kLWdmeCBtYWlsaW5nIGxpc3QKYW1kLWdmeEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6 Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9hbWQtZ2Z4Cg==