From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?Q?Christian_K=c3=b6nig?= Subject: Re: [PATCH 1/2] drm/amdgpu: return bo itself if userptr is cpu addr of bo (v3) Date: Mon, 30 Jul 2018 12:47:48 +0200 Message-ID: <3d423f44-e47a-e70c-dfd9-80c1ff843e45@gmail.com> References: <1532944950-28619-1-git-send-email-Jerry.Zhang@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: <1532944950-28619-1-git-send-email-Jerry.Zhang-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: Junwei Zhang , amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org Cc: Chunming Zhou , christian.koenig-5C7GfCeVMHo@public.gmane.org QW0gMzAuMDcuMjAxOCB1bSAxMjowMiBzY2hyaWViIEp1bndlaSBaaGFuZzoKPiBGcm9tOiBDaHVu bWluZyBaaG91IDxEYXZpZDEuWmhvdUBhbWQuY29tPgo+Cj4gdjI6IGdldCBvcmlnaW5hbCBnZW0g aGFuZGxlIGZyb20gZ29iago+IHYzOiB1cGRhdGUgZmluZCBibyBkYXRhIHN0cnVjdHVyZSBhcyB1 bmlvbihpbiwgb3V0KQo+ICAgICAgc2ltcGx5IHNvbWUgY29kZSBsb2dpYwoKRG8gd2Ugbm93IGhh dmUgYW4gb3BlbiBzb3VyY2UgdXNlciBmb3IgdGhpcywgc28gdGhhdCB3ZSBjYW4gdXBzdHJlYW0g aXQ/IApPbmUgbW9yZSBwb2ludCBiZWxvdy4KCj4KPiBTaWduZWQtb2ZmLWJ5OiBDaHVubWluZyBa aG91IDxEYXZpZDEuWmhvdUBhbWQuY29tPgo+IFNpZ25lZC1vZmYtYnk6IEp1bndlaSBaaGFuZyA8 SmVycnkuWmhhbmdAYW1kLmNvbT4gKHYzKQo+IFJldmlld2VkLWJ5OiBDaHJpc3RpYW4gS8O2bmln IDxjaHJpc3RpYW4ua29lbmlnQGFtZC5jb20+Cj4gUmV2aWV3ZWQtYnk6IEphbW15IFpob3UgPEph bW15Llpob3VAYW1kLmNvbT4KPiAtLS0KPiAgIGRyaXZlcnMvZ3B1L2RybS9hbWQvYW1kZ3B1L2Ft ZGdwdS5oICAgICB8ICAyICsrCj4gICBkcml2ZXJzL2dwdS9kcm0vYW1kL2FtZGdwdS9hbWRncHVf Z2VtLmMgfCA2MyArKysrKysrKysrKysrKysrKysrKysrKysrKysrKysrKysKPiAgIGRyaXZlcnMv Z3B1L2RybS9hbWQvYW1kZ3B1L2FtZGdwdV9rbXMuYyB8ICAzICstCj4gICBpbmNsdWRlL3VhcGkv ZHJtL2FtZGdwdV9kcm0uaCAgICAgICAgICAgfCAyMSArKysrKysrKysrKwo+ICAgNCBmaWxlcyBj aGFuZ2VkLCA4OCBpbnNlcnRpb25zKCspLCAxIGRlbGV0aW9uKC0pCj4KPiBkaWZmIC0tZ2l0IGEv ZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1kZ3B1LmggYi9kcml2ZXJzL2dwdS9kcm0vYW1k L2FtZGdwdS9hbWRncHUuaAo+IGluZGV4IDRjZDIwZTcuLjQ2YzM3MGIgMTAwNjQ0Cj4gLS0tIGEv ZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1kZ3B1LmgKPiArKysgYi9kcml2ZXJzL2dwdS9k cm0vYW1kL2FtZGdwdS9hbWRncHUuaAo+IEBAIC0xMjEzLDYgKzEyMTMsOCBAQCBpbnQgYW1kZ3B1 X2dlbV9pbmZvX2lvY3RsKHN0cnVjdCBkcm1fZGV2aWNlICpkZXYsIHZvaWQgKmRhdGEsCj4gICAJ CQkgIHN0cnVjdCBkcm1fZmlsZSAqZmlscCk7Cj4gICBpbnQgYW1kZ3B1X2dlbV91c2VycHRyX2lv Y3RsKHN0cnVjdCBkcm1fZGV2aWNlICpkZXYsIHZvaWQgKmRhdGEsCj4gICAJCQlzdHJ1Y3QgZHJt X2ZpbGUgKmZpbHApOwo+ICtpbnQgYW1kZ3B1X2dlbV9maW5kX2JvX2J5X2NwdV9tYXBwaW5nX2lv Y3RsKHN0cnVjdCBkcm1fZGV2aWNlICpkZXYsIHZvaWQgKmRhdGEsCj4gKwkJCQkJICAgIHN0cnVj dCBkcm1fZmlsZSAqZmlscCk7Cj4gICBpbnQgYW1kZ3B1X2dlbV9tbWFwX2lvY3RsKHN0cnVjdCBk cm1fZGV2aWNlICpkZXYsIHZvaWQgKmRhdGEsCj4gICAJCQkgIHN0cnVjdCBkcm1fZmlsZSAqZmls cCk7Cj4gICBpbnQgYW1kZ3B1X2dlbV93YWl0X2lkbGVfaW9jdGwoc3RydWN0IGRybV9kZXZpY2Ug KmRldiwgdm9pZCAqZGF0YSwKPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRn cHUvYW1kZ3B1X2dlbS5jIGIvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1kZ3B1X2dlbS5j Cj4gaW5kZXggNzE3OTJkOC4uYmFlODQxNyAxMDA2NDQKPiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0v YW1kL2FtZGdwdS9hbWRncHVfZ2VtLmMKPiArKysgYi9kcml2ZXJzL2dwdS9kcm0vYW1kL2FtZGdw dS9hbWRncHVfZ2VtLmMKPiBAQCAtMjg4LDYgKzI4OCw2OSBAQCBpbnQgYW1kZ3B1X2dlbV9jcmVh dGVfaW9jdGwoc3RydWN0IGRybV9kZXZpY2UgKmRldiwgdm9pZCAqZGF0YSwKPiAgIAlyZXR1cm4g MDsKPiAgIH0KPiAgIAo+ICtzdGF0aWMgaW50IGFtZGdwdV9nZW1fZ2V0X2hhbmRsZV9mcm9tX29i amVjdChzdHJ1Y3QgZHJtX2ZpbGUgKmZpbHAsCj4gKwkJCQkJICAgICBzdHJ1Y3QgZHJtX2dlbV9v YmplY3QgKm9iaikKPiArewo+ICsJaW50IGk7Cj4gKwlzdHJ1Y3QgZHJtX2dlbV9vYmplY3QgKnRt cDsKPiArCj4gKwlzcGluX2xvY2soJmZpbHAtPnRhYmxlX2xvY2spOwo+ICsJaWRyX2Zvcl9lYWNo X2VudHJ5KCZmaWxwLT5vYmplY3RfaWRyLCB0bXAsIGkpIHsKPiArCQlpZiAob2JqID09IHRtcCkg ewo+ICsJCQlkcm1fZ2VtX29iamVjdF9yZWZlcmVuY2Uob2JqKTsKPiArCQkJc3Bpbl91bmxvY2so JmZpbHAtPnRhYmxlX2xvY2spOwo+ICsJCQlyZXR1cm4gaTsKPiArCQl9Cj4gKwl9CgpQbGVhc2Ug ZG91YmxlIGNoZWNrIGlmIHRoYXQgaXMgc3RpbGwgdXAgdG8gZGF0ZS4KCkkgdGhpbmsgd2UgY291 bGQgYXMgd2VsbCB0cnkgdG8gdXNlIHRoZSBETUEtYnVmIGhhbmRsZSB0cmVlIGZvciB0aGF0LgoK Q2hyaXN0aWFuLgoKPiArCXNwaW5fdW5sb2NrKCZmaWxwLT50YWJsZV9sb2NrKTsKPiArCj4gKwly ZXR1cm4gMDsKPiArfQo+ICsKPiAraW50IGFtZGdwdV9nZW1fZmluZF9ib19ieV9jcHVfbWFwcGlu Z19pb2N0bChzdHJ1Y3QgZHJtX2RldmljZSAqZGV2LCB2b2lkICpkYXRhLAo+ICsJCQkJCSAgICBz dHJ1Y3QgZHJtX2ZpbGUgKmZpbHApCj4gK3sKPiArCXVuaW9uIGRybV9hbWRncHVfZ2VtX2ZpbmRf Ym8gKmFyZ3MgPSBkYXRhOwo+ICsJc3RydWN0IGRybV9nZW1fb2JqZWN0ICpnb2JqOwo+ICsJc3Ry dWN0IGFtZGdwdV9ibyAqYm87Cj4gKwlzdHJ1Y3QgdHRtX2J1ZmZlcl9vYmplY3QgKnRibzsKPiAr CXN0cnVjdCB2bV9hcmVhX3N0cnVjdCAqdm1hOwo+ICsJdWludDMyX3QgaGFuZGxlOwo+ICsJaW50 IHI7Cj4gKwo+ICsJaWYgKG9mZnNldF9pbl9wYWdlKGFyZ3MtPmluLmFkZHIgfCBhcmdzLT5pbi5z aXplKSkKPiArCQlyZXR1cm4gLUVJTlZBTDsKPiArCj4gKwlkb3duX3JlYWQoJmN1cnJlbnQtPm1t LT5tbWFwX3NlbSk7Cj4gKwl2bWEgPSBmaW5kX3ZtYShjdXJyZW50LT5tbSwgYXJncy0+aW4uYWRk cik7Cj4gKwlpZiAoIXZtYSB8fCB2bWEtPnZtX2ZpbGUgIT0gZmlscC0+ZmlscCB8fAo+ICsJICAg IChhcmdzLT5pbi5zaXplID4gKHZtYS0+dm1fZW5kIC0gYXJncy0+aW4uYWRkcikpKSB7Cj4gKwkJ YXJncy0+b3V0LmhhbmRsZSA9IDA7Cj4gKwkJdXBfcmVhZCgmY3VycmVudC0+bW0tPm1tYXBfc2Vt KTsKPiArCQlyZXR1cm4gLUVJTlZBTDsKPiArCX0KPiArCWFyZ3MtPm91dC5vZmZzZXQgPSBhcmdz LT5pbi5hZGRyIC0gdm1hLT52bV9zdGFydDsKPiArCj4gKwl0Ym8gPSB2bWEtPnZtX3ByaXZhdGVf ZGF0YTsKPiArCWJvID0gY29udGFpbmVyX29mKHRibywgc3RydWN0IGFtZGdwdV9ibywgdGJvKTsK PiArCWFtZGdwdV9ib19yZWYoYm8pOwo+ICsJZ29iaiA9ICZiby0+Z2VtX2Jhc2U7Cj4gKwo+ICsJ aGFuZGxlID0gYW1kZ3B1X2dlbV9nZXRfaGFuZGxlX2Zyb21fb2JqZWN0KGZpbHAsIGdvYmopOwo+ ICsJaWYgKCFoYW5kbGUpIHsKPiArCQlyID0gZHJtX2dlbV9oYW5kbGVfY3JlYXRlKGZpbHAsIGdv YmosICZoYW5kbGUpOwo+ICsJCWlmIChyKSB7Cj4gKwkJCURSTV9FUlJPUigiY3JlYXRlIGdlbSBo YW5kbGUgZmFpbGVkXG4iKTsKPiArCQkJdXBfcmVhZCgmY3VycmVudC0+bW0tPm1tYXBfc2VtKTsK PiArCQkJcmV0dXJuIHI7Cj4gKwkJfQo+ICsJfQo+ICsJYXJncy0+b3V0LmhhbmRsZSA9IGhhbmRs ZTsKPiArCXVwX3JlYWQoJmN1cnJlbnQtPm1tLT5tbWFwX3NlbSk7Cj4gKwo+ICsJcmV0dXJuIDA7 Cj4gK30KPiArCj4gICBpbnQgYW1kZ3B1X2dlbV91c2VycHRyX2lvY3RsKHN0cnVjdCBkcm1fZGV2 aWNlICpkZXYsIHZvaWQgKmRhdGEsCj4gICAJCQkgICAgIHN0cnVjdCBkcm1fZmlsZSAqZmlscCkK PiAgIHsKPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1kZ3B1X2tt cy5jIGIvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1kZ3B1X2ttcy5jCj4gaW5kZXggNzk1 Njg0OC4uMWJkMmNjMSAxMDA2NDQKPiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vYW1kL2FtZGdwdS9h bWRncHVfa21zLmMKPiArKysgYi9kcml2ZXJzL2dwdS9kcm0vYW1kL2FtZGdwdS9hbWRncHVfa21z LmMKPiBAQCAtMTEwMCw3ICsxMTAwLDggQEAgdm9pZCBhbWRncHVfZGlzYWJsZV92Ymxhbmtfa21z KHN0cnVjdCBkcm1fZGV2aWNlICpkZXYsIHVuc2lnbmVkIGludCBwaXBlKQo+ICAgCURSTV9JT0NU TF9ERUZfRFJWKEFNREdQVV9HRU1fVkEsIGFtZGdwdV9nZW1fdmFfaW9jdGwsIERSTV9BVVRIfERS TV9SRU5ERVJfQUxMT1cpLAo+ICAgCURSTV9JT0NUTF9ERUZfRFJWKEFNREdQVV9HRU1fT1AsIGFt ZGdwdV9nZW1fb3BfaW9jdGwsIERSTV9BVVRIfERSTV9SRU5ERVJfQUxMT1cpLAo+ICAgCURSTV9J T0NUTF9ERUZfRFJWKEFNREdQVV9HRU1fVVNFUlBUUiwgYW1kZ3B1X2dlbV91c2VycHRyX2lvY3Rs LCBEUk1fQVVUSHxEUk1fUkVOREVSX0FMTE9XKSwKPiAtCURSTV9JT0NUTF9ERUZfRFJWKEFNREdQ VV9GUkVFU1lOQywgYW1kZ3B1X2Rpc3BsYXlfZnJlZXN5bmNfaW9jdGwsIERSTV9NQVNURVIpCj4g KwlEUk1fSU9DVExfREVGX0RSVihBTURHUFVfRlJFRVNZTkMsIGFtZGdwdV9kaXNwbGF5X2ZyZWVz eW5jX2lvY3RsLCBEUk1fTUFTVEVSKSwKPiArCURSTV9JT0NUTF9ERUZfRFJWKEFNREdQVV9HRU1f RklORF9CTywgYW1kZ3B1X2dlbV9maW5kX2JvX2J5X2NwdV9tYXBwaW5nX2lvY3RsLCBEUk1fQVVU SHxEUk1fUkVOREVSX0FMTE9XKQo+ICAgfTsKPiAgIGNvbnN0IGludCBhbWRncHVfbWF4X2ttc19p b2N0bCA9IEFSUkFZX1NJWkUoYW1kZ3B1X2lvY3Rsc19rbXMpOwo+ICAgCj4gZGlmZiAtLWdpdCBh L2luY2x1ZGUvdWFwaS9kcm0vYW1kZ3B1X2RybS5oIGIvaW5jbHVkZS91YXBpL2RybS9hbWRncHVf ZHJtLmgKPiBpbmRleCA5NDQ0NGVlLi4wMDBjNDE1IDEwMDY0NAo+IC0tLSBhL2luY2x1ZGUvdWFw aS9kcm0vYW1kZ3B1X2RybS5oCj4gKysrIGIvaW5jbHVkZS91YXBpL2RybS9hbWRncHVfZHJtLmgK PiBAQCAtNTQsNiArNTQsNyBAQAo+ICAgI2RlZmluZSBEUk1fQU1ER1BVX1ZNCQkJMHgxMwo+ICAg I2RlZmluZSBEUk1fQU1ER1BVX0ZFTkNFX1RPX0hBTkRMRQkweDE0Cj4gICAjZGVmaW5lIERSTV9B TURHUFVfU0NIRUQJCTB4MTUKPiArI2RlZmluZSBEUk1fQU1ER1BVX0dFTV9GSU5EX0JPCQkweDE2 Cj4gICAvKiBub3QgdXBzdHJlYW0gKi8KPiAgICNkZWZpbmUgRFJNX0FNREdQVV9GUkVFU1lOQwkg ICAgICAgIDB4NWQKPiAgIAo+IEBAIC03NCw2ICs3NSw3IEBACj4gICAjZGVmaW5lIERSTV9JT0NU TF9BTURHUFVfRkVOQ0VfVE9fSEFORExFIERSTV9JT1dSKERSTV9DT01NQU5EX0JBU0UgKyBEUk1f QU1ER1BVX0ZFTkNFX1RPX0hBTkRMRSwgdW5pb24gZHJtX2FtZGdwdV9mZW5jZV90b19oYW5kbGUp Cj4gICAjZGVmaW5lIERSTV9JT0NUTF9BTURHUFVfU0NIRUQJCURSTV9JT1coRFJNX0NPTU1BTkRf QkFTRSArIERSTV9BTURHUFVfU0NIRUQsIHVuaW9uIGRybV9hbWRncHVfc2NoZWQpCj4gICAjZGVm aW5lIERSTV9JT0NUTF9BTURHUFVfRlJFRVNZTkMJRFJNX0lPV1IoRFJNX0NPTU1BTkRfQkFTRSAr IERSTV9BTURHUFVfRlJFRVNZTkMsIHN0cnVjdCBkcm1fYW1kZ3B1X2ZyZWVzeW5jKQo+ICsjZGVm aW5lIERSTV9JT0NUTF9BTURHUFVfR0VNX0ZJTkRfQk8JRFJNX0lPV1IoRFJNX0NPTU1BTkRfQkFT RSArIERSTV9BTURHUFVfR0VNX0ZJTkRfQk8sIHVuaW9uIGRybV9hbWRncHVfZ2VtX2ZpbmRfYm8p Cj4gICAKPiAgIC8qKgo+ICAgICogRE9DOiBtZW1vcnkgZG9tYWlucwo+IEBAIC0zOTIsNiArMzk0 LDI1IEBAIHN0cnVjdCBkcm1fYW1kZ3B1X2dlbV93YWl0X2lkbGVfb3V0IHsKPiAgIAlzdHJ1Y3Qg ZHJtX2FtZGdwdV9nZW1fd2FpdF9pZGxlX291dCBvdXQ7Cj4gICB9Owo+ICAgCj4gK3N0cnVjdCBk cm1fYW1kZ3B1X2dlbV9maW5kX2JvX2luIHsKPiArCS8qIENQVSBhZGRyZXNzICovCj4gKwlfX3U2 NAkJCWFkZHI7Cj4gKwkvKiBtZW1vcnkgc2l6ZSBmcm9tIENQVSBhZGRyZXNzICovCj4gKwlfX3U2 NAkJCXNpemU7Cj4gK307Cj4gKwo+ICtzdHJ1Y3QgZHJtX2FtZGdwdV9nZW1fZmluZF9ib19vdXQg ewo+ICsJLyogb2Zmc2V0IGluIGJvICovCj4gKwlfX3U2NAkJCW9mZnNldDsKPiArCS8qIHJlc3Vs dGluZyBHRU0gaGFuZGxlICovCj4gKwlfX3UzMgkJCWhhbmRsZTsKPiArfTsKPiArCj4gK3VuaW9u IGRybV9hbWRncHVfZ2VtX2ZpbmRfYm8gewo+ICsJc3RydWN0IGRybV9hbWRncHVfZ2VtX2ZpbmRf Ym9faW4gaW47Cj4gKwlzdHJ1Y3QgZHJtX2FtZGdwdV9nZW1fZmluZF9ib19vdXQgb3V0Owo+ICt9 Owo+ICsKPiAgIHN0cnVjdCBkcm1fYW1kZ3B1X3dhaXRfY3NfaW4gewo+ICAgCS8qIENvbW1hbmQg c3VibWlzc2lvbiBoYW5kbGUKPiAgICAgICAgICAgICogaGFuZGxlIGVxdWFscyAwIG1lYW5zIG5v bmUgdG8gd2FpdCBmb3IKCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fCmFtZC1nZnggbWFpbGluZyBsaXN0CmFtZC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3Jn Cmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vYW1kLWdmeAo=