From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Koenig, Christian" Subject: Re: [Intel-gfx] [PATCH 06/10] drm/syncobj: use the timeline point in drm_syncobj_find_fence v3 Date: Thu, 13 Dec 2018 16:47:35 +0000 Message-ID: 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> <20181213160148.GG21184@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20181213160148.GG21184-dv86pmgwkMBes7Z6vYuT8azUEOm+Xw19@public.gmane.org> Content-Language: en-US Content-ID: <9103B33CF24F3E4F95BF956E322ED0BD-asWib9pRmPqcE4WynfumptQqCkab/8FMAL8bYrjMMd8@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: amd-gfx-bounces-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org Sender: "amd-gfx" To: Daniel Vetter Cc: "Zhou, David(ChunMing)" , =?utf-8?B?Q2hyaXN0aWFuIEvDtm5pZw==?= , "intel-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org" , "dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org" , Chris Wilson , "amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org" List-Id: intel-gfx@lists.freedesktop.org QW0gMTMuMTIuMTggdW0gMTc6MDEgc2NocmllYiBEYW5pZWwgVmV0dGVyOg0KPiBPbiBUaHUsIERl YyAxMywgMjAxOCBhdCAxMjoyNDo1N1BNICswMDAwLCBLb2VuaWcsIENocmlzdGlhbiB3cm90ZToN Cj4+IEFtIDEzLjEyLjE4IHVtIDEzOjIxIHNjaHJpZWIgQ2hyaXMgV2lsc29uOg0KPj4+IFF1b3Rp bmcgS29lbmlnLCBDaHJpc3RpYW4gKDIwMTgtMTItMTMgMTI6MTE6MTApDQo+Pj4+IEFtIDEzLjEy LjE4IHVtIDEyOjM3IHNjaHJpZWIgQ2hyaXMgV2lsc29uOg0KPj4+Pj4gUXVvdGluZyBDaHVubWlu ZyBaaG91ICgyMDE4LTEyLTExIDEwOjM0OjQ1KQ0KPj4+Pj4+IEZyb206IENocmlzdGlhbiBLw7Zu aWcgPGNrb2VuaWcubGVpY2h0enVtZXJrZW5AZ21haWwuY29tPg0KPj4+Pj4+DQo+Pj4+Pj4gSW1w bGVtZW50IGZpbmRpbmcgdGhlIHJpZ2h0IHRpbWVsaW5lIHBvaW50IGluIGRybV9zeW5jb2JqX2Zp bmRfZmVuY2UuDQo+Pj4+Pj4NCj4+Pj4+PiB2MjogcmV0dXJuIC1FSU5WQUwgd2hlbiB0aGUgcG9p bnQgaXMgbm90IHN1Ym1pdHRlZCB5ZXQuDQo+Pj4+Pj4gdjM6IGZpeCByZWZlcmVuY2UgY291bnRp bmcgYnVnLCBhZGQgZmxhZ3MgaGFuZGxpbmcgYXMgd2VsbA0KPj4+Pj4+DQo+Pj4+Pj4gU2lnbmVk LW9mZi1ieTogQ2hyaXN0aWFuIEvDtm5pZyA8Y2hyaXN0aWFuLmtvZW5pZ0BhbWQuY29tPg0KPj4+ Pj4+IC0tLQ0KPj4+Pj4+ICAgICBkcml2ZXJzL2dwdS9kcm0vZHJtX3N5bmNvYmouYyB8IDQzICsr KysrKysrKysrKysrKysrKysrKysrKysrKysrKysrLS0tDQo+Pj4+Pj4gICAgIDEgZmlsZSBjaGFu Z2VkLCA0MCBpbnNlcnRpb25zKCspLCAzIGRlbGV0aW9ucygtKQ0KPj4+Pj4+DQo+Pj4+Pj4gZGlm ZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9kcm1fc3luY29iai5jIGIvZHJpdmVycy9ncHUvZHJt L2RybV9zeW5jb2JqLmMNCj4+Pj4+PiBpbmRleCA3NmNlMTNkYWZjNGQuLmQ5NjRiMzQ4ZWNiYSAx MDA2NDQNCj4+Pj4+PiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vZHJtX3N5bmNvYmouYw0KPj4+Pj4+ ICsrKyBiL2RyaXZlcnMvZ3B1L2RybS9kcm1fc3luY29iai5jDQo+Pj4+Pj4gQEAgLTIzMSwxNiAr MjMxLDUzIEBAIGludCBkcm1fc3luY29ial9maW5kX2ZlbmNlKHN0cnVjdCBkcm1fZmlsZSAqZmls ZV9wcml2YXRlLA0KPj4+Pj4+ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIHN0cnVjdCBk bWFfZmVuY2UgKipmZW5jZSkNCj4+Pj4+PiAgICAgew0KPj4+Pj4+ICAgICAgICAgICAgc3RydWN0 IGRybV9zeW5jb2JqICpzeW5jb2JqID0gZHJtX3N5bmNvYmpfZmluZChmaWxlX3ByaXZhdGUsIGhh bmRsZSk7DQo+Pj4+Pj4gLSAgICAgICBpbnQgcmV0ID0gMDsNCj4+Pj4+PiArICAgICAgIHN0cnVj dCBzeW5jb2JqX3dhaXRfZW50cnkgd2FpdDsNCj4+Pj4+PiArICAgICAgIGludCByZXQ7DQo+Pj4+ Pj4gICAgIA0KPj4+Pj4+ICAgICAgICAgICAgaWYgKCFzeW5jb2JqKQ0KPj4+Pj4+ICAgICAgICAg ICAgICAgICAgICByZXR1cm4gLUVOT0VOVDsNCj4+Pj4+PiAgICAgDQo+Pj4+Pj4gICAgICAgICAg ICAqZmVuY2UgPSBkcm1fc3luY29ial9mZW5jZV9nZXQoc3luY29iaik7DQo+Pj4+Pj4gLSAgICAg ICBpZiAoISpmZW5jZSkgew0KPj4+Pj4+ICsgICAgICAgZHJtX3N5bmNvYmpfcHV0KHN5bmNvYmop Ow0KPj4+Pj4+ICsNCj4+Pj4+PiArICAgICAgIGlmICgqZmVuY2UpIHsNCj4+Pj4+PiArICAgICAg ICAgICAgICAgcmV0ID0gZG1hX2ZlbmNlX2NoYWluX2ZpbmRfc2Vxbm8oZmVuY2UsIHBvaW50KTsN Cj4+Pj4+PiArICAgICAgICAgICAgICAgaWYgKCFyZXQpDQo+Pj4+Pj4gKyAgICAgICAgICAgICAg ICAgICAgICAgcmV0dXJuIDA7DQo+Pj4+Pj4gKyAgICAgICAgICAgICAgIGRtYV9mZW5jZV9wdXQo KmZlbmNlKTsNCj4+Pj4+PiArICAgICAgIH0gZWxzZSB7DQo+Pj4+Pj4gICAgICAgICAgICAgICAg ICAgIHJldCA9IC1FSU5WQUw7DQo+Pj4+Pj4gICAgICAgICAgICB9DQo+Pj4+Pj4gLSAgICAgICBk cm1fc3luY29ial9wdXQoc3luY29iaik7DQo+Pj4+Pj4gKw0KPj4+Pj4+ICsgICAgICAgaWYgKCEo ZmxhZ3MgJiBEUk1fU1lOQ09CSl9XQUlUX0ZMQUdTX1dBSVRfRk9SX1NVQk1JVCkpDQo+Pj4+Pj4g KyAgICAgICAgICAgICAgIHJldHVybiByZXQ7DQo+Pj4+Pj4gKw0KPj4+Pj4+ICsgICAgICAgbWVt c2V0KCZ3YWl0LCAwLCBzaXplb2Yod2FpdCkpOw0KPj4+Pj4+ICsgICAgICAgd2FpdC50YXNrID0g Y3VycmVudDsNCj4+Pj4+PiArICAgICAgIHdhaXQucG9pbnQgPSBwb2ludDsNCj4+Pj4+PiArICAg ICAgIGRybV9zeW5jb2JqX2ZlbmNlX2FkZF93YWl0KHN5bmNvYmosICZ3YWl0KTsNCj4+Pj4+PiAr DQo+Pj4+Pj4gKyAgICAgICBkbyB7DQo+Pj4+Pj4gKyAgICAgICAgICAgICAgIHNldF9jdXJyZW50 X3N0YXRlKFRBU0tfSU5URVJSVVBUSUJMRSk7DQo+Pj4+Pj4gKyAgICAgICAgICAgICAgIGlmICh3 YWl0LmZlbmNlKSB7DQo+Pj4+Pj4gKyAgICAgICAgICAgICAgICAgICAgICAgcmV0ID0gMDsNCj4+ Pj4+PiArICAgICAgICAgICAgICAgICAgICAgICBicmVhazsNCj4+Pj4+PiArICAgICAgICAgICAg ICAgfQ0KPj4+Pj4+ICsNCj4+Pj4+PiArICAgICAgICAgICAgICAgaWYgKHNpZ25hbF9wZW5kaW5n KGN1cnJlbnQpKSB7DQo+Pj4+Pj4gKyAgICAgICAgICAgICAgICAgICAgICAgcmV0ID0gLUVSRVNU QVJUU1lTOw0KPj4+Pj4+ICsgICAgICAgICAgICAgICAgICAgICAgIGJyZWFrOw0KPj4+Pj4+ICsg ICAgICAgICAgICAgICB9DQo+Pj4+Pj4gKw0KPj4+Pj4+ICsgICAgICAgICAgICAgICBzY2hlZHVs ZSgpOw0KPj4+Pj4+ICsgICAgICAgfSB3aGlsZSAoMSk7DQo+Pj4+PiBJJ3ZlIHByZXZpb3VzbHkg dXNlZCBhIGRtYV9mZW5jZV9wcm94eSBzbyB0aGF0IHdlIGNvdWxkIGRvIG5vbmJsb2NraW5nDQo+ Pj4+PiB3YWl0cyBvbiBmdXR1cmUgc3VibWl0cy4gVGhhdCB3b3VsZCBiZSBwcmVmZXJyYWJsZSAo YSByZXF1aXJlbWVudCBmb3INCj4+Pj4+IG91ciBzdHVwaWQgQktMLWRyaXZlbiBjb2RlKS4NCj4+ Pj4gVGhhdCBpcyBleGFjdGx5IHdoYXQgSSB3b3VsZCBkZWZpbml0ZWx5IE5BSy4NCj4+Pj4NCj4+ Pj4gSSB3b3VsZCByYXRoZXIgc2F5IHdlIHNob3VsZCBjb21lIHVwIHdpdGggYSB3YWl0X211bHRp cGxlX2V2ZW50cygpIG1hY3JvDQo+Pj4+IGFuZCBjb21wbGV0ZWx5IG51a2UgdGhlIGN1c3RvbSBp bXBsZW1lbnRhdGlvbiBvZiB0aGlzIGluOg0KPj4+PiAxLiBkbWFfZmVuY2VfZGVmYXVsdF93YWl0 IGFuZCBkbWFfZmVuY2Vfd2FpdF9hbnlfdGltZW91dA0KPj4+PiAyLiB0aGUgcmFkZW9uIGZlbmNl IGltcGxlbWVudGF0aW9uDQo+Pj4+IDMuIHRoZSBub3V2ZWF1IGZlbmNlIGltcGxlbWVudGF0aW9u DQo+Pj4+IDQuIHRoZSBzeW5jb2JqIGNvZGUNCj4+Pj4NCj4+Pj4gQ2F1c2UgYWxsIG9mIHRoZW0g ZG8gZXhhY3RseSB0aGUgc2FtZS4gVGhlIGRtYV9mZW5jZSBpbXBsZW1lbnRhdGlvbg0KPj4+PiB1 bmZvcnR1bmF0ZWx5IGNhbWUgdXAgd2l0aCBhIGN1c3RvbSBldmVudCBoYW5kbGluZyBtZWNoYW5p c20gaW5zdGVhZCBvZg0KPj4+PiBleHRlbmRpbmcgdGhlIGNvcmUgTGludXggd2FpdF9ldmVudCgp IHN5c3RlbS4NCj4+PiBJIGRvbid0IHdhbnQgYSBibG9ja2luZyB3YWl0IGF0IGFsbC4NCj4+IE9r IEkgd2Fzbid0IGNsZWFyIGVub3VnaCA6KSBUaGF0IGlzIGV4YWN0bHkgd2hhdCBJIHdvdWxkIE5B SyENCj4+DQo+PiBUaGUgd2FpdCBtdXN0IGJlIGJsb2NraW5nIG9yIG90aGVyd2lzZSB5b3Ugd291 bGQgYWxsb3cgd2FpdC1iZWZvcmUtc2lnbmFsLg0KPiBXZWxsIHRoZSBjdXJyZW50IGltcGxlbWVu dGF0aW9uIGlzIHB1bGxpbmcgYSByYXRoZXIgYmlnIHRyaWNrIG9uIHJlYWRlcnMNCj4gaW4gdGhp cyByZWdhcmQ6IEl0IGxvb2tzIGxpa2UgYSBkbWFfZmVuY2UsIGl0J3MgaW1wbGVtZW50ZWQgYXMg b25lIGV2ZW4sDQo+IGhlY2sgeW91IGV2ZW4gb3Blbi1jb2RlIGEgZG1hX2ZlbmNlX3dhaXQgaGVy ZS4NCj4NCj4gRXhjZXB0IHRoZSBzZW1hbnRpY3MgYXJlIGNvbXBsZXRlbHkgZGlmZmVyZW50Lg0K Pg0KPiBTbyBhc2lkZSBmcm9tIHRoZSBkaXNjdXNzaW9uIHdoZXRoZXIgd2UgcmVhbGx5IHdhbnQg dG8gZnVsbHkgY2hhaW4gdGhlbSBJDQo+IHRoaW5rIGl0IGp1c3QgZG9lc24ndCBtYWtlIHNlbnNl IHRvIGltcGxlbWVudCB0aGUgIndhaXQgZm9yIGZlbmNlIHN1Ym1pdCINCj4gYXMgYSBkbWFfZmVu Y2Ugd2FpdC4gQW5kIEknZCBvdXRyaWdodCByZW1vdmUgdGhhdCBwYXJ0IGZyb20gdGhlIHVhcGks IGFuZA0KPiBmb3JjZSB0aGUgd2FpdC4gVGhlIGN1cnJlbnQgcmFkdi9hbnYgcGxhbnMgSSBkaXNj dXNzZWQgd2l0aCBKYXNvbiB3YXMgdGhhdA0KPiB3ZSdkIGhhdmUgYSBzZXBhcmF0ZSBzdWJtaXQg dGhyZWFkLCBhbmQgaGVuY2UgdW5jb25kaXRpb25hbGx5IGJsb2NraW5nDQo+IHVudGlsIHRoZSBm ZW5jZSBoYXMgbWF0ZXJpYWxpemVkIGlzIHRoZSByaWdodCB0aGluZyB0byBkby4gRXZlbiBhbGxv d2luZw0KPiB0aGF0IG9wdGlvbiwgZWl0aGVyIHRocm91Z2ggYSBmbGFnLCBvciBtYWtpbmcgdGhl c2UgdGhpbmdzIGxvb2sgbGlrZQ0KPiBkbWFfZmVuY2VzICh0aGV5IGFyZSBfbm90XykganVzdCB0 cmlja3MgZm9sa3MgaW50byBtaXN1bmRlcnN0YW5kaW5nIHdoYXQncw0KPiBnb2luZyBvbi4NCg0K R29vZCwgdGhhdCBzb3VuZHMgc3Ryb25nbHkgbGlrZSBzb21ldGhpbmcgSSBjYW4gYWdyZWUgb24g YXMgd2VsbC4NCg0KPiBDb2RlIHNoYXJpbmcganVzdCBiZWNhdXNlIHRoZSBjb2RlIGxvb2tzIHNp bWlsYXIgaXMgaW1vIGEgcmVhbGx5DQo+IGJhZCBpZGVhLCB3aGVuIHRoZSBzZW1hbnRpY3MgYXJl IGVudGlyZWx5IGRpZmZlcmVudCAodGhhdCB3YXMgYWxzbyB0aGUNCj4gcmVhc29uIGJlaGluZCBu b3QgcmV1c2luZyBhbGwgdGhlIGNwdSBldmVudCBzdHVmZiBmb3IgZG1hX2ZlbmNlLCB0aGV5J3Jl DQo+IG5vdCBub3JtYWwgY3B1IGV2ZW50cykuDQoNCk9rLCB0aGUgbGFzdCBzZW50ZW5jZSBpcyB3 aGF0IEkgZG9uJ3QgdW5kZXJzdGFuZC4NCg0KV2hhdCBleGFjdGx5IGlzIHRoZSBzZW1hbnRpYyBk aWZmZXJlbmNlIGJldHdlZW4gdGhlIGRtYV9mZW5jZV93YWl0IGFuZCANCnRoZSB3YWl0X2V2ZW50 IGludGVyZmFjZT8NCg0KSSBtZWFuIHRoZSB3YWl0X2V2ZW50IGludGVyZmFjZSB3YXMgaW50cm9k dWNlZCB0byBwcmV2ZW50IGRyaXZlcnMgZnJvbSANCm9wZW5seSBjb2RpbmcgYW4gZXZlbnQgaW50 ZXJmYWNlIGFuZCBnZXR0aW5nIGl0IHdyb25nIGFsbCB0aGUgdGltZS4NCg0KU28gYSBnb29kIHBh cnQgb2YgdGhlIGJ1Z3Mgd2UgaGF2ZSBzZWVuIGFyb3VuZCB3YWl0aW5nIGZvciBkbWEtZmVuY2Vz IA0KYXJlIGV4YWN0bHkgd2h5IHdhaXRfZXZlbnQgd2FzIGludmVudGVkIGluIHRoZSBmaXJzdCBw bGFjZS4NCg0KVGhlIG9ubHkgYmlnIHRoaW5nIEkgY2FuIHNlZSBtaXNzaW5nIGluIHRoZSB3YWl0 X2V2ZW50IGludGVyZmFjZSBpcyANCndhaXRpbmcgZm9yIG1hbnkgZXZlbnRzIGF0IHRoZSBzYW1l IHRpbWUsIGJ1dCB0aGF0IHNob3VsZCBiZSBhIHJhdGhlciANCmVhc3kgYWRkaXRpb24uDQoNClJl Z2FyZHMsDQpDaHJpc3RpYW4uDQoNCj4gLURhbmllbA0KPg0KPj4gQ2hyaXN0aWFuLg0KPj4NCj4+ PiAtQ2hyaXMNCj4+IF9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fDQo+PiBJbnRlbC1nZnggbWFpbGluZyBsaXN0DQo+PiBJbnRlbC1nZnhAbGlzdHMuZnJlZWRl c2t0b3Aub3JnDQo+PiBodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3Rp bmZvL2ludGVsLWdmeA0KDQpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fXwphbWQtZ2Z4IG1haWxpbmcgbGlzdAphbWQtZ2Z4QGxpc3RzLmZyZWVkZXNrdG9wLm9y ZwpodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2FtZC1nZngK