From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?Q?Christian_K=c3=b6nig?= Subject: Re: [PATCH 3/4] drm/ttm: handle already locked BOs during eviction and swapout. Date: Fri, 23 Feb 2018 13:05:39 +0100 Message-ID: <6eadbbea-8eca-a1ab-93bb-d37ae6e5f29d@gmail.com> References: <20180220125829.27060-1-christian.koenig@amd.com> <20180220125829.27060-3-christian.koenig@amd.com> Reply-To: christian.koenig@amd.com Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: Content-Language: en-US List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: "He, Roger" , "amd-gfx@lists.freedesktop.org" , "dri-devel@lists.freedesktop.org" , "linux-kernel@vger.kernel.org" List-Id: dri-devel@lists.freedesktop.org QW0gMjMuMDIuMjAxOCB1bSAxMDo0NiBzY2hyaWViIEhlLCBSb2dlcjoKPgo+IC0tLS0tT3JpZ2lu YWwgTWVzc2FnZS0tLS0tCj4gRnJvbTogZHJpLWRldmVsIFttYWlsdG86ZHJpLWRldmVsLWJvdW5j ZXNAbGlzdHMuZnJlZWRlc2t0b3Aub3JnXSBPbiBCZWhhbGYgT2YgQ2hyaXN0aWFuIEs/bmlnCj4g U2VudDogVHVlc2RheSwgRmVicnVhcnkgMjAsIDIwMTggODo1OCBQTQo+IFRvOiBhbWQtZ2Z4QGxp c3RzLmZyZWVkZXNrdG9wLm9yZzsgZHJpLWRldmVsQGxpc3RzLmZyZWVkZXNrdG9wLm9yZzsgbGlu dXgta2VybmVsQHZnZXIua2VybmVsLm9yZwo+IFN1YmplY3Q6IFtQQVRDSCAzLzRdIGRybS90dG06 IGhhbmRsZSBhbHJlYWR5IGxvY2tlZCBCT3MgZHVyaW5nIGV2aWN0aW9uIGFuZCBzd2Fwb3V0Lgo+ Cj4gVGhpcyBzb2x2ZXMgdGhlIHByb2JsZW0gdGhhdCB3aGVuIHdlIHN3YXBvdXQgYSBCTyBmcm9t IGEgZG9tYWluIHdlIHNvbWV0aW1lcyBjb3VsZG4ndCBtYWtlIHJvb20gZm9yIGl0IGJlY2F1c2Ug aG9sZGluZyB0aGUgbG9jayBibG9ja3MgYWxsIG90aGVyIEJPcyB3aXRoIHRoaXMgcmVzZXJ2YXRp b24gb2JqZWN0Lgo+Cj4gU2lnbmVkLW9mZi1ieTogQ2hyaXN0aWFuIEvDtm5pZyA8Y2hyaXN0aWFu LmtvZW5pZ0BhbWQuY29tPgo+IC0tLQo+ICAgZHJpdmVycy9ncHUvZHJtL3R0bS90dG1fYm8uYyB8 IDMzICsrKysrKysrKysrKysrKystLS0tLS0tLS0tLS0tLS0tLQo+ICAgMSBmaWxlIGNoYW5nZWQs IDE2IGluc2VydGlvbnMoKyksIDE3IGRlbGV0aW9ucygtKQo+Cj4gZGlmZiAtLWdpdCBhL2RyaXZl cnMvZ3B1L2RybS90dG0vdHRtX2JvLmMgYi9kcml2ZXJzL2dwdS9kcm0vdHRtL3R0bV9iby5jIGlu ZGV4IGQ5MGIxY2YxMGIyNy4uM2E0NGMyZWU0MTU1IDEwMDY0NAo+IC0tLSBhL2RyaXZlcnMvZ3B1 L2RybS90dG0vdHRtX2JvLmMKPiArKysgYi9kcml2ZXJzL2dwdS9kcm0vdHRtL3R0bV9iby5jCj4g QEAgLTcxMywzMSArNzEzLDMwIEBAIGJvb2wgdHRtX2JvX2V2aWN0aW9uX3ZhbHVhYmxlKHN0cnVj dCB0dG1fYnVmZmVyX29iamVjdCAqYm8sICBFWFBPUlRfU1lNQk9MKHR0bV9ib19ldmljdGlvbl92 YWx1YWJsZSk7Cj4gICAKPiAgIC8qKgo+IC0gKiBDaGVjayB0aGUgdGFyZ2V0IGJvIGlzIGFsbG93 YWJsZSB0byBiZSBldmljdGVkIG9yIHN3YXBvdXQsIGluY2x1ZGluZyBjYXNlczoKPiAtICoKPiAt ICogYS4gaWYgc2hhcmUgc2FtZSByZXNlcnZhdGlvbiBvYmplY3Qgd2l0aCBjdHgtPnJlc3YsIGhh dmUgYXNzdW1wdGlvbgo+IC0gKiByZXNlcnZhdGlvbiBvYmplY3RzIHNob3VsZCBhbHJlYWR5IGJl IGxvY2tlZCwgc28gbm90IGxvY2sgYWdhaW4gYW5kCj4gLSAqIHJldHVybiB0cnVlIGRpcmVjdGx5 IHdoZW4gZWl0aGVyIHRoZSBvcHJlYXRpb24gYWxsb3dfcmVzZXJ2ZWRfZXZpY3Rpb24KPiAtICog b3IgdGhlIHRhcmdldCBibyBhbHJlYWR5IGlzIGluIGRlbGF5ZWQgZnJlZSBsaXN0Owo+IC0gKgo+ IC0gKiBiLiBPdGhlcndpc2UsIHRyeWxvY2sgaXQuCj4gKyAqIENoZWNrIGlmIHRoZSB0YXJnZXQg Ym8gaXMgYWxsb3dlZCB0byBiZSBldmljdGVkIG9yIHN3YXBlZG91dC4KPiAgICAqLwo+ICAgc3Rh dGljIGJvb2wgdHRtX2JvX2V2aWN0X3N3YXBvdXRfYWxsb3dhYmxlKHN0cnVjdCB0dG1fYnVmZmVy X29iamVjdCAqYm8sCj4gLQkJCXN0cnVjdCB0dG1fb3BlcmF0aW9uX2N0eCAqY3R4LCBib29sICps b2NrZWQpCj4gKwkJCQkJICAgc3RydWN0IHR0bV9vcGVyYXRpb25fY3R4ICpjdHgsCj4gKwkJCQkJ ICAgYm9vbCAqbG9ja2VkKQo+ICAgewo+IC0JYm9vbCByZXQgPSBmYWxzZTsKPiArCS8qIEZpcnN0 IGNoZWNrIGlmIHdlIGNhbiBsb2NrIGl0ICovCj4gKwkqbG9ja2VkID0gcmVzZXJ2YXRpb25fb2Jq ZWN0X3RyeWxvY2soYm8tPnJlc3YpOwo+ICsJaWYgKCpsb2NrZWQpCj4gKwkJcmV0dXJuIHRydWU7 Cj4gICAKPiAtCSpsb2NrZWQgPSBmYWxzZTsKPiArCS8qIENoZWNrIGlmIGl0J3MgbG9ja2VkIGJl Y2F1c2UgaXQgaXMgcGFydCBvZiB0aGUgY3VycmVudCBvcGVyYXRpb24gKi8KPiAgIAlpZiAoYm8t PnJlc3YgPT0gY3R4LT5yZXN2KSB7Cj4gICAJCXJlc2VydmF0aW9uX29iamVjdF9hc3NlcnRfaGVs ZChiby0+cmVzdik7Cj4gLQkJaWYgKGN0eC0+YWxsb3dfcmVzZXJ2ZWRfZXZpY3Rpb24gfHwgIWxp c3RfZW1wdHkoJmJvLT5kZGVzdHJveSkpCj4gLQkJCXJldCA9IHRydWU7Cj4gLQl9IGVsc2Ugewo+ IC0JCSpsb2NrZWQgPSByZXNlcnZhdGlvbl9vYmplY3RfdHJ5bG9jayhiby0+cmVzdik7Cj4gLQkJ cmV0ID0gKmxvY2tlZDsKPiArCQlyZXR1cm4gY3R4LT5hbGxvd19yZXNlcnZlZF9ldmljdGlvbiB8 fAo+ICsJCQkhbGlzdF9lbXB0eSgmYm8tPmRkZXN0cm95KTsKPiAgIAl9Cj4gICAKPiAtCXJldHVy biByZXQ7Cj4gKwkvKiBDaGVjayBpZiBpdCdzIGxvY2tlZCBiZWNhdXNlIGl0IHdhcyBhbHJlYWR5 IGV2aWN0ZWQgKi8KPiArCWlmICh3d19tdXRleF9pc19vd25lZF9ieSgmYm8tPnJlc3YtPmxvY2ss IE5VTEwpKQo+ICsJCXJldHVybiB0cnVlOwo+Cj4gRm9yIHRoZSBzcGVjaWFsIGNhc2U6IHdoZW4g Y29tbWFuZCBzdWJtaXNzaW9uIHdpdGggUGVyLVZNLUJPIGVuYWJsZWQsCj4gQWxsIEJPcyAgYS9i L2MgYXJlIGFsd2F5cyB2YWxpZCBCTy4gQWZ0ZXIgdGhlIHZhbGlkYXRpb24gb2YgQk9zIGEgYW5k IGIsCj4gd2hlbiB2YWxpZGF0aW9uIG9mIEJPIGMsIGlzIGl0IHBvc3NpYmxlIHRvIHJldHVybiB0 cnVlIGFuZCB0aGVuIGV2aWN0IEJPIGEgYW5kIGIgYnkgbWlzdGFrZSA/Cj4gQmVjYXVzZSBhL2Iv YyBzaGFyZSBzYW1lIHRhc2tfc3RydWN0LgoKTm8sIHRoYXQncyB3aHkgSSBjaGVjayB0aGUgY29u dGV4dCBhcyB3ZWxsLiBCT3MgZXhwbGljaXRseSByZXNlcnZlZCBoYXZlIAphIG5vbiBOVUxMIGNv bnRleHQgd2hpbGUgQk9zIHRyeWxvY2tlZCBmb3Igc3dhcG91dCBoYXZlIGEgTlVMTCBjb250ZXh0 LgoKQ2hyaXN0aWFuLgoKPgo+IFRoYW5rcwo+IFJvZ2VyKEhvbmdiby5IZSkKPgo+ICsJLyogU29t ZSBvdGhlciB0aHJlYWQgaXMgdXNpbmcgaXQsIGRvbid0IHRvdWNoIGl0ICovCj4gKwlyZXR1cm4g ZmFsc2U7Cj4gICB9Cj4gICAKPiAgIHN0YXRpYyBpbnQgdHRtX21lbV9ldmljdF9maXJzdChzdHJ1 Y3QgdHRtX2JvX2RldmljZSAqYmRldiwKPiAtLQo+IDIuMTQuMQo+Cj4gX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KPiBkcmktZGV2ZWwgbWFpbGluZyBsaXN0 Cj4gZHJpLWRldmVsQGxpc3RzLmZyZWVkZXNrdG9wLm9yZwo+IGh0dHBzOi8vbGlzdHMuZnJlZWRl c2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCgpfX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fXwpkcmktZGV2ZWwgbWFpbGluZyBsaXN0CmRyaS1k ZXZlbEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcv bWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwK