From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?Q?Christian_K=c3=b6nig?= Subject: Re: [PATCH v5] drm/amdgpu: Transfer fences to dmabuf importer Date: Tue, 7 Aug 2018 20:53:24 +0200 Message-ID: <73e34a5d-e3d1-222f-85e2-7426d726fbe5@gmail.com> References: <20180807104500.31264-1-chris@chris-wilson.co.uk> <20180807183207.12584-1-chris@chris-wilson.co.uk> Reply-To: christian.koenig@amd.com Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20180807183207.12584-1-chris@chris-wilson.co.uk> Content-Language: en-US List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Chris Wilson , intel-gfx@lists.freedesktop.org, amd-gfx@lists.freedesktop.org Cc: Alex Deucher , =?UTF-8?Q?Christian_K=c3=b6nig?= , dri-devel@lists.freedesktop.org List-Id: amd-gfx.lists.freedesktop.org QW0gMDcuMDguMjAxOCB1bSAyMDozMiBzY2hyaWViIENocmlzIFdpbHNvbjoKPiBhbWRncHUgb25s eSB1c2VzIHNoYXJlZC1mZW5jZXMgaW50ZXJuYWxseSwgYnV0IGRtYWJ1ZiBpbXBvcnRlcnMgcmVs eSBvbgo+IGltcGxpY2l0IHdyaXRlIGhhemFyZCB0cmFja2luZyB2aWEgdGhlIHJlc2VydmF0aW9u X29iamVjdC5mZW5jZV9leGNsLgo+IEZvciBleGFtcGxlLCB0aGUgaW1wb3J0ZXIgdXNlIHRoZSB3 cml0ZSBoYXphcmQgZm9yIHRpbWluZyBhIHBhZ2UgZmxpcCB0bwo+IG9ubHkgb2NjdXIgYWZ0ZXIg dGhlIGV4cG9ydGVyIGhhcyBmaW5pc2hlZCBmbHVzaGluZyBpdHMgd3JpdGUgaW50byB0aGUKPiBz dXJmYWNlLiBBcyBzdWNoLCBvbiBleHBvcnRpbmcgYSBkbWFidWYsIHdlIG11c3QgZWl0aGVyIGZs dXNoIGFsbAo+IG91dHN0YW5kaW5nIGZlbmNlcyAoZm9yIHdlIGRvIG5vdCBrbm93IHdoaWNoIGFy ZSB3cml0ZXMgYW5kIHNob3VsZCBoYXZlCj4gYmVlbiBleGNsdXNpdmUpIG9yIGFsdGVybmF0aXZl bHkgY3JlYXRlIGEgbmV3IGV4Y2x1c2l2ZSBmZW5jZSB0aGF0IGlzCj4gdGhlIGNvbXBvc2l0ZSBv ZiBhbGwgdGhlIGV4aXN0aW5nIHNoYXJlZCBmZW5jZXMsIGFuZCBzbyB3aWxsIG9ubHkgYmUKPiBz aWduYWxlZCB3aGVuIGFsbCBlYXJsaWVyIGZlbmNlcyBhcmUgc2lnbmFsZWQgKGVuc3VyaW5nIHRo YXQgd2UgY2FuIG5vdAo+IGJlIHNpZ25hbGVkIGJlZm9yZSB0aGUgY29tcGxldGlvbiBvZiBhbnkg ZWFybGllciB3cml0ZSkuCj4KPiB2MjogcmVzZXJ2YXRpb25fb2JqZWN0IGlzIGFscmVhZHkgbG9j a2VkIGJ5IGFtZGdwdV9ib19yZXNlcnZlKCkKPiB2MzogUmVwbGFjZSBsb29waW5nIHdpdGggZ2V0 X2ZlbmNlc19yY3UgYW5kIHNwZWNpYWwgY2FzZSB0aGUgcHJvbW90aW9uCj4gb2YgYSBzaW5nbGUg c2hhcmVkIGZlbmNlIGRpcmVjdGx5IHRvIGFuIGV4Y2x1c2l2ZSBmZW5jZSwgYnlwYXNzaW5nIHRo ZQo+IGZlbmNlIGFycmF5Lgo+Cj4gQnVnemlsbGE6IGh0dHBzOi8vYnVncy5mcmVlZGVza3RvcC5v cmcvc2hvd19idWcuY2dpP2lkPTEwNzM0MQo+IFRlc3RjYXNlOiBpZ3QvYW1kX3ByaW1lL2FtZC10 by1pOTE1Cj4gUmVmZXJlbmNlczogOGU5NGE0NmMxNzcwICgiZHJtL2FtZGdwdTogQXR0YWNoIGV4 Y2x1c2l2ZSBmZW5jZSB0byBwcmltZSBleHBvcnRlZCBibydzLiAodjUpIikKPiBTaWduZWQtb2Zm LWJ5OiBDaHJpcyBXaWxzb24gPGNocmlzQGNocmlzLXdpbHNvbi5jby51az4KPiBDYzogQWxleCBE ZXVjaGVyIDxhbGV4YW5kZXIuZGV1Y2hlckBhbWQuY29tPgo+IENjOiAiQ2hyaXN0aWFuIEvDtm5p ZyIgPGNocmlzdGlhbi5rb2VuaWdAYW1kLmNvbT4KClJldmlld2VkLWJ5OiBDaHJpc3RpYW4gS8O2 bmlnIDxjaHJpc3RpYW4ua29lbmlnQGFtZC5jb20+Cgo+IC0tLQo+ICAgZHJpdmVycy9ncHUvZHJt L2FtZC9hbWRncHUvYW1kZ3B1X3ByaW1lLmMgfCA1OCArKysrKysrKysrKysrKysrKysrLS0tLQo+ ICAgMSBmaWxlIGNoYW5nZWQsIDUwIGluc2VydGlvbnMoKyksIDggZGVsZXRpb25zKC0pCj4KPiBk aWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1kZ3B1X3ByaW1lLmMgYi9k cml2ZXJzL2dwdS9kcm0vYW1kL2FtZGdwdS9hbWRncHVfcHJpbWUuYwo+IGluZGV4IDNmZGQ1Njg4 ZGEwYi4uMDZhMzEwYjViMDdiIDEwMDY0NAo+IC0tLSBhL2RyaXZlcnMvZ3B1L2RybS9hbWQvYW1k Z3B1L2FtZGdwdV9wcmltZS5jCj4gKysrIGIvZHJpdmVycy9ncHUvZHJtL2FtZC9hbWRncHUvYW1k Z3B1X3ByaW1lLmMKPiBAQCAtMzcsNiArMzcsNyBAQAo+ICAgI2luY2x1ZGUgImFtZGdwdV9kaXNw bGF5LmgiCj4gICAjaW5jbHVkZSA8ZHJtL2FtZGdwdV9kcm0uaD4KPiAgICNpbmNsdWRlIDxsaW51 eC9kbWEtYnVmLmg+Cj4gKyNpbmNsdWRlIDxsaW51eC9kbWEtZmVuY2UtYXJyYXkuaD4KPiAgIAo+ ICAgc3RhdGljIGNvbnN0IHN0cnVjdCBkbWFfYnVmX29wcyBhbWRncHVfZG1hYnVmX29wczsKPiAg IAo+IEBAIC0xODgsNiArMTg5LDQ3IEBAIGFtZGdwdV9nZW1fcHJpbWVfaW1wb3J0X3NnX3RhYmxl KHN0cnVjdCBkcm1fZGV2aWNlICpkZXYsCj4gICAJcmV0dXJuIEVSUl9QVFIocmV0KTsKPiAgIH0K PiAgIAo+ICtzdGF0aWMgaW50Cj4gK19fcmVzZXJ2YXRpb25fb2JqZWN0X21ha2VfZXhjbHVzaXZl KHN0cnVjdCByZXNlcnZhdGlvbl9vYmplY3QgKm9iaikKPiArewo+ICsJc3RydWN0IGRtYV9mZW5j ZSAqKmZlbmNlczsKPiArCXVuc2lnbmVkIGludCBjb3VudDsKPiArCWludCByOwo+ICsKPiArCWlm ICghcmVzZXJ2YXRpb25fb2JqZWN0X2dldF9saXN0KG9iaikpIC8qIG5vIHNoYXJlZCBmZW5jZXMg dG8gY29udmVydCAqLwo+ICsJCXJldHVybiAwOwo+ICsKPiArCXIgPSByZXNlcnZhdGlvbl9vYmpl Y3RfZ2V0X2ZlbmNlc19yY3Uob2JqLCBOVUxMLCAmY291bnQsICZmZW5jZXMpOwo+ICsJaWYgKHIp Cj4gKwkJcmV0dXJuIHI7Cj4gKwo+ICsJaWYgKGNvdW50ID09IDApIHsKPiArCQkvKiBOb3cgdGhh dCB3YXMgdW5leHBlY3RlZC4gKi8KPiArCX0gZWxzZSBpZiAoY291bnQgPT0gMSkgewo+ICsJCXJl c2VydmF0aW9uX29iamVjdF9hZGRfZXhjbF9mZW5jZShvYmosIGZlbmNlc1swXSk7Cj4gKwkJZG1h X2ZlbmNlX3B1dChmZW5jZXNbMF0pOwo+ICsJCWtmcmVlKGZlbmNlcyk7Cj4gKwl9IGVsc2Ugewo+ ICsJCXN0cnVjdCBkbWFfZmVuY2VfYXJyYXkgKmFycmF5Owo+ICsKPiArCQlhcnJheSA9IGRtYV9m ZW5jZV9hcnJheV9jcmVhdGUoY291bnQsIGZlbmNlcywKPiArCQkJCQkgICAgICAgZG1hX2ZlbmNl X2NvbnRleHRfYWxsb2MoMSksIDAsCj4gKwkJCQkJICAgICAgIGZhbHNlKTsKPiArCQlpZiAoIWFy cmF5KQo+ICsJCQlnb3RvIGVycl9mZW5jZXNfcHV0Owo+ICsKPiArCQlyZXNlcnZhdGlvbl9vYmpl Y3RfYWRkX2V4Y2xfZmVuY2Uob2JqLCAmYXJyYXktPmJhc2UpOwo+ICsJfQo+ICsKPiArCXJldHVy biAwOwo+ICsKPiArZXJyX2ZlbmNlc19wdXQ6Cj4gKwl3aGlsZSAoY291bnQtLSkKPiArCQlkbWFf ZmVuY2VfcHV0KGZlbmNlc1tjb3VudF0pOwo+ICsJa2ZyZWUoZmVuY2VzKTsKPiArCXJldHVybiAt RU5PTUVNOwo+ICt9Cj4gKwo+ICAgLyoqCj4gICAgKiBhbWRncHVfZ2VtX21hcF9hdHRhY2ggLSAm ZG1hX2J1Zl9vcHMuYXR0YWNoIGltcGxlbWVudGF0aW9uCj4gICAgKiBAZG1hX2J1Zjogc2hhcmVk IERNQSBidWZmZXIKPiBAQCAtMjE5LDE2ICsyNjEsMTYgQEAgc3RhdGljIGludCBhbWRncHVfZ2Vt X21hcF9hdHRhY2goc3RydWN0IGRtYV9idWYgKmRtYV9idWYsCj4gICAKPiAgIAlpZiAoYXR0YWNo LT5kZXYtPmRyaXZlciAhPSBhZGV2LT5kZXYtPmRyaXZlcikgewo+ICAgCQkvKgo+IC0JCSAqIFdh aXQgZm9yIGFsbCBzaGFyZWQgZmVuY2VzIHRvIGNvbXBsZXRlIGJlZm9yZSB3ZSBzd2l0Y2ggdG8g ZnV0dXJlCj4gLQkJICogdXNlIG9mIGV4Y2x1c2l2ZSBmZW5jZSBvbiB0aGlzIHByaW1lIHNoYXJl ZCBiby4KPiArCQkgKiBXZSBvbmx5IGNyZWF0ZSBzaGFyZWQgZmVuY2VzIGZvciBpbnRlcm5hbCB1 c2UsIGJ1dCBpbXBvcnRlcnMKPiArCQkgKiBvZiB0aGUgZG1hYnVmIHJlbHkgb24gZXhjbHVzaXZl IGZlbmNlcyBmb3IgaW1wbGljaXRseQo+ICsJCSAqIHRyYWNraW5nIHdyaXRlIGhhemFyZHMuIEFz IGFueSBvZiB0aGUgY3VycmVudCBmZW5jZXMgbWF5Cj4gKwkJICogY29ycmVzcG9uZCB0byBhIHdy aXRlLCB3ZSBuZWVkIHRvIGNvbnZlcnQgYWxsIGV4aXN0aW5nCj4gKwkJICogZmVuY2VzIG9uIHRo ZSByZXNlcnZhdGlvbiBvYmplY3QgaW50byBhIHNpbmdsZSBleGNsdXNpdmUKPiArCQkgKiBmZW5j ZS4KPiAgIAkJICovCj4gLQkJciA9IHJlc2VydmF0aW9uX29iamVjdF93YWl0X3RpbWVvdXRfcmN1 KGJvLT50Ym8ucmVzdiwKPiAtCQkJCQkJCXRydWUsIGZhbHNlLAo+IC0JCQkJCQkJTUFYX1NDSEVE VUxFX1RJTUVPVVQpOwo+IC0JCWlmICh1bmxpa2VseShyIDwgMCkpIHsKPiAtCQkJRFJNX0RFQlVH X1BSSU1FKCJGZW5jZSB3YWl0IGZhaWxlZDogJWxpXG4iLCByKTsKPiArCQlyID0gX19yZXNlcnZh dGlvbl9vYmplY3RfbWFrZV9leGNsdXNpdmUoYm8tPnRiby5yZXN2KTsKPiArCQlpZiAocikKPiAg IAkJCWdvdG8gZXJyb3JfdW5yZXNlcnZlOwo+IC0JCX0KPiAgIAl9Cj4gICAKPiAgIAkvKiBwaW4g YnVmZmVyIGludG8gR1RUICovCgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fXwpJbnRlbC1nZnggbWFpbGluZyBsaXN0CkludGVsLWdmeEBsaXN0cy5mcmVlZGVz a3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9p bnRlbC1nZngK