From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?Q?Christian_K=c3=b6nig?= Subject: Re: [PATCH] drm/amdgpu: use new HMM APIs and helpers Date: Mon, 3 Jun 2019 12:17:13 +0200 Message-ID: <972ef129-3dae-d5eb-100f-b9be83d0afcc@gmail.com> References: <20190530144049.1996-1-Philip.Yang@amd.com> <704410a5-be01-f423-11ef-01a640cd469c@amd.com> <9ae26883-4326-c60f-9a8e-d95d0d458e31@amd.com> <9f5f4060-5f8f-b688-2cc2-39aca92c7505@amd.com> 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: <9f5f4060-5f8f-b688-2cc2-39aca92c7505-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: "Kuehling, Felix" , "Yang, Philip" , "amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org" QW0gMDEuMDYuMTkgdW0gMDA6MDEgc2NocmllYiBLdWVobGluZywgRmVsaXg6Cj4gT24gMjAxOS0w NS0zMSA1OjMyIHAubS4sIFlhbmcsIFBoaWxpcCB3cm90ZToKPj4gT24gMjAxOS0wNS0zMSAzOjQy IHAubS4sIEt1ZWhsaW5nLCBGZWxpeCB3cm90ZToKPj4+IE9uIDIwMTktMDUtMzEgMToyOCBwLm0u LCBZYW5nLCBQaGlsaXAgd3JvdGU6Cj4+Pj4gT24gMjAxOS0wNS0zMCA2OjM2IHAubS4sIEt1ZWhs aW5nLCBGZWxpeCB3cm90ZToKPj4+Pj4+ICAgICAgICAKPj4+Pj4+ICAgICAgICAjaWYgSVNfRU5B QkxFRChDT05GSUdfRFJNX0FNREdQVV9VU0VSUFRSKQo+Pj4+Pj4gLQlpZiAoZ3R0LT5yYW5nZXMg JiYKPj4+Pj4+IC0JICAgIHR0bS0+cGFnZXNbMF0gPT0gaG1tX3Bmbl90b19wYWdlKCZndHQtPnJh bmdlc1swXSwKPj4+Pj4+IC0JCQkJCSAgICAgZ3R0LT5yYW5nZXNbMF0ucGZuc1swXSkpCj4+Pj4+ PiArCWlmIChndHQtPnJhbmdlICYmCj4+Pj4+PiArCSAgICB0dG0tPnBhZ2VzWzBdID09IGhtbV9k ZXZpY2VfZW50cnlfdG9fcGFnZShndHQtPnJhbmdlLAo+Pj4+Pj4gKwkJCQkJCSAgICAgIGd0dC0+ cmFuZ2UtPnBmbnNbMF0pKQo+Pj4+PiBJIHRoaW5rIGp1c3QgY2hlY2tpbmcgZ3R0LT5yYW5nZSBo ZXJlIGlzIGVub3VnaC4gSWYgZ3R0LT5yYW5nZSBpcyBub3QKPj4+Pj4gTlVMTCBoZXJlLCB3ZSdy ZSBsZWFraW5nIG1lbW9yeS4KPj4+Pj4KPj4+PiBJZiBqdXN0IGNoZWNraW5nIGd0dC0+cmFuZ2Us IHRoZXJlIGlzIGEgZmFsc2Ugd2FybmluZyBpbiBhbWRncHVfdGVzdAo+Pj4+IHVzZXJwdHIgY2Fz ZSBpbiBhbWRncHVfY3NfbGlzdF92YWxpZGF0ZSBwYXRoLiBJZiB1c2VycHRyIGlzIGludmFsaWRh dGVkLAo+Pj4+IHRoZW4gdHRtLT5wYWdlc1swXSBpcyBvdXRkYXRlZCBwYWdlcywgbG9iai0+dXNl cl9wYWdlcyBpcyBuZXcgcGFnZXMsIGl0Cj4+Pj4gZ29lcyB0byB0dG1fdHRfdW5iaW5kIGZpcnN0 IHRvIHVucGluIG9sZCBwYWdlcyAodGhpcyBjYXVzZXMgZmFsc2UKPj4+PiB3YXJuaW5nKSB0aGVu IGNhbGwgYW1kZ3B1X3R0bV90dF9zZXRfdXNlcl9wYWdlcy4KPj4+IEJ1dCBkb2Vzbid0IHRoYXQg bWVhbiB3ZSdyZSBsZWFraW5nIHRoZSBndHQtPnJhbmdlIHNvbWV3aGVyZT8KPj4+Cj4+IHR0bV90 dF91bmJpbmQgaXMgY2FsbGVkIGZyb20gdHRtX3R0X2Rlc3Ryb3kgYW5kIGFtZGdwdV9jc19saXN0 X3ZhbGlkYXRlLAo+PiB0aGUgbGF0ZXIgb25lIGNhdXNlcyBmYWxzZSB3YXJuaW5nLiB0dG1fdHRt X2Rlc3RvcnkgcGF0aCBpcyBmaW5lIHRvIG9ubHkKPj4gY2hlY2sgZ3R0LT5yYW5nZS4KPj4KPj4g RG91YmxlIGNoZWNrZWQsIGFtZGdwdV90dG1fdHRfZ2V0X3VzZXJfcGFnZXMgYW5kCj4+IGFtZGdw dV90dG1fdHRfZ2V0X3VzZXJfcGFnZXNfZG9uZSBhbHdheXMgbWF0Y2ggaW4gYm90aCBwYXRocywg c28gbm8gbGVhawo+PiBndHQtPnJhbmdlLgo+Pgo+PiAxLiBhbWRncHVfZ2VtX3VzZXJwdHJfaW9j dGwKPj4gICAgICAgICAgYW1kZ3B1X3R0bV90dF9nZXRfdXNlcl9wYWdlcwo+PiAgICAgICAgICAg ICAgdHRtLT5wYWdlcyBmb3IgdXNlcnB0ciBwYWdlcwo+PiAgICAgICAgICBhbWRncHVfdHRtX3R0 X2dldF91c2VyX3BhZ2VzX2RvbmUKPj4KPj4gMi4gYW1kZ3B1X2NzX2lvY3RsCj4+ICAgICAgICAg IGFtZGdwdV9jc19wYXJzZXJfYm9zCj4+ICAgICAgICAgICAgICBhbWRncHVfdHRtX3R0X2dldF91 c2VyX3BhZ2VzCj4+ICAgICAgICAgICAgICBpZiAodXNlcnBhZ2VfaW52YWxpZGF0ZWQpCj4+ICAg ICAgICAgICAgICAgICAgZS0+dXNlcl9wYWdlcyBmb3IgbmV3IHBhZ2VzCj4+ICAgICAgICAgICAg ICBhbWRncHVfY3NfbGlzdF92YWxpZGF0ZQo+PiAgICAgICAgICAgICAgICAgIGlmICh1c2VycGFn ZV9pbnZhbGlkYXRlZCkKPj4gICAgICAgICAgICAgICAgICAgICB0dG1fdHRfdW5iaW5kIHR0bS0+ cGFnZXMgLy8gdGhpcyBjYXVzZXMgd2FybmluZwo+PiAgICAgICAgICAgICAgICAgICAgIGFtZGdw dV90dG1fdHRfc2V0X3VzZXJfcGFnZXModHRtLT5wYWdlcywgZS0+dXNlcl9wYWdlcykKPiBIbW0s IEkgdGhpbmsgYW1kZ3B1X2NzIGlzIGRvaW5nIHNvbWV0aGluZyB3ZWlyZCB0aGVyZS4gSXQgZG9l cyBzb21lCj4gZG91YmxlIGJvb2sta2VlcGluZyBvZiB0aGUgdXNlciBwYWdlcyBpbiB0aGUgQk8g bGlzdCBhbmQgdGhlIFRUTSBCTy4gV2UKPiBkaWQgc29tZXRoaW5nIHNpbWlsYXIgZm9yIEtGRCBh bmQgc2ltcGxpZmllZCBpdCB3aGVuIG1vdmluZyB0byBITU0uIEl0Cj4gY291bGQgcHJvYmFibHkg YmUgc2ltcGxpZmllZCBmb3IgYW1kZ3B1X2NzIGFzIHdlbGwuIEJ1dCBub3QgaW4gdGhpcwo+IHBh dGNoIHNlcmllcy4KClRoYXQgYWN0dWFsbHkgc291bmRzIGxpa2UgYSBidWcgdG8gbWUuCgpJdCBp cyBtb3N0IGxpa2VseSBhIGxlZnRvdmVyIGZyb20gdGhlIHRpbWUgd2hlbiB3ZSBjb3VsZG4ndCBn ZXQgdGhlIApwYWdlcyBmb3IgYSBCTyB3aGlsZSB0aGUgQk8gd2FzIHJlc2VydmVkLgoKR29pbmcg dG8gdGFrZSBhIGNsb3NlciBsb29rLApDaHJpc3RpYW4uCgo+Cj4gSSdsbCByZXZpZXcgeW91ciB1 cGRhdGVkIGNoYW5nZS4KPgo+IFRoYW5rcywKPiAgIMKgIEZlbGl4Cj4KPgo+PiAgICAgICAgICBh bWRncHVfY3Nfc3VibWl0Cj4+ICAgICAgICAgICAgICBhbWRncHVfdHRtX3R0X2dldF91c2VyX3Bh Z2VzX2RvbmUKPj4KPj4+IFJlZ2FyZHMsCj4+PiAgICAgwqAgRmVsaXgKPj4+Cj4+Pgo+Pj4+IEkg d2lsbCBzdWJtaXQgcGF0Y2ggdjIsIHRvIGFkZCByZXRyeSBpZiBobW1fcmFuZ2VfZmF1bHQgcmV0 dXJucyAtRUFHQUlOLgo+Pj4+IHVzZSBremFsbG9jIHRvIGFsbG9jYXRlIHNtYWxsIHNpemUgcmFu Z2UuCj4+Pj4KPj4+PiBUaGFua3MsCj4+Pj4gUGhpbGlwCj4+Pj4KPj4+Pj4gUmVnYXJkcywKPj4+ Pj4gICAgICAgwqAgRmVsaXgKPj4+Pj4KPj4+Pj4KPiBfX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fXwo+IGFtZC1nZnggbWFpbGluZyBsaXN0Cj4gYW1kLWdmeEBs aXN0cy5mcmVlZGVza3RvcC5vcmcKPiBodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWls bWFuL2xpc3RpbmZvL2FtZC1nZngKCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fCmFtZC1nZnggbWFpbGluZyBsaXN0CmFtZC1nZnhAbGlzdHMuZnJlZWRlc2t0 b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vYW1k LWdmeA==