From mboxrd@z Thu Jan  1 00:00:00 1970
From: "Yang, Philip" <Philip.Yang-5C7GfCeVMHo@public.gmane.org>
Subject: Re: [PATCH 1/3] drm/amdkfd: support concurrent userptr update for HMM
Date: Thu, 7 Mar 2019 00:54:19 +0000
Message-ID: <eac9f002-ca1e-af38-a15e-e3b667c55452@amd.com>
References: <20190305180915.15216-1-Philip.Yang@amd.com>
 <20190305180915.15216-2-Philip.Yang@amd.com>
 <772b1109-b6db-05cc-4f36-6428724ae4a3@amd.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
Return-path: <amd-gfx-bounces-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org>
In-Reply-To: <772b1109-b6db-05cc-4f36-6428724ae4a3-5C7GfCeVMHo@public.gmane.org>
Content-Language: en-US
Content-ID: <F36DF6D282385C4F8FF918221228B71C-asWib9pRmPqcE4WynfumptQqCkab/8FMAL8bYrjMMd8@public.gmane.org>
List-Id: Discussion list for AMD gfx <amd-gfx.lists.freedesktop.org>
List-Unsubscribe: <https://lists.freedesktop.org/mailman/options/amd-gfx>,
 <mailto:amd-gfx-request-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org?subject=unsubscribe>
List-Archive: <https://lists.freedesktop.org/archives/amd-gfx>
List-Post: <mailto:amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org>
List-Help: <mailto:amd-gfx-request-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org?subject=help>
List-Subscribe: <https://lists.freedesktop.org/mailman/listinfo/amd-gfx>,
 <mailto:amd-gfx-request-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org?subject=subscribe>
Errors-To: amd-gfx-bounces-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
Sender: "amd-gfx" <amd-gfx-bounces-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org>
To: "Kuehling, Felix" <Felix.Kuehling-5C7GfCeVMHo@public.gmane.org>, "amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org" <amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org>

SGkgRmVsaXgsDQoNClRoYW5rcywgdGhlcmUgYXJlIG90aGVyIGNvcm5lciBjYXNlcywgY2FsbCB1
bnRyYWNrIGF0IGVuZCBvZiByZXN0b3JlIA0KdXNlcnB0ciB3b3JrZXIgaXMgYmV0dGVyIHBsYWNl
IHRvIGNsZWFudXAuIEkgd2lsbCBzdWJtaXQgdjIgcGF0Y2gsIHRvIA0KZml4IHRoaXMgaXNzdWUg
Y29tcGxldGVseS4NCg0KUGhpbGlwDQoNCk9uIDIwMTktMDMtMDYgMzowMSBwLm0uLCBLdWVobGlu
ZywgRmVsaXggd3JvdGU6DQo+IEhtbSwgSSdtIG5vdCBzdXJlLiBUaGlzIGNoYW5nZSBwcm9iYWJs
eSBmaXhlcyB0aGlzIGlzc3VlLCBidXQgdGhlcmUgbWF5DQo+IGJlIG90aGVyIHNpbWlsYXIgY29y
bmVyIGNhc2VzIGluIG90aGVyIHNpdHVhdGlvbnMgd2hlcmUgdGhlIHJlc3RvcmUNCj4gd29ya2Vy
IGZhaWxzIGFuZCBuZWVkcyB0byByZXRyeS4gVGhlIGJldHRlciBwbGFjZSB0byBjYWxsIHVudHJh
Y2sgaW4NCj4gYW1kZ3B1X2FtZGtmZF9yZXN0b3JlX3VzZXJwdHJfd29ya2VyIHdvdWxkIGJlIGF0
IHRoZSB2ZXJ5IGVuZC4gQW55dGhpbmcNCj4gdGhhdCdzIGxlZnQgaW4gdGhlIHVzZXJwdHJfaW52
YWxfbGlzdCBhdCB0aGF0IHBvaW50IG5lZWRzIHRvIGJlIHVudHJhY2tlZC4NCj4gDQo+IEZvciBu
b3cgYXMgYSBxdWljayBmaXggZm9yIGFuIHVyZ2VudCBidWcsIHRoaXMgY2hhbmdlIGlzIFJldmll
d2VkLWJ5Og0KPiBGZWxpeCBLdWVobGluZyA8RmVsaXguS3VlaGxpbmdAYW1kLmNvbT4uIEJ1dCBw
bGVhc2UgcmV2aXNpdCB0aGlzIGFuZA0KPiBjaGVjayBpZiB0aGVyZSBhcmUgc2ltaWxhciBjb3Ju
ZXIgY2FzZXMgYXMgSSBleHBsYWluZWQgYWJvdmUuDQo+IA0KPiBSZWdhcmRzLA0KPiAgIMKgIEZl
bGl4DQo+IA0KPiBPbiAzLzUvMjAxOSAxOjA5IFBNLCBZYW5nLCBQaGlsaXAgd3JvdGU6DQo+PiBV
c2VycHRyIHJlc3RvcmUgbWF5IGhhdmUgY29uY3VycmVudCB1c2VycHRyIGludmFsaWRhdGlvbiBh
ZnRlcg0KPj4gaG1tX3ZtYV9mYXVsdCBhZGRzIHRoZSByYW5nZSB0byB0aGUgaG1tLT5yYW5nZXMg
bGlzdCwgbmVlZHMgY2FsbA0KPj4gaG1tX3ZtYV9yYW5nZV9kb25lIHRvIHJlbW92ZSB0aGUgcmFu
Z2UgZnJvbSBobW0tPnJhbmdlcyBsaXN0IGZpcnN0LA0KPj4gdGhlbiByZXNjaGVkdWxlIHRoZSBy
ZXN0b3JlIHdvcmtlci4gT3RoZXJ3aXNlIGhtbV92bWFfZmF1bHQgd2lsbCBhZGQNCj4+IHNhbWUg
cmFuZ2UgdG8gdGhlIGxpc3QsIHRoaXMgd2lsbCBjYXVzZSBsb29wIGluIHRoZSBsaXN0IGJlY2F1
c2UNCj4+IHJhbmdlLT5uZXh0IHBvaW50IHRvIHJhbmdlIGl0c2VsZi4NCj4+DQo+PiBBZGQgZnVu
Y3Rpb24gdW50cmFja19pbnZhbGlkX3VzZXJfcGFnZXMgdG8gcmVkdWNlIGNvZGUgZHVwbGljYXRp
b24uDQo+Pg0KPj4gQ2hhbmdlLUlkOiBJMzE0MDc3MzlkYzEwNTU0ZjhlNDE4YzdhMGUwNDE1ZDNk
OTU1NTJmMQ0KPj4gU2lnbmVkLW9mZi1ieTogUGhpbGlwIFlhbmcgPFBoaWxpcC5ZYW5nQGFtZC5j
b20+DQo+PiAtLS0NCj4+ICAgIC4uLi9ncHUvZHJtL2FtZC9hbWRncHUvYW1kZ3B1X2FtZGtmZF9n
cHV2bS5jICB8IDI4ICsrKysrKysrKysrKysrLS0tLS0NCj4+ICAgIDEgZmlsZSBjaGFuZ2VkLCAy
MSBpbnNlcnRpb25zKCspLCA3IGRlbGV0aW9ucygtKQ0KPj4NCj4+IGRpZmYgLS1naXQgYS9kcml2
ZXJzL2dwdS9kcm0vYW1kL2FtZGdwdS9hbWRncHVfYW1ka2ZkX2dwdXZtLmMgYi9kcml2ZXJzL2dw
dS9kcm0vYW1kL2FtZGdwdS9hbWRncHVfYW1ka2ZkX2dwdXZtLmMNCj4+IGluZGV4IDMxNGMwNDhm
Y2FjNi4uNzgzZDc2MGNjZmUzIDEwMDY0NA0KPj4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2FtZC9h
bWRncHUvYW1kZ3B1X2FtZGtmZF9ncHV2bS5jDQo+PiArKysgYi9kcml2ZXJzL2dwdS9kcm0vYW1k
L2FtZGdwdS9hbWRncHVfYW1ka2ZkX2dwdXZtLmMNCj4+IEBAIC0xNzMxLDYgKzE3MzEsMjMgQEAg
c3RhdGljIGludCB1cGRhdGVfaW52YWxpZF91c2VyX3BhZ2VzKHN0cnVjdCBhbWRrZmRfcHJvY2Vz
c19pbmZvICpwcm9jZXNzX2luZm8sDQo+PiAgICAJcmV0dXJuIDA7DQo+PiAgICB9DQo+PiAgICAN
Cj4+ICsvKiBSZW1vdmUgaW52YWxpZCB1c2VycHRyIEJPcyBmcm9tIGhtbSB0cmFjayBsaXN0DQo+
PiArICoNCj4+ICsgKiBTdG9wIEhNTSB0cmFjayB0aGUgdXNlcnB0ciB1cGRhdGUNCj4+ICsgKi8N
Cj4+ICtzdGF0aWMgdm9pZCB1bnRyYWNrX2ludmFsaWRfdXNlcl9wYWdlcyhzdHJ1Y3QgYW1ka2Zk
X3Byb2Nlc3NfaW5mbyAqcHJvY2Vzc19pbmZvKQ0KPj4gK3sNCj4+ICsJc3RydWN0IGtnZF9tZW0g
Km1lbSwgKnRtcF9tZW07DQo+PiArCXN0cnVjdCBhbWRncHVfYm8gKmJvOw0KPj4gKw0KPj4gKwls
aXN0X2Zvcl9lYWNoX2VudHJ5X3NhZmUobWVtLCB0bXBfbWVtLA0KPj4gKwkJCQkgJnByb2Nlc3Nf
aW5mby0+dXNlcnB0cl9pbnZhbF9saXN0LA0KPj4gKwkJCQkgdmFsaWRhdGVfbGlzdC5oZWFkKSB7
DQo+PiArCQlibyA9IG1lbS0+Ym87DQo+PiArCQlhbWRncHVfdHRtX3R0X2dldF91c2VyX3BhZ2Vz
X2RvbmUoYm8tPnRiby50dG0pOw0KPj4gKwl9DQo+PiArfQ0KPj4gKw0KPj4gICAgLyogVmFsaWRh
dGUgaW52YWxpZCB1c2VycHRyIEJPcw0KPj4gICAgICoNCj4+ICAgICAqIFZhbGlkYXRlcyBCT3Mg
b24gdGhlIHVzZXJwdHJfaW52YWxfbGlzdCwgYW5kIG1vdmVzIHRoZW0gYmFjayB0byB0aGUNCj4+
IEBAIC0xODQ4LDEyICsxODY1LDcgQEAgc3RhdGljIGludCB2YWxpZGF0ZV9pbnZhbGlkX3VzZXJf
cGFnZXMoc3RydWN0IGFtZGtmZF9wcm9jZXNzX2luZm8gKnByb2Nlc3NfaW5mbykNCj4+ICAgIG91
dF9mcmVlOg0KPj4gICAgCWtmcmVlKHBkX2JvX2xpc3RfZW50cmllcyk7DQo+PiAgICBvdXRfbm9f
bWVtOg0KPj4gLQlsaXN0X2Zvcl9lYWNoX2VudHJ5X3NhZmUobWVtLCB0bXBfbWVtLA0KPj4gLQkJ
CQkgJnByb2Nlc3NfaW5mby0+dXNlcnB0cl9pbnZhbF9saXN0LA0KPj4gLQkJCQkgdmFsaWRhdGVf
bGlzdC5oZWFkKSB7DQo+PiAtCQlibyA9IG1lbS0+Ym87DQo+PiAtCQlhbWRncHVfdHRtX3R0X2dl
dF91c2VyX3BhZ2VzX2RvbmUoYm8tPnRiby50dG0pOw0KPj4gLQl9DQo+PiArCXVudHJhY2tfaW52
YWxpZF91c2VyX3BhZ2VzKHByb2Nlc3NfaW5mbyk7DQo+PiAgICANCj4+ICAgIAlyZXR1cm4gcmV0
Ow0KPj4gICAgfQ0KPj4gQEAgLTE4OTcsOCArMTkwOSwxMCBAQCBzdGF0aWMgdm9pZCBhbWRncHVf
YW1ka2ZkX3Jlc3RvcmVfdXNlcnB0cl93b3JrZXIoc3RydWN0IHdvcmtfc3RydWN0ICp3b3JrKQ0K
Pj4gICAgCSAqIGFuZCB3ZSBjYW4ganVzdCByZXN0YXJ0IHRoZSBxdWV1ZXMuDQo+PiAgICAJICov
DQo+PiAgICAJaWYgKCFsaXN0X2VtcHR5KCZwcm9jZXNzX2luZm8tPnVzZXJwdHJfaW52YWxfbGlz
dCkpIHsNCj4+IC0JCWlmIChhdG9taWNfcmVhZCgmcHJvY2Vzc19pbmZvLT5ldmljdGVkX2Jvcykg
IT0gZXZpY3RlZF9ib3MpDQo+PiArCQlpZiAoYXRvbWljX3JlYWQoJnByb2Nlc3NfaW5mby0+ZXZp
Y3RlZF9ib3MpICE9IGV2aWN0ZWRfYm9zKSB7DQo+PiArCQkJdW50cmFja19pbnZhbGlkX3VzZXJf
cGFnZXMocHJvY2Vzc19pbmZvKTsNCj4+ICAgIAkJCWdvdG8gdW5sb2NrX291dDsgLyogQ29uY3Vy
cmVudCBldmljdGlvbiwgdHJ5IGFnYWluICovDQo+PiArCQl9DQo+PiAgICANCj4+ICAgIAkJaWYg
KHZhbGlkYXRlX2ludmFsaWRfdXNlcl9wYWdlcyhwcm9jZXNzX2luZm8pKQ0KPj4gICAgCQkJZ290
byB1bmxvY2tfb3V0Ow0KX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f
X19fX18KYW1kLWdmeCBtYWlsaW5nIGxpc3QKYW1kLWdmeEBsaXN0cy5mcmVlZGVza3RvcC5vcmcK
aHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9hbWQtZ2Z4