From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Zhang, Jerry (Junwei)" Subject: Re: [PATCH v2 1/2] drm/amdgpu: add optional fence out-parameter to amdgpu_vm_clear_freed Date: Fri, 24 Mar 2017 11:40:33 +0800 Message-ID: <58D49531.4080805@amd.com> References: <20170323192705.5474-1-nhaehnle@gmail.com> <58D484CB.1020306@amd.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <58D484CB.1020306-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: zhoucm1 , =?UTF-8?B?Tmljb2xhaSBIw6Robmxl?= , amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org Cc: =?UTF-8?B?Tmljb2xhaSBIw6Robmxl?= T24gMDMvMjQvMjAxNyAxMDozMCBBTSwgemhvdWNtMSB3cm90ZToKPgo+Cj4gT24gMjAxN+W5tDAz 5pyIMjTml6UgMDM6MjcsIE5pY29sYWkgSMOkaG5sZSB3cm90ZToKPj4gRnJvbTogTmljb2xhaSBI w6RobmxlIDxuaWNvbGFpLmhhZWhubGVAYW1kLmNvbT4KPj4KPj4gV2Ugd2lsbCBhZGQgdGhlIGZl bmNlIHRvIGZyZWVkIGJ1ZmZlciBvYmplY3RzIGluIGEgbGF0ZXIgY29tbWl0LCB0byBlbnN1cmUK Pj4gdGhhdCB0aGUgdW5kZXJseWluZyBtZW1vcnkgY2FuIG9ubHkgYmUgcmUtdXNlZCBhZnRlciBh bGwgcmVmZXJlbmNlcyBpbgo+PiBwYWdlIHRhYmxlcyBoYXZlIGJlZW4gY2xlYXJlZC4KPj4KPj4g U2lnbmVkLW9mZi1ieTogTmljb2xhaSBIw6RobmxlIDxuaWNvbGFpLmhhZWhubGVAYW1kLmNvbT4K PiBSZXZpZXdlZC1ieTogQ2h1bm1pbmcgWmhvdSA8ZGF2aWQxLnpob3VAYW1kLmNvbT4KUmV2aWV3 ZWQtYnk6IEp1bndlaSBaaGFuZyA8SmVycnkuWmhhbmdAYW1kLmNvbT4KCj4KPj4gLS0tCj4+ICAg ZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1kZ3B1X2NzLmMgIHwgIDIgKy0KPj4gICBkcml2 ZXJzL2dwdS9kcm0vYW1kL2FtZGdwdS9hbWRncHVfZ2VtLmMgfCAgMiArLQo+PiAgIGRyaXZlcnMv Z3B1L2RybS9hbWQvYW1kZ3B1L2FtZGdwdV92bS5jICB8IDIxICsrKysrKysrKysrKysrKy0tLS0t LQo+PiAgIGRyaXZlcnMvZ3B1L2RybS9hbWQvYW1kZ3B1L2FtZGdwdV92bS5oICB8ICAzICsrLQo+ PiAgIDQgZmlsZXMgY2hhbmdlZCwgMTkgaW5zZXJ0aW9ucygrKSwgOSBkZWxldGlvbnMoLSkKPj4K Pj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9hbWQvYW1kZ3B1L2FtZGdwdV9jcy5jCj4+ IGIvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1kZ3B1X2NzLmMKPj4gaW5kZXggNTVkNTUz YS4uODVlNjA3MCAxMDA2NDQKPj4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1k Z3B1X2NzLmMKPj4gKysrIGIvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1kZ3B1X2NzLmMK Pj4gQEAgLTc3OCwyMSArNzc4LDIxIEBAIHN0YXRpYyBpbnQgYW1kZ3B1X2JvX3ZtX3VwZGF0ZV9w dGUoc3RydWN0Cj4+IGFtZGdwdV9jc19wYXJzZXIgKnApCj4+ICAgICAgIGludCBpLCByOwo+PiAg ICAgICByID0gYW1kZ3B1X3ZtX3VwZGF0ZV9wYWdlX2RpcmVjdG9yeShhZGV2LCB2bSk7Cj4+ICAg ICAgIGlmIChyKQo+PiAgICAgICAgICAgcmV0dXJuIHI7Cj4+ICAgICAgIHIgPSBhbWRncHVfc3lu Y19mZW5jZShhZGV2LCAmcC0+am9iLT5zeW5jLCB2bS0+cGFnZV9kaXJlY3RvcnlfZmVuY2UpOwo+ PiAgICAgICBpZiAocikKPj4gICAgICAgICAgIHJldHVybiByOwo+PiAtICAgIHIgPSBhbWRncHVf dm1fY2xlYXJfZnJlZWQoYWRldiwgdm0pOwo+PiArICAgIHIgPSBhbWRncHVfdm1fY2xlYXJfZnJl ZWQoYWRldiwgdm0sIE5VTEwpOwo+PiAgICAgICBpZiAocikKPj4gICAgICAgICAgIHJldHVybiBy Owo+PiAgICAgICByID0gYW1kZ3B1X3ZtX2JvX3VwZGF0ZShhZGV2LCBmcHJpdi0+cHJ0X3ZhLCBm YWxzZSk7Cj4+ICAgICAgIGlmIChyKQo+PiAgICAgICAgICAgcmV0dXJuIHI7Cj4+ICAgICAgIHIg PSBhbWRncHVfc3luY19mZW5jZShhZGV2LCAmcC0+am9iLT5zeW5jLAo+PiAgICAgICAgICAgICAg ICAgICAgIGZwcml2LT5wcnRfdmEtPmxhc3RfcHRfdXBkYXRlKTsKPj4gICAgICAgaWYgKHIpCj4+ IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vYW1kL2FtZGdwdS9hbWRncHVfZ2VtLmMKPj4g Yi9kcml2ZXJzL2dwdS9kcm0vYW1kL2FtZGdwdS9hbWRncHVfZ2VtLmMKPj4gaW5kZXggYmU5ZmIy Yy4uNGE1M2M0MyAxMDA2NDQKPj4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1k Z3B1X2dlbS5jCj4+ICsrKyBiL2RyaXZlcnMvZ3B1L2RybS9hbWQvYW1kZ3B1L2FtZGdwdV9nZW0u Ywo+PiBAQCAtNTM1LDIxICs1MzUsMjEgQEAgc3RhdGljIHZvaWQgYW1kZ3B1X2dlbV92YV91cGRh dGVfdm0oc3RydWN0Cj4+IGFtZGdwdV9kZXZpY2UgKmFkZXYsCj4+ICAgICAgIHIgPSBhbWRncHVf dm1fdmFsaWRhdGVfcHRfYm9zKGFkZXYsIHZtLCBhbWRncHVfZ2VtX3ZhX2NoZWNrLAo+PiAgICAg ICAgICAgICAgICAgICAgICAgICBOVUxMKTsKPj4gICAgICAgaWYgKHIpCj4+ICAgICAgICAgICBn b3RvIGVycm9yOwo+PiAgICAgICByID0gYW1kZ3B1X3ZtX3VwZGF0ZV9wYWdlX2RpcmVjdG9yeShh ZGV2LCB2bSk7Cj4+ICAgICAgIGlmIChyKQo+PiAgICAgICAgICAgZ290byBlcnJvcjsKPj4gLSAg ICByID0gYW1kZ3B1X3ZtX2NsZWFyX2ZyZWVkKGFkZXYsIHZtKTsKPj4gKyAgICByID0gYW1kZ3B1 X3ZtX2NsZWFyX2ZyZWVkKGFkZXYsIHZtLCBOVUxMKTsKPj4gICAgICAgaWYgKHIpCj4+ICAgICAg ICAgICBnb3RvIGVycm9yOwo+PiAgICAgICBpZiAob3BlcmF0aW9uID09IEFNREdQVV9WQV9PUF9N QVAgfHwKPj4gICAgICAgICAgIG9wZXJhdGlvbiA9PSBBTURHUFVfVkFfT1BfUkVQTEFDRSkKPj4g ICAgICAgICAgIHIgPSBhbWRncHVfdm1fYm9fdXBkYXRlKGFkZXYsIGJvX3ZhLCBmYWxzZSk7Cj4+ ICAgZXJyb3I6Cj4+ICAgICAgIGlmIChyICYmIHIgIT0gLUVSRVNUQVJUU1lTKQo+PiAgICAgICAg ICAgRFJNX0VSUk9SKCJDb3VsZG4ndCB1cGRhdGUgQk9fVkEgKCVkKVxuIiwgcik7Cj4+IGRpZmYg LS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vYW1kL2FtZGdwdS9hbWRncHVfdm0uYwo+PiBiL2RyaXZl cnMvZ3B1L2RybS9hbWQvYW1kZ3B1L2FtZGdwdV92bS5jCj4+IGluZGV4IGRkN2RmNDUuLjJjOTVh NzUgMTAwNjQ0Cj4+IC0tLSBhL2RyaXZlcnMvZ3B1L2RybS9hbWQvYW1kZ3B1L2FtZGdwdV92bS5j Cj4+ICsrKyBiL2RyaXZlcnMvZ3B1L2RybS9hbWQvYW1kZ3B1L2FtZGdwdV92bS5jCj4+IEBAIC0x Mzk3LDQ4ICsxMzk3LDU3IEBAIHN0YXRpYyB2b2lkIGFtZGdwdV92bV9wcnRfZmluaShzdHJ1Y3Qg YW1kZ3B1X2RldmljZQo+PiAqYWRldiwgc3RydWN0IGFtZGdwdV92bSAqdm0pCj4+ICAgICAgIH0K Pj4gICAgICAga2ZyZWUoc2hhcmVkKTsKPj4gICB9Cj4+ICAgLyoqCj4+ICAgICogYW1kZ3B1X3Zt X2NsZWFyX2ZyZWVkIC0gY2xlYXIgZnJlZWQgQk9zIGluIHRoZSBQVAo+PiAgICAqCj4+ICAgICog QGFkZXY6IGFtZGdwdV9kZXZpY2UgcG9pbnRlcgo+PiAgICAqIEB2bTogcmVxdWVzdGVkIHZtCj4+ ICsgKiBAZmVuY2U6IG9wdGlvbmFsIHJlc3VsdGluZyBmZW5jZSAodW5jaGFuZ2VkIGlmIG5vIHdv cmsgbmVlZGVkIHRvIGJlIGRvbmUKPj4gKyAqIG9yIGlmIGFuIGVycm9yIG9jY3VycmVkKQo+PiAg ICAqCj4+ICAgICogTWFrZSBzdXJlIGFsbCBmcmVlZCBCT3MgYXJlIGNsZWFyZWQgaW4gdGhlIFBU Lgo+PiAgICAqIFJldHVybnMgMCBmb3Igc3VjY2Vzcy4KPj4gICAgKgo+PiAgICAqIFBUcyBoYXZl IHRvIGJlIHJlc2VydmVkIGFuZCBtdXRleCBtdXN0IGJlIGxvY2tlZCEKPj4gICAgKi8KPj4gICBp bnQgYW1kZ3B1X3ZtX2NsZWFyX2ZyZWVkKHN0cnVjdCBhbWRncHVfZGV2aWNlICphZGV2LAo+PiAt ICAgICAgICAgICAgICBzdHJ1Y3QgYW1kZ3B1X3ZtICp2bSkKPj4gKyAgICAgICAgICAgICAgc3Ry dWN0IGFtZGdwdV92bSAqdm0sCj4+ICsgICAgICAgICAgICAgIHN0cnVjdCBmZW5jZSAqKmZlbmNl KQo+PiAgIHsKPj4gICAgICAgc3RydWN0IGFtZGdwdV9ib192YV9tYXBwaW5nICptYXBwaW5nOwo+ PiAtICAgIHN0cnVjdCBmZW5jZSAqZmVuY2UgPSBOVUxMOwo+PiArICAgIHN0cnVjdCBmZW5jZSAq ZiA9IE5VTEw7Cj4+ICAgICAgIGludCByOwo+PiAgICAgICB3aGlsZSAoIWxpc3RfZW1wdHkoJnZt LT5mcmVlZCkpIHsKPj4gICAgICAgICAgIG1hcHBpbmcgPSBsaXN0X2ZpcnN0X2VudHJ5KCZ2bS0+ ZnJlZWQsCj4+ICAgICAgICAgICAgICAgc3RydWN0IGFtZGdwdV9ib192YV9tYXBwaW5nLCBsaXN0 KTsKPj4gICAgICAgICAgIGxpc3RfZGVsKCZtYXBwaW5nLT5saXN0KTsKPj4gICAgICAgICAgIHIg PSBhbWRncHVfdm1fYm9fc3BsaXRfbWFwcGluZyhhZGV2LCBOVUxMLCAwLCBOVUxMLCB2bSwgbWFw cGluZywKPj4gLSAgICAgICAgICAgICAgICAgICAgICAgICAgIDAsIDAsICZmZW5jZSk7Cj4+IC0g ICAgICAgIGFtZGdwdV92bV9mcmVlX21hcHBpbmcoYWRldiwgdm0sIG1hcHBpbmcsIGZlbmNlKTsK Pj4gKyAgICAgICAgICAgICAgICAgICAgICAgICAgIDAsIDAsICZmKTsKPj4gKyAgICAgICAgYW1k Z3B1X3ZtX2ZyZWVfbWFwcGluZyhhZGV2LCB2bSwgbWFwcGluZywgZik7Cj4+ICAgICAgICAgICBp ZiAocikgewo+PiAtICAgICAgICAgICAgZmVuY2VfcHV0KGZlbmNlKTsKPj4gKyAgICAgICAgICAg IGZlbmNlX3B1dChmKTsKPj4gICAgICAgICAgICAgICByZXR1cm4gcjsKPj4gICAgICAgICAgIH0K Pj4gKyAgICB9Cj4+ICsgICAgaWYgKGZlbmNlICYmIGYpIHsKPj4gKyAgICAgICAgZmVuY2VfcHV0 KCpmZW5jZSk7Cj4+ICsgICAgICAgICpmZW5jZSA9IGY7Cj4+ICsgICAgfSBlbHNlIHsKPj4gKyAg ICAgICAgZmVuY2VfcHV0KGYpOwo+PiAgICAgICB9Cj4+IC0gICAgZmVuY2VfcHV0KGZlbmNlKTsK Pj4gKwo+PiAgICAgICByZXR1cm4gMDsKPj4gICB9Cj4+ICAgLyoqCj4+ICAgICogYW1kZ3B1X3Zt X2NsZWFyX2ludmFsaWRzIC0gY2xlYXIgaW52YWxpZGF0ZWQgQk9zIGluIHRoZSBQVAo+PiAgICAq Cj4+ICAgICogQGFkZXY6IGFtZGdwdV9kZXZpY2UgcG9pbnRlcgo+PiAgICAqIEB2bTogcmVxdWVz dGVkIHZtCj4+ICAgICoKPj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9hbWQvYW1kZ3B1 L2FtZGdwdV92bS5oCj4+IGIvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1kZ3B1X3ZtLmgK Pj4gaW5kZXggZmYxMGZhNS4uOWQ1YTU3MiAxMDA2NDQKPj4gLS0tIGEvZHJpdmVycy9ncHUvZHJt L2FtZC9hbWRncHUvYW1kZ3B1X3ZtLmgKPj4gKysrIGIvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRn cHUvYW1kZ3B1X3ZtLmgKPj4gQEAgLTE4NywyMSArMTg3LDIyIEBAIGludCBhbWRncHVfdm1fYWxs b2NfcHRzKHN0cnVjdCBhbWRncHVfZGV2aWNlICphZGV2LAo+PiAgICAgICAgICAgICAgIHN0cnVj dCBhbWRncHVfdm0gKnZtLAo+PiAgICAgICAgICAgICAgIHVpbnQ2NF90IHNhZGRyLCB1aW50NjRf dCBzaXplKTsKPj4gICBpbnQgYW1kZ3B1X3ZtX2dyYWJfaWQoc3RydWN0IGFtZGdwdV92bSAqdm0s IHN0cnVjdCBhbWRncHVfcmluZyAqcmluZywKPj4gICAgICAgICAgICAgICAgIHN0cnVjdCBhbWRn cHVfc3luYyAqc3luYywgc3RydWN0IGZlbmNlICpmZW5jZSwKPj4gICAgICAgICAgICAgICAgIHN0 cnVjdCBhbWRncHVfam9iICpqb2IpOwo+PiAgIGludCBhbWRncHVfdm1fZmx1c2goc3RydWN0IGFt ZGdwdV9yaW5nICpyaW5nLCBzdHJ1Y3QgYW1kZ3B1X2pvYiAqam9iKTsKPj4gICB2b2lkIGFtZGdw dV92bV9yZXNldF9pZChzdHJ1Y3QgYW1kZ3B1X2RldmljZSAqYWRldiwgdW5zaWduZWQgdm1faWQp Owo+PiAgIGludCBhbWRncHVfdm1fdXBkYXRlX3BhZ2VfZGlyZWN0b3J5KHN0cnVjdCBhbWRncHVf ZGV2aWNlICphZGV2LAo+PiAgICAgICAgICAgICAgICAgICAgICAgc3RydWN0IGFtZGdwdV92bSAq dm0pOwo+PiAgIGludCBhbWRncHVfdm1fY2xlYXJfZnJlZWQoc3RydWN0IGFtZGdwdV9kZXZpY2Ug KmFkZXYsCj4+IC0gICAgICAgICAgICAgIHN0cnVjdCBhbWRncHVfdm0gKnZtKTsKPj4gKyAgICAg ICAgICAgICAgc3RydWN0IGFtZGdwdV92bSAqdm0sCj4+ICsgICAgICAgICAgICAgIHN0cnVjdCBm ZW5jZSAqKmZlbmNlKTsKPj4gICBpbnQgYW1kZ3B1X3ZtX2NsZWFyX2ludmFsaWRzKHN0cnVjdCBh bWRncHVfZGV2aWNlICphZGV2LCBzdHJ1Y3QgYW1kZ3B1X3ZtICp2bSwKPj4gICAgICAgICAgICAg ICAgICAgIHN0cnVjdCBhbWRncHVfc3luYyAqc3luYyk7Cj4+ICAgaW50IGFtZGdwdV92bV9ib191 cGRhdGUoc3RydWN0IGFtZGdwdV9kZXZpY2UgKmFkZXYsCj4+ICAgICAgICAgICAgICAgc3RydWN0 IGFtZGdwdV9ib192YSAqYm9fdmEsCj4+ICAgICAgICAgICAgICAgYm9vbCBjbGVhcik7Cj4+ICAg dm9pZCBhbWRncHVfdm1fYm9faW52YWxpZGF0ZShzdHJ1Y3QgYW1kZ3B1X2RldmljZSAqYWRldiwK Pj4gICAgICAgICAgICAgICAgICAgIHN0cnVjdCBhbWRncHVfYm8gKmJvKTsKPj4gICBzdHJ1Y3Qg YW1kZ3B1X2JvX3ZhICphbWRncHVfdm1fYm9fZmluZChzdHJ1Y3QgYW1kZ3B1X3ZtICp2bSwKPj4g ICAgICAgICAgICAgICAgICAgICAgICAgIHN0cnVjdCBhbWRncHVfYm8gKmJvKTsKPj4gICBzdHJ1 Y3QgYW1kZ3B1X2JvX3ZhICphbWRncHVfdm1fYm9fYWRkKHN0cnVjdCBhbWRncHVfZGV2aWNlICph ZGV2LAo+Cj4gX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18K PiBhbWQtZ2Z4IG1haWxpbmcgbGlzdAo+IGFtZC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCj4g aHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9hbWQtZ2Z4Cl9f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmFtZC1nZnggbWFp bGluZyBsaXN0CmFtZC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJl ZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vYW1kLWdmeAo=