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:45:59 +0800 Message-ID: <58D49677.1060002@amd.com> References: <20170323192705.5474-1-nhaehnle@gmail.com> <20170323192705.5474-2-nhaehnle@gmail.com> <58D495AA.9010607@amd.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <58D495AA.9010607-5C7GfCeVMHo@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?= T24gMDMvMjQvMjAxNyAxMTo0MiBBTSwgWmhhbmcsIEplcnJ5IChKdW53ZWkpIHdyb3RlOgo+IE9u IDAzLzI0LzIwMTcgMDM6MjcgQU0sIE5pY29sYWkgSMOkaG5sZSB3cm90ZToKPj4gRnJvbTogTmlj b2xhaSBIw6RobmxlIDxuaWNvbGFpLmhhZWhubGVAYW1kLmNvbT4KPj4KPj4gQWxzbywgYWRkIHRo ZSBmZW5jZSBvZiB0aGUgY2xlYXIgb3BlcmF0aW9ucyB0byB0aGUgQk8gdG8gZW5zdXJlIHRoYXQK Pj4gdGhlIHVuZGVybHlpbmcgbWVtb3J5IGNhbiBvbmx5IGJlIHJlLXVzZWQgYWZ0ZXIgYWxsIFBU RXMgcG9pbnRpbmcgdG8KPj4gaXQgaGF2ZSBiZWVuIGNsZWFyZWQuCj4+Cj4+IFRoaXMgYXZvaWRz IHRoZSBmb2xsb3dpbmcgc2VxdWVuY2Ugb2YgZXZlbnRzIHRoYXQgY291bGQgYmUgdHJpZ2dlcmVk Cj4+IGJ5IHVzZXIgc3BhY2U6Cj4+Cj4+IDEuIFN1Ym1pdCBhIENTIHRoYXQgYWNjZXNzZXMgc29t ZSBCTyBfd2l0aG91dF8gYWRkaW5nIHRoYXQgQk8gdG8gdGhlCj4+ICAgICBidWZmZXIgbGlzdC4K Pj4gMi4gRnJlZSB0aGF0IEJPLgo+PiAzLiBTb21lIG90aGVyIHRhc2sgcmUtdXNlcyB0aGUgbWVt b3J5IHVuZGVybHlpbmcgdGhlIEJPLgo+PiA0LiBUaGUgQ1MgaXMgc3VibWl0dGVkIHRvIHRoZSBo YXJkd2FyZSBhbmQgYWNjZXNzZXMgbWVtb3J5IHRoYXQgaXMKPj4gICAgIG5vdyBhbHJlYWR5IGlu IHVzZSBieSBzb21lYm9keSBlbHNlLgo+Pgo+PiBCeSBjbGVhcmluZyB0aGUgcGFnZSB0YWJsZXMg aW1tZWRpYXRlbHkgaW4gc3RlcCAyLCBhIEdQVSBWTSBmYXVsdCB3aWxsCj4+IGJlIHRyaWdnZXJl ZCBpbiBzdGVwIDQgaW5zdGVhZCBvZiB3aWxkIG1lbW9yeSBhY2Nlc3Nlcy4KPj4KPj4gdjI6IHVz ZSBhbWRncHVfYm9fZmVuY2UgZGlyZWN0bHkKPj4KPj4gU2lnbmVkLW9mZi1ieTogTmljb2xhaSBI w6RobmxlIDxuaWNvbGFpLmhhZWhubGVAYW1kLmNvbT4KPj4gLS0tCj4+ICAgZHJpdmVycy9ncHUv ZHJtL2FtZC9hbWRncHUvYW1kZ3B1X2dlbS5jIHwgMTIgKysrKysrKysrKysrCj4+ICAgMSBmaWxl IGNoYW5nZWQsIDEyIGluc2VydGlvbnMoKykKPj4KPj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1 L2RybS9hbWQvYW1kZ3B1L2FtZGdwdV9nZW0uYwo+PiBiL2RyaXZlcnMvZ3B1L2RybS9hbWQvYW1k Z3B1L2FtZGdwdV9nZW0uYwo+PiBpbmRleCA0YTUzYzQzLi44YjBmNWYxOCAxMDA2NDQKPj4gLS0t IGEvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1kZ3B1X2dlbS5jCj4+ICsrKyBiL2RyaXZl cnMvZ3B1L2RybS9hbWQvYW1kZ3B1L2FtZGdwdV9nZW0uYwo+PiBAQCAtMTQ1LDIwICsxNDUsMjEg QEAgdm9pZCBhbWRncHVfZ2VtX29iamVjdF9jbG9zZShzdHJ1Y3QgZHJtX2dlbV9vYmplY3QgKm9i aiwKPj4gICAgICAgc3RydWN0IGFtZGdwdV9ibyAqYm8gPSBnZW1fdG9fYW1kZ3B1X2JvKG9iaik7 Cj4+ICAgICAgIHN0cnVjdCBhbWRncHVfZGV2aWNlICphZGV2ID0gYW1kZ3B1X3R0bV9hZGV2KGJv LT50Ym8uYmRldik7Cj4+ICAgICAgIHN0cnVjdCBhbWRncHVfZnByaXYgKmZwcml2ID0gZmlsZV9w cml2LT5kcml2ZXJfcHJpdjsKPj4gICAgICAgc3RydWN0IGFtZGdwdV92bSAqdm0gPSAmZnByaXYt PnZtOwo+Pgo+PiAgICAgICBzdHJ1Y3QgYW1kZ3B1X2JvX2xpc3RfZW50cnkgdm1fcGQ7Cj4+ICAg ICAgIHN0cnVjdCBsaXN0X2hlYWQgbGlzdCwgZHVwbGljYXRlczsKPj4gICAgICAgc3RydWN0IHR0 bV92YWxpZGF0ZV9idWZmZXIgdHY7Cj4+ICAgICAgIHN0cnVjdCB3d19hY3F1aXJlX2N0eCB0aWNr ZXQ7Cj4+ICAgICAgIHN0cnVjdCBhbWRncHVfYm9fdmEgKmJvX3ZhOwo+PiArICAgIHN0cnVjdCBm ZW5jZSAqZmVuY2UgPSBOVUxMOwo+PiAgICAgICBpbnQgcjsKPj4KPj4gICAgICAgSU5JVF9MSVNU X0hFQUQoJmxpc3QpOwo+PiAgICAgICBJTklUX0xJU1RfSEVBRCgmZHVwbGljYXRlcyk7Cj4+Cj4+ ICAgICAgIHR2LmJvID0gJmJvLT50Ym87Cj4+ICAgICAgIHR2LnNoYXJlZCA9IHRydWU7Cj4+ICAg ICAgIGxpc3RfYWRkKCZ0di5oZWFkLCAmbGlzdCk7Cj4+Cj4+ICAgICAgIGFtZGdwdV92bV9nZXRf cGRfYm8odm0sICZsaXN0LCAmdm1fcGQpOwo+PiBAQCAtMTY2LDIwICsxNjcsMzEgQEAgdm9pZCBh bWRncHVfZ2VtX29iamVjdF9jbG9zZShzdHJ1Y3QgZHJtX2dlbV9vYmplY3QgKm9iaiwKPj4gICAg ICAgciA9IHR0bV9ldV9yZXNlcnZlX2J1ZmZlcnMoJnRpY2tldCwgJmxpc3QsIGZhbHNlLCAmZHVw bGljYXRlcyk7Cj4+ICAgICAgIGlmIChyKSB7Cj4+ICAgICAgICAgICBkZXZfZXJyKGFkZXYtPmRl diwgImxlYWtpbmcgYm8gdmEgYmVjYXVzZSAiCj4+ICAgICAgICAgICAgICAgIndlIGZhaWwgdG8g cmVzZXJ2ZSBibyAoJWQpXG4iLCByKTsKPj4gICAgICAgICAgIHJldHVybjsKPj4gICAgICAgfQo+ PiAgICAgICBib192YSA9IGFtZGdwdV92bV9ib19maW5kKHZtLCBibyk7Cj4+ICAgICAgIGlmIChi b192YSkgewo+PiAgICAgICAgICAgaWYgKC0tYm9fdmEtPnJlZl9jb3VudCA9PSAwKSB7Cj4+ICAg ICAgICAgICAgICAgYW1kZ3B1X3ZtX2JvX3JtdihhZGV2LCBib192YSk7Cj4+ICsKPj4gKyAgICAg ICAgICAgIHIgPSBhbWRncHVfdm1fY2xlYXJfZnJlZWQoYWRldiwgdm0sICZmZW5jZSk7Cj4+ICsg ICAgICAgICAgICBpZiAodW5saWtlbHkocikpIHsKPj4gKyAgICAgICAgICAgICAgICBkZXZfZXJy KGFkZXYtPmRldiwgImZhaWxlZCB0byBjbGVhciBwYWdlICIKPj4gKyAgICAgICAgICAgICAgICAg ICAgInRhYmxlcyBvbiBHRU0gb2JqZWN0IGNsb3NlICglZClcbiIsIHIpOwo+PiArICAgICAgICAg ICAgfQo+PiArCj4+ICsgICAgICAgICAgICBpZiAoZmVuY2UpIHsKPgo+IEkgdGhpbmsgaXQncyBh bHdheXMgdHJ1ZS4KPiBNYXliZSB5b3UgbWVhbiAqZmVuY2U/CgpNeSBmYXVsdCB0byBwaWNrIGEg d3JvbmcgbWFpbCB0aHJlYWQgYXQgdGhlIHNhbWUgdGltZS4KaXQncyBhbHJlYWR5IGEgcG9pbnRl ciwgbm90ICoqZmVuY2UgZGVmaW5lZC4KClJldmlld2VkLWJ5OiBKdW53ZWkgWmhhbmcgPEplcnJ5 LlpoYW5nQGFtZC5jb20+Cgo+Cj4+ICsgICAgICAgICAgICAgICAgYW1kZ3B1X2JvX2ZlbmNlKGJv LCBmZW5jZSwgdHJ1ZSk7Cj4+ICsgICAgICAgICAgICAgICAgZmVuY2VfcHV0KGZlbmNlKTsKPj4g KyAgICAgICAgICAgIH0KPj4gICAgICAgICAgIH0KPj4gICAgICAgfQo+PiAgICAgICB0dG1fZXVf YmFja29mZl9yZXNlcnZhdGlvbigmdGlja2V0LCAmbGlzdCk7Cj4+ICAgfQo+Pgo+PiAgIHN0YXRp YyBpbnQgYW1kZ3B1X2dlbV9oYW5kbGVfbG9ja3VwKHN0cnVjdCBhbWRncHVfZGV2aWNlICphZGV2 LCBpbnQgcikKPj4gICB7Cj4+ICAgICAgIGlmIChyID09IC1FREVBRExLKSB7Cj4+ICAgICAgICAg ICByID0gYW1kZ3B1X2dwdV9yZXNldChhZGV2KTsKPj4gICAgICAgICAgIGlmICghcikKPj4KPiBf X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwo+IGFtZC1nZngg bWFpbGluZyBsaXN0Cj4gYW1kLWdmeEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKPiBodHRwczovL2xp c3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2FtZC1nZngKX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KYW1kLWdmeCBtYWlsaW5nIGxpc3QK YW1kLWdmeEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5v cmcvbWFpbG1hbi9saXN0aW5mby9hbWQtZ2Z4Cg==