From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?Q?Christian_K=c3=b6nig?= Subject: Re: [PATCH 13/17] drm/radeon: Remove custom dma_fence_ops->wait implementation Date: Mon, 30 Apr 2018 20:26:14 +0200 Message-ID: References: <20180427061724.28497-1-daniel.vetter@ffwll.ch> <20180427061724.28497-14-daniel.vetter@ffwll.ch> <27a1ffc4-857e-79e1-1d95-40b21709a2b0@amd.com> <20180430153854.GR12521@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: <20180430153854.GR12521-dv86pmgwkMBes7Z6vYuT8azUEOm+Xw19@public.gmane.org> Content-Language: en-US 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 , =?UTF-8?Q?Christian_K=c3=b6nig?= Cc: Alex Deucher , Daniel Vetter , Intel Graphics Development , DRI Development , amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org List-Id: dri-devel@lists.freedesktop.org QW0gMzAuMDQuMjAxOCB1bSAxNzozOCBzY2hyaWViIERhbmllbCBWZXR0ZXI6Cj4gT24gU3VuLCBB cHIgMjksIDIwMTggYXQgMDk6MDg6MzFBTSArMDIwMCwgQ2hyaXN0aWFuIEvDtm5pZyB3cm90ZToK Pj4gTkFLLCB0aGVyZSBpcyBhIHN1YnRpdGxlIGJ1dCBtYWpvciBkaWZmZXJlbmNlOgo+Pgo+Pj4g LQkJaWYgKHJkZXYtPm5lZWRzX3Jlc2V0KSB7Cj4+PiAtCQkJdCA9IC1FREVBRExLOwo+Pj4gLQkJ CWJyZWFrOwo+Pj4gLQkJfQo+PiBXaXRob3V0IHRoYXQgdGhlIHdob2xlIHJhZGVvbiBHUFUgcmVz ZXQgY29kZSBicmVha3MuCj4gT29wcyBJJ3ZlIG1pc3NlZCB0aGF0LiBIb3cgZG9lcyB0aGlzIHdv cmsgd2hlbiB5b3UgcmVnaXN0ZXIgYSBjYWxsYmFjawo+IHVzaW5nIC0+ZW5hYmxlX3NpZ25hbGlu ZyBhbmQgdGhlbiBibG9jayBvbiBpdD8gRXZlcnl0aGluZyBqdXN0IGRpZXM/CgpUaGUgc2hvcnQg YW5zd2VyIGlzIHdlIHNpbXBseSBhdm9pZCB1c2luZyBlbmFibGVfc2lnbmFsaW5nKCkgZnJvbSBp bnNpZGUgCmRyaXZlciBJT0NUTHMuCgo+IFdlIGhhdmUgbG90cyBvZiB1c2VycyBvZiB0aGF0IGZv ciBidWZmZXIvZmVuY2Ugc2hhcmluZy4gQSByZWFsbHkgdWdseSwgYnV0Cj4gcHJvYmFibHkgd29y a2luZyBmaXggZm9yIHRoaXMgd291bGQgYmUgYSBrdGhyZWFkIHdvcmtlciB0aGF0IGp1c3QgbG9v a3MKPiBmb3IgLT5uZWVkc19yZXNldCBhbmQgZm9yY2UtY29tcGxldGVzIGFsbCBmZW5jZXMgd2l0 aAo+IGRtYV9mZW5jZV9zZXRfZXJyb3IoLUVJTyksIHdoaWNoIGlzIGtpbmRhIHdoYXQncyBzdXBw b3NlZCB0byBoYXBwZW4gaGVyZQo+IGFueXdheS4KClRoYXQgYWN0dWFsbHkgd29uJ3QgaGVscC4g UmFkZW9uIGRvZXMgdGhpcyBkYW5jZSB0byByZXR1cm4gYW4gZXJyb3IgZnJvbSAKZG1hX2ZlbmNl X3dhaXQoKSB3aGVuIHRoZSBHUFUgbmVlZHMgYSByZXNldC4KClRoaXMgd2F5IGFsbCBJT0NUTHMg c2hvdWxkIHJldHVybiB0byB1c2Vyc3BhY2Ugd2l0aCAtRUFHQUlOIGFuZCB3aGVuIAp0aGV5IGFy ZSByZXN0YXJ0ZWQgd2UgYmxvY2sgZm9yIHRoZSBydW5uaW5nIEdQVSByZXNldCB0byBmaW5pc2gu CgpJIHdhcyBhZ2FpbnN0IHRoaXMgYXBwcm9hY2gsIGJ1dCBpdCB3b3JrcyBhcyBsb25nIGFzIHJh ZGVvbiBvbmx5IGhhcyB0byAKZGVhbCB3aXRoIGl0J3Mgb3duIGZlbmNlcy4KCkNocmlzdGlhbi4K Cj4gLURhbmllbAo+Cj4+IFJlZ2FyZHMsCj4+IENocmlzdGlhbi4KPj4KPj4KPj4gQW0gMjcuMDQu MjAxOCB1bSAwODoxNyBzY2hyaWViIERhbmllbCBWZXR0ZXI6Cj4+PiBJdCdzIGEgY29weSBvZiBk bWFfZmVuY2VfZGVmYXVsdF93YWl0LCB3cml0dGVuIHNsaWdodGx5IGRpZmZlcmVudGx5Lgo+Pj4K Pj4+IFNpZ25lZC1vZmYtYnk6IERhbmllbCBWZXR0ZXIgPGRhbmllbC52ZXR0ZXJAZmZ3bGwuY2g+ Cj4+PiBDYzogQWxleCBEZXVjaGVyIDxhbGV4YW5kZXIuZGV1Y2hlckBhbWQuY29tPgo+Pj4gQ2M6 ICJDaHJpc3RpYW4gS8O2bmlnIiA8Y2hyaXN0aWFuLmtvZW5pZ0BhbWQuY29tPgo+Pj4gQ2M6ICJE YXZpZCAoQ2h1bk1pbmcpIFpob3UiIDxEYXZpZDEuWmhvdUBhbWQuY29tPgo+Pj4gQ2M6IGFtZC1n ZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCj4+PiAtLS0KPj4+ICAgIGRyaXZlcnMvZ3B1L2RybS9y YWRlb24vcmFkZW9uX2ZlbmNlLmMgfCA2MyAtLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0KPj4+ ICAgIDEgZmlsZSBjaGFuZ2VkLCA2MyBkZWxldGlvbnMoLSkKPj4+Cj4+PiBkaWZmIC0tZ2l0IGEv ZHJpdmVycy9ncHUvZHJtL3JhZGVvbi9yYWRlb25fZmVuY2UuYyBiL2RyaXZlcnMvZ3B1L2RybS9y YWRlb24vcmFkZW9uX2ZlbmNlLmMKPj4+IGluZGV4IGU4NmYyYmQzODQxMC4uMzI2OTBhNTI1YmZj IDEwMDY0NAo+Pj4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL3JhZGVvbi9yYWRlb25fZmVuY2UuYwo+ Pj4gKysrIGIvZHJpdmVycy9ncHUvZHJtL3JhZGVvbi9yYWRlb25fZmVuY2UuYwo+Pj4gQEAgLTEw NTEsNzIgKzEwNTEsOSBAQCBzdGF0aWMgY29uc3QgY2hhciAqcmFkZW9uX2ZlbmNlX2dldF90aW1l bGluZV9uYW1lKHN0cnVjdCBkbWFfZmVuY2UgKmYpCj4+PiAgICAJfQo+Pj4gICAgfQo+Pj4gLXN0 YXRpYyBpbmxpbmUgYm9vbCByYWRlb25fdGVzdF9zaWduYWxlZChzdHJ1Y3QgcmFkZW9uX2ZlbmNl ICpmZW5jZSkKPj4+IC17Cj4+PiAtCXJldHVybiB0ZXN0X2JpdChETUFfRkVOQ0VfRkxBR19TSUdO QUxFRF9CSVQsICZmZW5jZS0+YmFzZS5mbGFncyk7Cj4+PiAtfQo+Pj4gLQo+Pj4gLXN0cnVjdCBy YWRlb25fd2FpdF9jYiB7Cj4+PiAtCXN0cnVjdCBkbWFfZmVuY2VfY2IgYmFzZTsKPj4+IC0Jc3Ry dWN0IHRhc2tfc3RydWN0ICp0YXNrOwo+Pj4gLX07Cj4+PiAtCj4+PiAtc3RhdGljIHZvaWQKPj4+ IC1yYWRlb25fZmVuY2Vfd2FpdF9jYihzdHJ1Y3QgZG1hX2ZlbmNlICpmZW5jZSwgc3RydWN0IGRt YV9mZW5jZV9jYiAqY2IpCj4+PiAtewo+Pj4gLQlzdHJ1Y3QgcmFkZW9uX3dhaXRfY2IgKndhaXQg PQo+Pj4gLQkJY29udGFpbmVyX29mKGNiLCBzdHJ1Y3QgcmFkZW9uX3dhaXRfY2IsIGJhc2UpOwo+ Pj4gLQo+Pj4gLQl3YWtlX3VwX3Byb2Nlc3Mod2FpdC0+dGFzayk7Cj4+PiAtfQo+Pj4gLQo+Pj4g LXN0YXRpYyBzaWduZWQgbG9uZyByYWRlb25fZmVuY2VfZGVmYXVsdF93YWl0KHN0cnVjdCBkbWFf ZmVuY2UgKmYsIGJvb2wgaW50ciwKPj4+IC0JCQkJCSAgICAgc2lnbmVkIGxvbmcgdCkKPj4+IC17 Cj4+PiAtCXN0cnVjdCByYWRlb25fZmVuY2UgKmZlbmNlID0gdG9fcmFkZW9uX2ZlbmNlKGYpOwo+ Pj4gLQlzdHJ1Y3QgcmFkZW9uX2RldmljZSAqcmRldiA9IGZlbmNlLT5yZGV2Owo+Pj4gLQlzdHJ1 Y3QgcmFkZW9uX3dhaXRfY2IgY2I7Cj4+PiAtCj4+PiAtCWNiLnRhc2sgPSBjdXJyZW50Owo+Pj4g LQo+Pj4gLQlpZiAoZG1hX2ZlbmNlX2FkZF9jYWxsYmFjayhmLCAmY2IuYmFzZSwgcmFkZW9uX2Zl bmNlX3dhaXRfY2IpKQo+Pj4gLQkJcmV0dXJuIHQ7Cj4+PiAtCj4+PiAtCXdoaWxlICh0ID4gMCkg ewo+Pj4gLQkJaWYgKGludHIpCj4+PiAtCQkJc2V0X2N1cnJlbnRfc3RhdGUoVEFTS19JTlRFUlJV UFRJQkxFKTsKPj4+IC0JCWVsc2UKPj4+IC0JCQlzZXRfY3VycmVudF9zdGF0ZShUQVNLX1VOSU5U RVJSVVBUSUJMRSk7Cj4+PiAtCj4+PiAtCQkvKgo+Pj4gLQkJICogcmFkZW9uX3Rlc3Rfc2lnbmFs ZWQgbXVzdCBiZSBjYWxsZWQgYWZ0ZXIKPj4+IC0JCSAqIHNldF9jdXJyZW50X3N0YXRlIHRvIHBy ZXZlbnQgYSByYWNlIHdpdGggd2FrZV91cF9wcm9jZXNzCj4+PiAtCQkgKi8KPj4+IC0JCWlmIChy YWRlb25fdGVzdF9zaWduYWxlZChmZW5jZSkpCj4+PiAtCQkJYnJlYWs7Cj4+PiAtCj4+PiAtCQlp ZiAocmRldi0+bmVlZHNfcmVzZXQpIHsKPj4+IC0JCQl0ID0gLUVERUFETEs7Cj4+PiAtCQkJYnJl YWs7Cj4+PiAtCQl9Cj4+PiAtCj4+PiAtCQl0ID0gc2NoZWR1bGVfdGltZW91dCh0KTsKPj4+IC0K Pj4+IC0JCWlmICh0ID4gMCAmJiBpbnRyICYmIHNpZ25hbF9wZW5kaW5nKGN1cnJlbnQpKQo+Pj4g LQkJCXQgPSAtRVJFU1RBUlRTWVM7Cj4+PiAtCX0KPj4+IC0KPj4+IC0JX19zZXRfY3VycmVudF9z dGF0ZShUQVNLX1JVTk5JTkcpOwo+Pj4gLQlkbWFfZmVuY2VfcmVtb3ZlX2NhbGxiYWNrKGYsICZj Yi5iYXNlKTsKPj4+IC0KPj4+IC0JcmV0dXJuIHQ7Cj4+PiAtfQo+Pj4gLQo+Pj4gICAgY29uc3Qg c3RydWN0IGRtYV9mZW5jZV9vcHMgcmFkZW9uX2ZlbmNlX29wcyA9IHsKPj4+ICAgIAkuZ2V0X2Ry aXZlcl9uYW1lID0gcmFkZW9uX2ZlbmNlX2dldF9kcml2ZXJfbmFtZSwKPj4+ICAgIAkuZ2V0X3Rp bWVsaW5lX25hbWUgPSByYWRlb25fZmVuY2VfZ2V0X3RpbWVsaW5lX25hbWUsCj4+PiAgICAJLmVu YWJsZV9zaWduYWxpbmcgPSByYWRlb25fZmVuY2VfZW5hYmxlX3NpZ25hbGluZywKPj4+ICAgIAku c2lnbmFsZWQgPSByYWRlb25fZmVuY2VfaXNfc2lnbmFsZWQsCj4+PiAtCS53YWl0ID0gcmFkZW9u X2ZlbmNlX2RlZmF1bHRfd2FpdCwKPj4+IC0JLnJlbGVhc2UgPSBOVUxMLAo+Pj4gICAgfTsKCl9f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmFtZC1nZnggbWFp bGluZyBsaXN0CmFtZC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJl ZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vYW1kLWdmeAo=