From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [Intel-gfx] [PATCH 06/10] drm/syncobj: use the timeline point in drm_syncobj_find_fence v3 Date: Thu, 13 Dec 2018 17:01:49 +0100 Message-ID: <20181213160148.GG21184@phenom.ffwll.local> References: <20181211103449.25899-1-david1.zhou@amd.com> <20181211103449.25899-6-david1.zhou@amd.com> <154470106444.11001.18115532802703573952@skylake-alporthouse-com> <36d34a20-2562-4265-9abc-4d3bd6d358ef@amd.com> <154470371691.11001.12393270310703413444@skylake-alporthouse-com> <6e30211b-8257-61dd-0d56-ffa8add02601@amd.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: <6e30211b-8257-61dd-0d56-ffa8add02601@amd.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: "Koenig, Christian" Cc: Christian =?iso-8859-1?Q?K=F6nig?= , "intel-gfx@lists.freedesktop.org" , "dri-devel@lists.freedesktop.org" , "amd-gfx@lists.freedesktop.org" List-Id: amd-gfx.lists.freedesktop.org T24gVGh1LCBEZWMgMTMsIDIwMTggYXQgMTI6MjQ6NTdQTSArMDAwMCwgS29lbmlnLCBDaHJpc3Rp YW4gd3JvdGU6Cj4gQW0gMTMuMTIuMTggdW0gMTM6MjEgc2NocmllYiBDaHJpcyBXaWxzb246Cj4g PiBRdW90aW5nIEtvZW5pZywgQ2hyaXN0aWFuICgyMDE4LTEyLTEzIDEyOjExOjEwKQo+ID4+IEFt IDEzLjEyLjE4IHVtIDEyOjM3IHNjaHJpZWIgQ2hyaXMgV2lsc29uOgo+ID4+PiBRdW90aW5nIENo dW5taW5nIFpob3UgKDIwMTgtMTItMTEgMTA6MzQ6NDUpCj4gPj4+PiBGcm9tOiBDaHJpc3RpYW4g S8O2bmlnIDxja29lbmlnLmxlaWNodHp1bWVya2VuQGdtYWlsLmNvbT4KPiA+Pj4+Cj4gPj4+PiBJ bXBsZW1lbnQgZmluZGluZyB0aGUgcmlnaHQgdGltZWxpbmUgcG9pbnQgaW4gZHJtX3N5bmNvYmpf ZmluZF9mZW5jZS4KPiA+Pj4+Cj4gPj4+PiB2MjogcmV0dXJuIC1FSU5WQUwgd2hlbiB0aGUgcG9p bnQgaXMgbm90IHN1Ym1pdHRlZCB5ZXQuCj4gPj4+PiB2MzogZml4IHJlZmVyZW5jZSBjb3VudGlu ZyBidWcsIGFkZCBmbGFncyBoYW5kbGluZyBhcyB3ZWxsCj4gPj4+Pgo+ID4+Pj4gU2lnbmVkLW9m Zi1ieTogQ2hyaXN0aWFuIEvDtm5pZyA8Y2hyaXN0aWFuLmtvZW5pZ0BhbWQuY29tPgo+ID4+Pj4g LS0tCj4gPj4+PiAgICBkcml2ZXJzL2dwdS9kcm0vZHJtX3N5bmNvYmouYyB8IDQzICsrKysrKysr KysrKysrKysrKysrKysrKysrKysrKysrLS0tCj4gPj4+PiAgICAxIGZpbGUgY2hhbmdlZCwgNDAg aW5zZXJ0aW9ucygrKSwgMyBkZWxldGlvbnMoLSkKPiA+Pj4+Cj4gPj4+PiBkaWZmIC0tZ2l0IGEv ZHJpdmVycy9ncHUvZHJtL2RybV9zeW5jb2JqLmMgYi9kcml2ZXJzL2dwdS9kcm0vZHJtX3N5bmNv YmouYwo+ID4+Pj4gaW5kZXggNzZjZTEzZGFmYzRkLi5kOTY0YjM0OGVjYmEgMTAwNjQ0Cj4gPj4+ PiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vZHJtX3N5bmNvYmouYwo+ID4+Pj4gKysrIGIvZHJpdmVy cy9ncHUvZHJtL2RybV9zeW5jb2JqLmMKPiA+Pj4+IEBAIC0yMzEsMTYgKzIzMSw1MyBAQCBpbnQg ZHJtX3N5bmNvYmpfZmluZF9mZW5jZShzdHJ1Y3QgZHJtX2ZpbGUgKmZpbGVfcHJpdmF0ZSwKPiA+ Pj4+ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgc3RydWN0IGRtYV9mZW5jZSAqKmZlbmNl KQo+ID4+Pj4gICAgewo+ID4+Pj4gICAgICAgICAgIHN0cnVjdCBkcm1fc3luY29iaiAqc3luY29i aiA9IGRybV9zeW5jb2JqX2ZpbmQoZmlsZV9wcml2YXRlLCBoYW5kbGUpOwo+ID4+Pj4gLSAgICAg ICBpbnQgcmV0ID0gMDsKPiA+Pj4+ICsgICAgICAgc3RydWN0IHN5bmNvYmpfd2FpdF9lbnRyeSB3 YWl0Owo+ID4+Pj4gKyAgICAgICBpbnQgcmV0Owo+ID4+Pj4gICAgCj4gPj4+PiAgICAgICAgICAg aWYgKCFzeW5jb2JqKQo+ID4+Pj4gICAgICAgICAgICAgICAgICAgcmV0dXJuIC1FTk9FTlQ7Cj4g Pj4+PiAgICAKPiA+Pj4+ICAgICAgICAgICAqZmVuY2UgPSBkcm1fc3luY29ial9mZW5jZV9nZXQo c3luY29iaik7Cj4gPj4+PiAtICAgICAgIGlmICghKmZlbmNlKSB7Cj4gPj4+PiArICAgICAgIGRy bV9zeW5jb2JqX3B1dChzeW5jb2JqKTsKPiA+Pj4+ICsKPiA+Pj4+ICsgICAgICAgaWYgKCpmZW5j ZSkgewo+ID4+Pj4gKyAgICAgICAgICAgICAgIHJldCA9IGRtYV9mZW5jZV9jaGFpbl9maW5kX3Nl cW5vKGZlbmNlLCBwb2ludCk7Cj4gPj4+PiArICAgICAgICAgICAgICAgaWYgKCFyZXQpCj4gPj4+ PiArICAgICAgICAgICAgICAgICAgICAgICByZXR1cm4gMDsKPiA+Pj4+ICsgICAgICAgICAgICAg ICBkbWFfZmVuY2VfcHV0KCpmZW5jZSk7Cj4gPj4+PiArICAgICAgIH0gZWxzZSB7Cj4gPj4+PiAg ICAgICAgICAgICAgICAgICByZXQgPSAtRUlOVkFMOwo+ID4+Pj4gICAgICAgICAgIH0KPiA+Pj4+ IC0gICAgICAgZHJtX3N5bmNvYmpfcHV0KHN5bmNvYmopOwo+ID4+Pj4gKwo+ID4+Pj4gKyAgICAg ICBpZiAoIShmbGFncyAmIERSTV9TWU5DT0JKX1dBSVRfRkxBR1NfV0FJVF9GT1JfU1VCTUlUKSkK PiA+Pj4+ICsgICAgICAgICAgICAgICByZXR1cm4gcmV0Owo+ID4+Pj4gKwo+ID4+Pj4gKyAgICAg ICBtZW1zZXQoJndhaXQsIDAsIHNpemVvZih3YWl0KSk7Cj4gPj4+PiArICAgICAgIHdhaXQudGFz ayA9IGN1cnJlbnQ7Cj4gPj4+PiArICAgICAgIHdhaXQucG9pbnQgPSBwb2ludDsKPiA+Pj4+ICsg ICAgICAgZHJtX3N5bmNvYmpfZmVuY2VfYWRkX3dhaXQoc3luY29iaiwgJndhaXQpOwo+ID4+Pj4g Kwo+ID4+Pj4gKyAgICAgICBkbyB7Cj4gPj4+PiArICAgICAgICAgICAgICAgc2V0X2N1cnJlbnRf c3RhdGUoVEFTS19JTlRFUlJVUFRJQkxFKTsKPiA+Pj4+ICsgICAgICAgICAgICAgICBpZiAod2Fp dC5mZW5jZSkgewo+ID4+Pj4gKyAgICAgICAgICAgICAgICAgICAgICAgcmV0ID0gMDsKPiA+Pj4+ ICsgICAgICAgICAgICAgICAgICAgICAgIGJyZWFrOwo+ID4+Pj4gKyAgICAgICAgICAgICAgIH0K PiA+Pj4+ICsKPiA+Pj4+ICsgICAgICAgICAgICAgICBpZiAoc2lnbmFsX3BlbmRpbmcoY3VycmVu dCkpIHsKPiA+Pj4+ICsgICAgICAgICAgICAgICAgICAgICAgIHJldCA9IC1FUkVTVEFSVFNZUzsK PiA+Pj4+ICsgICAgICAgICAgICAgICAgICAgICAgIGJyZWFrOwo+ID4+Pj4gKyAgICAgICAgICAg ICAgIH0KPiA+Pj4+ICsKPiA+Pj4+ICsgICAgICAgICAgICAgICBzY2hlZHVsZSgpOwo+ID4+Pj4g KyAgICAgICB9IHdoaWxlICgxKTsKPiA+Pj4gSSd2ZSBwcmV2aW91c2x5IHVzZWQgYSBkbWFfZmVu Y2VfcHJveHkgc28gdGhhdCB3ZSBjb3VsZCBkbyBub25ibG9ja2luZwo+ID4+PiB3YWl0cyBvbiBm dXR1cmUgc3VibWl0cy4gVGhhdCB3b3VsZCBiZSBwcmVmZXJyYWJsZSAoYSByZXF1aXJlbWVudCBm b3IKPiA+Pj4gb3VyIHN0dXBpZCBCS0wtZHJpdmVuIGNvZGUpLgo+ID4+IFRoYXQgaXMgZXhhY3Rs eSB3aGF0IEkgd291bGQgZGVmaW5pdGVseSBOQUsuCj4gPj4KPiA+PiBJIHdvdWxkIHJhdGhlciBz YXkgd2Ugc2hvdWxkIGNvbWUgdXAgd2l0aCBhIHdhaXRfbXVsdGlwbGVfZXZlbnRzKCkgbWFjcm8K PiA+PiBhbmQgY29tcGxldGVseSBudWtlIHRoZSBjdXN0b20gaW1wbGVtZW50YXRpb24gb2YgdGhp cyBpbjoKPiA+PiAxLiBkbWFfZmVuY2VfZGVmYXVsdF93YWl0IGFuZCBkbWFfZmVuY2Vfd2FpdF9h bnlfdGltZW91dAo+ID4+IDIuIHRoZSByYWRlb24gZmVuY2UgaW1wbGVtZW50YXRpb24KPiA+PiAz LiB0aGUgbm91dmVhdSBmZW5jZSBpbXBsZW1lbnRhdGlvbgo+ID4+IDQuIHRoZSBzeW5jb2JqIGNv ZGUKPiA+Pgo+ID4+IENhdXNlIGFsbCBvZiB0aGVtIGRvIGV4YWN0bHkgdGhlIHNhbWUuIFRoZSBk bWFfZmVuY2UgaW1wbGVtZW50YXRpb24KPiA+PiB1bmZvcnR1bmF0ZWx5IGNhbWUgdXAgd2l0aCBh IGN1c3RvbSBldmVudCBoYW5kbGluZyBtZWNoYW5pc20gaW5zdGVhZCBvZgo+ID4+IGV4dGVuZGlu ZyB0aGUgY29yZSBMaW51eCB3YWl0X2V2ZW50KCkgc3lzdGVtLgo+ID4gSSBkb24ndCB3YW50IGEg YmxvY2tpbmcgd2FpdCBhdCBhbGwuCj4gCj4gT2sgSSB3YXNuJ3QgY2xlYXIgZW5vdWdoIDopIFRo YXQgaXMgZXhhY3RseSB3aGF0IEkgd291bGQgTkFLIQo+IAo+IFRoZSB3YWl0IG11c3QgYmUgYmxv Y2tpbmcgb3Igb3RoZXJ3aXNlIHlvdSB3b3VsZCBhbGxvdyB3YWl0LWJlZm9yZS1zaWduYWwuCgpX ZWxsIHRoZSBjdXJyZW50IGltcGxlbWVudGF0aW9uIGlzIHB1bGxpbmcgYSByYXRoZXIgYmlnIHRy aWNrIG9uIHJlYWRlcnMKaW4gdGhpcyByZWdhcmQ6IEl0IGxvb2tzIGxpa2UgYSBkbWFfZmVuY2Us IGl0J3MgaW1wbGVtZW50ZWQgYXMgb25lIGV2ZW4sCmhlY2sgeW91IGV2ZW4gb3Blbi1jb2RlIGEg ZG1hX2ZlbmNlX3dhaXQgaGVyZS4KCkV4Y2VwdCB0aGUgc2VtYW50aWNzIGFyZSBjb21wbGV0ZWx5 IGRpZmZlcmVudC4KClNvIGFzaWRlIGZyb20gdGhlIGRpc2N1c3Npb24gd2hldGhlciB3ZSByZWFs bHkgd2FudCB0byBmdWxseSBjaGFpbiB0aGVtIEkKdGhpbmsgaXQganVzdCBkb2Vzbid0IG1ha2Ug c2Vuc2UgdG8gaW1wbGVtZW50IHRoZSAid2FpdCBmb3IgZmVuY2Ugc3VibWl0IgphcyBhIGRtYV9m ZW5jZSB3YWl0LiBBbmQgSSdkIG91dHJpZ2h0IHJlbW92ZSB0aGF0IHBhcnQgZnJvbSB0aGUgdWFw aSwgYW5kCmZvcmNlIHRoZSB3YWl0LiBUaGUgY3VycmVudCByYWR2L2FudiBwbGFucyBJIGRpc2N1 c3NlZCB3aXRoIEphc29uIHdhcyB0aGF0CndlJ2QgaGF2ZSBhIHNlcGFyYXRlIHN1Ym1pdCB0aHJl YWQsIGFuZCBoZW5jZSB1bmNvbmRpdGlvbmFsbHkgYmxvY2tpbmcKdW50aWwgdGhlIGZlbmNlIGhh cyBtYXRlcmlhbGl6ZWQgaXMgdGhlIHJpZ2h0IHRoaW5nIHRvIGRvLiBFdmVuIGFsbG93aW5nCnRo YXQgb3B0aW9uLCBlaXRoZXIgdGhyb3VnaCBhIGZsYWcsIG9yIG1ha2luZyB0aGVzZSB0aGluZ3Mg bG9vayBsaWtlCmRtYV9mZW5jZXMgKHRoZXkgYXJlIF9ub3RfKSBqdXN0IHRyaWNrcyBmb2xrcyBp bnRvIG1pc3VuZGVyc3RhbmRpbmcgd2hhdCdzCmdvaW5nIG9uLiBDb2RlIHNoYXJpbmcganVzdCBi ZWNhdXNlIHRoZSBjb2RlIGxvb2tzIHNpbWlsYXIgaXMgaW1vIGEgcmVhbGx5CmJhZCBpZGVhLCB3 aGVuIHRoZSBzZW1hbnRpY3MgYXJlIGVudGlyZWx5IGRpZmZlcmVudCAodGhhdCB3YXMgYWxzbyB0 aGUKcmVhc29uIGJlaGluZCBub3QgcmV1c2luZyBhbGwgdGhlIGNwdSBldmVudCBzdHVmZiBmb3Ig ZG1hX2ZlbmNlLCB0aGV5J3JlCm5vdCBub3JtYWwgY3B1IGV2ZW50cykuCi1EYW5pZWwKCj4gCj4g Q2hyaXN0aWFuLgo+IAo+ID4gLUNocmlzCj4gCj4gX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX18KPiBJbnRlbC1nZnggbWFpbGluZyBsaXN0Cj4gSW50ZWwtZ2Z4 QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwo+IGh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21h aWxtYW4vbGlzdGluZm8vaW50ZWwtZ2Z4CgotLSAKRGFuaWVsIFZldHRlcgpTb2Z0d2FyZSBFbmdp bmVlciwgSW50ZWwgQ29ycG9yYXRpb24KaHR0cDovL2Jsb2cuZmZ3bGwuY2gKX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVsIG1haWxpbmcgbGlz dApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0 b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg==