From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?Q?Christian_K=c3=b6nig?= Subject: Re: [PATCH v2 2/2] drm/amdgpu: Move old fence waiting before reservation lock is aquired. Date: Wed, 11 Oct 2017 09:57:33 +0200 Message-ID: <35727ec2-b806-0944-ed55-bda3136b760a@amd.com> References: <715810ad-26cc-506d-624d-8e5024968ea0@gmail.com> <1507668617-27299-1-git-send-email-andrey.grodzovsky@amd.com> <1507668617-27299-2-git-send-email-andrey.grodzovsky@amd.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <1507668617-27299-2-git-send-email-andrey.grodzovsky-5C7GfCeVMHo@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: Andrey Grodzovsky , Monk.Liu-5C7GfCeVMHo@public.gmane.org, amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org QW0gMTAuMTAuMjAxNyB1bSAyMjo1MCBzY2hyaWViIEFuZHJleSBHcm9kem92c2t5Ogo+IEhlbHBz IGF2b2lkaW5nIGRlYWRsb2NrIGR1cmluZyBHUFUgcmVzZXQuCj4gQWRkZWQgbXV0ZXggdG8gYW1k Z3B1X2N0eCB0byBwcmVzZXJ2ZSBvcmRlciBvZiBmZW5jZXMgb24gYSByaW5nLgo+Cj4gdjI6Cj4g UHV0IHdhaXRpbmcgbG9naWMgaW4gYSBmdW5jdGlvbiBpbiBhIHNlcGVyYXRlIGZ1bmN0aW9uIGlu IGFtZGdwdV9jdHguYwo+Cj4gU2lnbmVkLW9mZi1ieTogQW5kcmV5IEdyb2R6b3Zza3kgPGFuZHJl eS5ncm9kem92c2t5QGFtZC5jb20+Cj4gLS0tCj4gICBkcml2ZXJzL2dwdS9kcm0vYW1kL2FtZGdw dS9hbWRncHUuaCAgICAgfCAgNCArKysrCj4gICBkcml2ZXJzL2dwdS9kcm0vYW1kL2FtZGdwdS9h bWRncHVfY3MuYyAgfCAgOCArKysrKystLQo+ICAgZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUv YW1kZ3B1X2N0eC5jIHwgMzAgKysrKysrKysrKysrKysrKysrKysrKysrLS0tLS0tCj4gICAzIGZp bGVzIGNoYW5nZWQsIDM0IGluc2VydGlvbnMoKyksIDggZGVsZXRpb25zKC0pCj4KPiBkaWZmIC0t Z2l0IGEvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1kZ3B1LmggYi9kcml2ZXJzL2dwdS9k cm0vYW1kL2FtZGdwdS9hbWRncHUuaAo+IGluZGV4IGRhNDhmOTcuLjIzNWVjYTUgMTAwNjQ0Cj4g LS0tIGEvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1kZ3B1LmgKPiArKysgYi9kcml2ZXJz L2dwdS9kcm0vYW1kL2FtZGdwdS9hbWRncHUuaAo+IEBAIC03NDEsNiArNzQxLDcgQEAgc3RydWN0 IGFtZGdwdV9jdHggewo+ICAgCWJvb2wgCQkJcHJlYW1ibGVfcHJlc2VudGVkOwo+ICAgCWVudW0g YW1kX3NjaGVkX3ByaW9yaXR5IGluaXRfcHJpb3JpdHk7Cj4gICAJZW51bSBhbWRfc2NoZWRfcHJp b3JpdHkgb3ZlcnJpZGVfcHJpb3JpdHk7Cj4gKwlzdHJ1Y3QgbXV0ZXggICAgICAgICAgICBsb2Nr Owo+ICAgfTsKPiAgIAo+ICAgc3RydWN0IGFtZGdwdV9jdHhfbWdyIHsKPiBAQCAtNzYzLDkgKzc2 NCwxMiBAQCB2b2lkIGFtZGdwdV9jdHhfcHJpb3JpdHlfb3ZlcnJpZGUoc3RydWN0IGFtZGdwdV9j dHggKmN0eCwKPiAgIGludCBhbWRncHVfY3R4X2lvY3RsKHN0cnVjdCBkcm1fZGV2aWNlICpkZXYs IHZvaWQgKmRhdGEsCj4gICAJCSAgICAgc3RydWN0IGRybV9maWxlICpmaWxwKTsKPiAgIAo+ICtp bnQgYW1kZ3B1X2N0eF93YWl0X3ByZXZfZmVuY2Uoc3RydWN0IGFtZGdwdV9jdHggKmN0eCwgdW5z aWduZWQgcmluZ19pZCk7Cj4gKwo+ICAgdm9pZCBhbWRncHVfY3R4X21ncl9pbml0KHN0cnVjdCBh bWRncHVfY3R4X21nciAqbWdyKTsKPiAgIHZvaWQgYW1kZ3B1X2N0eF9tZ3JfZmluaShzdHJ1Y3Qg YW1kZ3B1X2N0eF9tZ3IgKm1ncik7Cj4gICAKPiArCj4gICAvKgo+ICAgICogZmlsZSBwcml2YXRl IHN0cnVjdHVyZQo+ICAgICovCj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9hbWQvYW1k Z3B1L2FtZGdwdV9jcy5jIGIvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1kZ3B1X2NzLmMK PiBpbmRleCAxYTU0ZTUzLi5jMzYyOTdjIDEwMDY0NAo+IC0tLSBhL2RyaXZlcnMvZ3B1L2RybS9h bWQvYW1kZ3B1L2FtZGdwdV9jcy5jCj4gKysrIGIvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUv YW1kZ3B1X2NzLmMKPiBAQCAtOTAsNiArOTAsOCBAQCBzdGF0aWMgaW50IGFtZGdwdV9jc19wYXJz ZXJfaW5pdChzdHJ1Y3QgYW1kZ3B1X2NzX3BhcnNlciAqcCwgdm9pZCAqZGF0YSkKPiAgIAkJZ290 byBmcmVlX2NodW5rOwo+ICAgCX0KPiAgIAo+ICsJbXV0ZXhfbG9jaygmcC0+Y3R4LT5sb2NrKTsK PiArCgpUaGVyZSBpcyBhIGJ1ZyBpbiBhbWRncHVfY3NfcGFyc2VyX2luaXQoKSwgdGFrZSBhIGxv b2sgYXQgdGhlIHB1dF9jdHggCmxhYmVsLiBJdCBjYWxscyBhbWRncHVfY3R4X3B1dCgpIHdpdGhv dXQgc2V0dGluZyBwLT5jdHggdG8gTlVMTCBhZnRlciB0aGF0LgoKVGhpcyB3YXkgYW1kZ3B1X2Nz X3BhcnNlcl9maW5pKCkgd2lsbCBjYWxsIGFtZGdwdV9jdHhfcHV0KCkgYWdhaW4gYW5kIAptZXNz IHVwIHRoZSByZWZlcmVuY2UgY291bnQuCgpUaGF0IGlzIG5vdCBhIGJ1ZyBpbnRyb2R1Y2VkIGJ5 IHRoaXMgcGF0Y2gsIHNvIHRoaXMgb25lIGlzIFJldmlld2VkLWJ5OiAKQ2hyaXN0aWFuIEvDtm5p ZyA8Y2hyaXN0aWFuLmtvZW5pZ0BhbWQuY29tPi4KCkJ1dCBwbGVhc2UgcHJvdmlkZSBhIGZvbGxv dyB1cCBwYXRjaCBqdXN0IHJlbW92aW5nIHRoZSBleHRyYSAKYW1kZ3B1X2N0eF9wdXQoKSBmcm9t IGFtZGdwdV9jc19wYXJzZXJfaW5pdCgpLgoKVGhhbmtzIGZvciB0aGUgaGVscCwKQ2hyaXN0aWFu LgoKPiAgIAkvKiBnZXQgY2h1bmtzICovCj4gICAJY2h1bmtfYXJyYXlfdXNlciA9IHU2NF90b191 c2VyX3B0cihjcy0+aW4uY2h1bmtzKTsKPiAgIAlpZiAoY29weV9mcm9tX3VzZXIoY2h1bmtfYXJy YXksIGNodW5rX2FycmF5X3VzZXIsCj4gQEAgLTczNyw4ICs3MzksMTAgQEAgc3RhdGljIHZvaWQg YW1kZ3B1X2NzX3BhcnNlcl9maW5pKHN0cnVjdCBhbWRncHVfY3NfcGFyc2VyICpwYXJzZXIsIGlu dCBlcnJvciwKPiAgIAo+ICAgCWRtYV9mZW5jZV9wdXQocGFyc2VyLT5mZW5jZSk7Cj4gICAKPiAt CWlmIChwYXJzZXItPmN0eCkKPiArCWlmIChwYXJzZXItPmN0eCkgewo+ICsJCW11dGV4X3VubG9j aygmcGFyc2VyLT5jdHgtPmxvY2spOwo+ICAgCQlhbWRncHVfY3R4X3B1dChwYXJzZXItPmN0eCk7 Cj4gKwl9Cj4gICAKPiAgIAlpZiAocGFyc2VyLT5ib19saXN0KQo+ICAgCQlhbWRncHVfYm9fbGlz dF9wdXQocGFyc2VyLT5ib19saXN0KTsKPiBAQCAtOTkyLDcgKzk5Niw3IEBAIHN0YXRpYyBpbnQg YW1kZ3B1X2NzX2liX2ZpbGwoc3RydWN0IGFtZGdwdV9kZXZpY2UgKmFkZXYsCj4gICAJICAgIHBh cnNlci0+am9iLT5yaW5nLT5mdW5jcy0+dHlwZSA9PSBBTURHUFVfUklOR19UWVBFX1ZDRSkpCj4g ICAJCXJldHVybiAtRUlOVkFMOwo+ICAgCj4gLQlyZXR1cm4gMDsKPiArCXJldHVybiBhbWRncHVf Y3R4X3dhaXRfcHJldl9mZW5jZShwYXJzZXItPmN0eCwgcGFyc2VyLT5qb2ItPnJpbmctPmlkeCk7 Cj4gICB9Cj4gICAKPiAgIHN0YXRpYyBpbnQgYW1kZ3B1X2NzX3Byb2Nlc3NfZmVuY2VfZGVwKHN0 cnVjdCBhbWRncHVfY3NfcGFyc2VyICpwLAo+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0v YW1kL2FtZGdwdS9hbWRncHVfY3R4LmMgYi9kcml2ZXJzL2dwdS9kcm0vYW1kL2FtZGdwdS9hbWRn cHVfY3R4LmMKPiBpbmRleCBhNzhiMDNmNi4uNDMwOTgyMCAxMDA2NDQKPiAtLS0gYS9kcml2ZXJz L2dwdS9kcm0vYW1kL2FtZGdwdS9hbWRncHVfY3R4LmMKPiArKysgYi9kcml2ZXJzL2dwdS9kcm0v YW1kL2FtZGdwdS9hbWRncHVfY3R4LmMKPiBAQCAtNjcsNiArNjcsOCBAQCBzdGF0aWMgaW50IGFt ZGdwdV9jdHhfaW5pdChzdHJ1Y3QgYW1kZ3B1X2RldmljZSAqYWRldiwKPiAgIAlpZiAoIWN0eC0+ ZmVuY2VzKQo+ICAgCQlyZXR1cm4gLUVOT01FTTsKPiAgIAo+ICsJbXV0ZXhfaW5pdCgmY3R4LT5s b2NrKTsKPiArCj4gICAJZm9yIChpID0gMDsgaSA8IEFNREdQVV9NQVhfUklOR1M7ICsraSkgewo+ ICAgCQljdHgtPnJpbmdzW2ldLnNlcXVlbmNlID0gMTsKPiAgIAkJY3R4LT5yaW5nc1tpXS5mZW5j ZXMgPSAmY3R4LT5mZW5jZXNbYW1kZ3B1X3NjaGVkX2pvYnMgKiBpXTsKPiBAQCAtMTI2LDYgKzEy OCw4IEBAIHN0YXRpYyB2b2lkIGFtZGdwdV9jdHhfZmluaShzdHJ1Y3QgYW1kZ3B1X2N0eCAqY3R4 KQo+ICAgCQkJCSAgICAgICZjdHgtPnJpbmdzW2ldLmVudGl0eSk7Cj4gICAKPiAgIAlhbWRncHVf cXVldWVfbWdyX2ZpbmkoYWRldiwgJmN0eC0+cXVldWVfbWdyKTsKPiArCj4gKwltdXRleF9kZXN0 cm95KCZjdHgtPmxvY2spOwo+ICAgfQo+ICAgCj4gICBzdGF0aWMgaW50IGFtZGdwdV9jdHhfYWxs b2Moc3RydWN0IGFtZGdwdV9kZXZpY2UgKmFkZXYsCj4gQEAgLTI5NiwxMiArMzAwLDggQEAgaW50 IGFtZGdwdV9jdHhfYWRkX2ZlbmNlKHN0cnVjdCBhbWRncHVfY3R4ICpjdHgsIHN0cnVjdCBhbWRn cHVfcmluZyAqcmluZywKPiAgIAo+ICAgCWlkeCA9IHNlcSAmIChhbWRncHVfc2NoZWRfam9icyAt IDEpOwo+ICAgCW90aGVyID0gY3JpbmctPmZlbmNlc1tpZHhdOwo+IC0JaWYgKG90aGVyKSB7Cj4g LQkJc2lnbmVkIGxvbmcgcjsKPiAtCQlyID0gZG1hX2ZlbmNlX3dhaXRfdGltZW91dChvdGhlciwg dHJ1ZSwgTUFYX1NDSEVEVUxFX1RJTUVPVVQpOwo+IC0JCWlmIChyIDwgMCkKPiAtCQkJcmV0dXJu IHI7Cj4gLQl9Cj4gKwlpZiAob3RoZXIpCj4gKwkJQlVHX09OKCFkbWFfZmVuY2VfaXNfc2lnbmFs ZWQob3RoZXIpKTsKPiAgIAo+ICAgCWRtYV9mZW5jZV9nZXQoZmVuY2UpOwo+ICAgCj4gQEAgLTM3 Miw2ICszNzIsMjQgQEAgdm9pZCBhbWRncHVfY3R4X3ByaW9yaXR5X292ZXJyaWRlKHN0cnVjdCBh bWRncHVfY3R4ICpjdHgsCj4gICAJfQo+ICAgfQo+ICAgCj4gK2ludCBhbWRncHVfY3R4X3dhaXRf cHJldl9mZW5jZShzdHJ1Y3QgYW1kZ3B1X2N0eCAqY3R4LCB1bnNpZ25lZCByaW5nX2lkKQo+ICt7 Cj4gKwlzdHJ1Y3QgYW1kZ3B1X2N0eF9yaW5nICpjcmluZyA9ICZjdHgtPnJpbmdzW3JpbmdfaWRd Owo+ICsJdW5zaWduZWQgaWR4ID0gY3JpbmctPnNlcXVlbmNlICYgKGFtZGdwdV9zY2hlZF9qb2Jz IC0gMSk7Cj4gKwlzdHJ1Y3QgZG1hX2ZlbmNlICpvdGhlciA9IGNyaW5nLT5mZW5jZXNbaWR4XTsK PiArCj4gKwlpZiAob3RoZXIpIHsKPiArCQlzaWduZWQgbG9uZyByOwo+ICsJCXIgPSBkbWFfZmVu Y2Vfd2FpdF90aW1lb3V0KG90aGVyLCBmYWxzZSwgTUFYX1NDSEVEVUxFX1RJTUVPVVQpOwo+ICsJ CWlmIChyIDwgMCkgewo+ICsJCQlEUk1fRVJST1IoIkVycm9yICglbGQpIHdhaXRpbmcgZm9yIGZl bmNlIVxuIiwgcik7Cj4gKwkJCXJldHVybiByOwo+ICsJCX0KPiArCX0KPiArCj4gKwlyZXR1cm4g MDsKPiArfQo+ICsKPiAgIHZvaWQgYW1kZ3B1X2N0eF9tZ3JfaW5pdChzdHJ1Y3QgYW1kZ3B1X2N0 eF9tZ3IgKm1ncikKPiAgIHsKPiAgIAltdXRleF9pbml0KCZtZ3ItPmxvY2spOwoKCl9fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmFtZC1nZnggbWFpbGluZyBs aXN0CmFtZC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0 b3Aub3JnL21haWxtYW4vbGlzdGluZm8vYW1kLWdmeAo=