From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Zhang, Jerry (Junwei)" Subject: Re: [PATCH v2 2/2] drm/amdgpu: clear freed mappings immediately when BO may be freed Date: Fri, 24 Mar 2017 11:42:34 +0800 Message-ID: <58D495AA.9010607@amd.com> References: <20170323192705.5474-1-nhaehnle@gmail.com> <20170323192705.5474-2-nhaehnle@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20170323192705.5474-2-nhaehnle-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> 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: =?UTF-8?B?Tmljb2xhaSBIw6Robmxl?= , amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org Cc: =?UTF-8?B?Tmljb2xhaSBIw6Robmxl?= T24gMDMvMjQvMjAxNyAwMzoyNyBBTSwgTmljb2xhaSBIw6RobmxlIHdyb3RlOgo+IEZyb206IE5p Y29sYWkgSMOkaG5sZSA8bmljb2xhaS5oYWVobmxlQGFtZC5jb20+Cj4KPiBBbHNvLCBhZGQgdGhl IGZlbmNlIG9mIHRoZSBjbGVhciBvcGVyYXRpb25zIHRvIHRoZSBCTyB0byBlbnN1cmUgdGhhdAo+ IHRoZSB1bmRlcmx5aW5nIG1lbW9yeSBjYW4gb25seSBiZSByZS11c2VkIGFmdGVyIGFsbCBQVEVz IHBvaW50aW5nIHRvCj4gaXQgaGF2ZSBiZWVuIGNsZWFyZWQuCj4KPiBUaGlzIGF2b2lkcyB0aGUg Zm9sbG93aW5nIHNlcXVlbmNlIG9mIGV2ZW50cyB0aGF0IGNvdWxkIGJlIHRyaWdnZXJlZAo+IGJ5 IHVzZXIgc3BhY2U6Cj4KPiAxLiBTdWJtaXQgYSBDUyB0aGF0IGFjY2Vzc2VzIHNvbWUgQk8gX3dp dGhvdXRfIGFkZGluZyB0aGF0IEJPIHRvIHRoZQo+ICAgICBidWZmZXIgbGlzdC4KPiAyLiBGcmVl IHRoYXQgQk8uCj4gMy4gU29tZSBvdGhlciB0YXNrIHJlLXVzZXMgdGhlIG1lbW9yeSB1bmRlcmx5 aW5nIHRoZSBCTy4KPiA0LiBUaGUgQ1MgaXMgc3VibWl0dGVkIHRvIHRoZSBoYXJkd2FyZSBhbmQg YWNjZXNzZXMgbWVtb3J5IHRoYXQgaXMKPiAgICAgbm93IGFscmVhZHkgaW4gdXNlIGJ5IHNvbWVi b2R5IGVsc2UuCj4KPiBCeSBjbGVhcmluZyB0aGUgcGFnZSB0YWJsZXMgaW1tZWRpYXRlbHkgaW4g c3RlcCAyLCBhIEdQVSBWTSBmYXVsdCB3aWxsCj4gYmUgdHJpZ2dlcmVkIGluIHN0ZXAgNCBpbnN0 ZWFkIG9mIHdpbGQgbWVtb3J5IGFjY2Vzc2VzLgo+Cj4gdjI6IHVzZSBhbWRncHVfYm9fZmVuY2Ug ZGlyZWN0bHkKPgo+IFNpZ25lZC1vZmYtYnk6IE5pY29sYWkgSMOkaG5sZSA8bmljb2xhaS5oYWVo bmxlQGFtZC5jb20+Cj4gLS0tCj4gICBkcml2ZXJzL2dwdS9kcm0vYW1kL2FtZGdwdS9hbWRncHVf Z2VtLmMgfCAxMiArKysrKysrKysrKysKPiAgIDEgZmlsZSBjaGFuZ2VkLCAxMiBpbnNlcnRpb25z KCspCj4KPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1kZ3B1X2dl bS5jIGIvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1kZ3B1X2dlbS5jCj4gaW5kZXggNGE1 M2M0My4uOGIwZjVmMTggMTAwNjQ0Cj4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUv YW1kZ3B1X2dlbS5jCj4gKysrIGIvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1kZ3B1X2dl bS5jCj4gQEAgLTE0NSwyMCArMTQ1LDIxIEBAIHZvaWQgYW1kZ3B1X2dlbV9vYmplY3RfY2xvc2Uo c3RydWN0IGRybV9nZW1fb2JqZWN0ICpvYmosCj4gICAJc3RydWN0IGFtZGdwdV9ibyAqYm8gPSBn ZW1fdG9fYW1kZ3B1X2JvKG9iaik7Cj4gICAJc3RydWN0IGFtZGdwdV9kZXZpY2UgKmFkZXYgPSBh bWRncHVfdHRtX2FkZXYoYm8tPnRiby5iZGV2KTsKPiAgIAlzdHJ1Y3QgYW1kZ3B1X2Zwcml2ICpm cHJpdiA9IGZpbGVfcHJpdi0+ZHJpdmVyX3ByaXY7Cj4gICAJc3RydWN0IGFtZGdwdV92bSAqdm0g PSAmZnByaXYtPnZtOwo+Cj4gICAJc3RydWN0IGFtZGdwdV9ib19saXN0X2VudHJ5IHZtX3BkOwo+ ICAgCXN0cnVjdCBsaXN0X2hlYWQgbGlzdCwgZHVwbGljYXRlczsKPiAgIAlzdHJ1Y3QgdHRtX3Zh bGlkYXRlX2J1ZmZlciB0djsKPiAgIAlzdHJ1Y3Qgd3dfYWNxdWlyZV9jdHggdGlja2V0Owo+ICAg CXN0cnVjdCBhbWRncHVfYm9fdmEgKmJvX3ZhOwo+ICsJc3RydWN0IGZlbmNlICpmZW5jZSA9IE5V TEw7Cj4gICAJaW50IHI7Cj4KPiAgIAlJTklUX0xJU1RfSEVBRCgmbGlzdCk7Cj4gICAJSU5JVF9M SVNUX0hFQUQoJmR1cGxpY2F0ZXMpOwo+Cj4gICAJdHYuYm8gPSAmYm8tPnRibzsKPiAgIAl0di5z aGFyZWQgPSB0cnVlOwo+ICAgCWxpc3RfYWRkKCZ0di5oZWFkLCAmbGlzdCk7Cj4KPiAgIAlhbWRn cHVfdm1fZ2V0X3BkX2JvKHZtLCAmbGlzdCwgJnZtX3BkKTsKPiBAQCAtMTY2LDIwICsxNjcsMzEg QEAgdm9pZCBhbWRncHVfZ2VtX29iamVjdF9jbG9zZShzdHJ1Y3QgZHJtX2dlbV9vYmplY3QgKm9i aiwKPiAgIAlyID0gdHRtX2V1X3Jlc2VydmVfYnVmZmVycygmdGlja2V0LCAmbGlzdCwgZmFsc2Us ICZkdXBsaWNhdGVzKTsKPiAgIAlpZiAocikgewo+ICAgCQlkZXZfZXJyKGFkZXYtPmRldiwgImxl YWtpbmcgYm8gdmEgYmVjYXVzZSAiCj4gICAJCQkid2UgZmFpbCB0byByZXNlcnZlIGJvICglZClc biIsIHIpOwo+ICAgCQlyZXR1cm47Cj4gICAJfQo+ICAgCWJvX3ZhID0gYW1kZ3B1X3ZtX2JvX2Zp bmQodm0sIGJvKTsKPiAgIAlpZiAoYm9fdmEpIHsKPiAgIAkJaWYgKC0tYm9fdmEtPnJlZl9jb3Vu dCA9PSAwKSB7Cj4gICAJCQlhbWRncHVfdm1fYm9fcm12KGFkZXYsIGJvX3ZhKTsKPiArCj4gKwkJ CXIgPSBhbWRncHVfdm1fY2xlYXJfZnJlZWQoYWRldiwgdm0sICZmZW5jZSk7Cj4gKwkJCWlmICh1 bmxpa2VseShyKSkgewo+ICsJCQkJZGV2X2VycihhZGV2LT5kZXYsICJmYWlsZWQgdG8gY2xlYXIg cGFnZSAiCj4gKwkJCQkJInRhYmxlcyBvbiBHRU0gb2JqZWN0IGNsb3NlICglZClcbiIsIHIpOwo+ ICsJCQl9Cj4gKwo+ICsJCQlpZiAoZmVuY2UpIHsKCkkgdGhpbmsgaXQncyBhbHdheXMgdHJ1ZS4K TWF5YmUgeW91IG1lYW4gKmZlbmNlPwoKPiArCQkJCWFtZGdwdV9ib19mZW5jZShibywgZmVuY2Us IHRydWUpOwo+ICsJCQkJZmVuY2VfcHV0KGZlbmNlKTsKPiArCQkJfQo+ICAgCQl9Cj4gICAJfQo+ ICAgCXR0bV9ldV9iYWNrb2ZmX3Jlc2VydmF0aW9uKCZ0aWNrZXQsICZsaXN0KTsKPiAgIH0KPgo+ ICAgc3RhdGljIGludCBhbWRncHVfZ2VtX2hhbmRsZV9sb2NrdXAoc3RydWN0IGFtZGdwdV9kZXZp Y2UgKmFkZXYsIGludCByKQo+ICAgewo+ICAgCWlmIChyID09IC1FREVBRExLKSB7Cj4gICAJCXIg PSBhbWRncHVfZ3B1X3Jlc2V0KGFkZXYpOwo+ICAgCQlpZiAoIXIpCj4KX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KYW1kLWdmeCBtYWlsaW5nIGxpc3QKYW1k LWdmeEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcv bWFpbG1hbi9saXN0aW5mby9hbWQtZ2Z4Cg==