From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?Q?Christian_K=c3=b6nig?= Subject: Re: [RFC 0/5] rework fences on struct sync_file Date: Fri, 24 Jun 2016 17:09:51 +0200 Message-ID: <576D4D3F.8080808@amd.com> References: <1466695790-2833-1-git-send-email-gustavo@padovan.org> <576CFD0B.6000501@amd.com> <20160624131724.GA2503@joana> <576D4032.3000001@vodafone.de> <20160624145933.GF2508@joana> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: Received: from na01-by2-obe.outbound.protection.outlook.com (mail-by2on0060.outbound.protection.outlook.com [207.46.100.60]) by gabe.freedesktop.org (Postfix) with ESMTPS id 8D1A86EAEC for ; Fri, 24 Jun 2016 15:25:15 +0000 (UTC) In-Reply-To: <20160624145933.GF2508@joana> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Gustavo Padovan , =?UTF-8?Q?Christian_K=c3=b6nig?= Cc: marcheu@google.com, Daniel Stone , seanpaul@google.com, Daniel Vetter , linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, laurent.pinchart@ideasonboard.com, Gustavo Padovan , John Harrison , m.chehab@samsung.com List-Id: dri-devel@lists.freedesktop.org QW0gMjQuMDYuMjAxNiB1bSAxNjo1OSBzY2hyaWViIEd1c3Rhdm8gUGFkb3ZhbjoKPiAyMDE2LTA2 LTI0IENocmlzdGlhbiBLw7ZuaWcgPGRlYXRoc2ltcGxlQHZvZGFmb25lLmRlPjoKPgo+PiBBbSAy NC4wNi4yMDE2IHVtIDE1OjE3IHNjaHJpZWIgR3VzdGF2byBQYWRvdmFuOgo+Pj4gSGkgQ2hyaXN0 aWFuLAo+Pj4KPj4+IDIwMTYtMDYtMjQgQ2hyaXN0aWFuIEvDtm5pZyA8Y2hyaXN0aWFuLmtvZW5p Z0BhbWQuY29tPjoKPj4+Cj4+Pj4gQW0gMjMuMDYuMjAxNiB1bSAxNzoyOSBzY2hyaWViIEd1c3Rh dm8gUGFkb3ZhbjoKPj4+Pj4gRnJvbTogR3VzdGF2byBQYWRvdmFuIDxndXN0YXZvLnBhZG92YW5A Y29sbGFib3JhLmNvLnVrPgo+Pj4+Pgo+Pj4+PiBIaSBhbGwsCj4+Pj4+Cj4+Pj4+IFRoaXMgaXMg YW4gYXR0ZW1wdCB0byBpbXByb3ZlIGZlbmNlIHN1cHBvcnQgb24gU3luYyBGaWxlLiBUaGUgYmFz aWMgaWRlYQo+Pj4+PiBpcyB0byBoYXZlIG9ubHkgc3luY19maWxlLT5mZW5jZSBhbmQgc3RvcmUg YWxsIGZlbmNlcyB0aGVyZSwgZWl0aGVyIGFzCj4+Pj4+IG5vcm1hbCBmZW5jZXMgb3IgZmVuY2Vf YXJyYXlzLiBUaGF0IHdheSB3ZSBjYW4gcmVtb3ZlIHNvbWUgcG90ZW50aWFsCj4+Pj4+IGR1cGxp Y2F0aW9uIHdoZW4gdXNpbmcgZmVuY2VfYXJyYXkgd2l0aCBzeW5jX2ZpbGU6IHRoZSBkdXBsaWNh dGlvbiBvZiB0aGUgYXJyYXkKPj4+Pj4gb2YgZmVuY2VzIGFuZCB0aGUgZHVwbGljYXRpb24gb2Yg ZmVuY2VfYWRkX2NhbGxiYWNrKCkgZm9yIGFsbCBmZW5jZXMuCj4+Pj4+Cj4+Pj4+IE5vdyB3aGVu IGNyZWF0aW5nIGEgbmV3IHN5bmNfZmlsZSBkdXJpbmcgdGhlIG1lcmdlIHByb2Nlc3Mgc3luY19m aWxlX3NldF9mZW5jZSgpCj4+Pj4+IHdpbGwgc2V0IHN5bmNfZmlsZS0+ZmVuY2UgYmFzZWQgb24g dGhlIG51bWJlciBvZiBmZW5jZXMgZm9yIHRoYXQgc3luY19maWxlLiBJZgo+Pj4+PiB0aGVyZSBp cyBtb3JlIHRoYW4gb25lIGZlbmNlIGEgZmVuY2VfYXJyYXkgaXMgY3JlYXRlZC4gT25lIGltcG9y dGFudCBhZHZhbnRhZ2UKPj4+Pj4gYXBwcm9hY2ggaXMgdGhhdCB3ZSBvbmx5IGFkZCBvbmUgZmVu Y2UgY2FsbGJhY2sgbm93LCBubyBtYXR0ZXIgaG93IG1hbnkgZmVuY2VzCj4+Pj4+IHRoZXJlIGFy ZSBpbiBhIHN5bmNfZmlsZSAtIHRoZSBpbmRpdmlkdWFsIGNhbGxiYWNrcyBhcmUgYWRkZWQgYnkg ZmVuY2VfYXJyYXkuCj4+Pj4+Cj4+Pj4+IFR3byBmZW5jZSBvcHMgaGFkIHRvIGJlIGNyZWF0ZWQg dG8gaGVscCBhYnN0cmFjdCB0aGUgZGlmZmVyZW5jZSBiZXR3ZWVuIGhhbmRsaW5nCj4+Pj4+IGZl bmNlcyBhbmQgZmVuY2VzX2FycmF5czogLnRlYXJkb3duKCkgYW5kIC5nZXRfZmVuY2VzKCkuIFRo ZSBmb3JtZXIgcnVuIG5lZWRlZAo+Pj4+PiBvbiBmZW5jZV9hcnJheSwgYW5kIHRoZSBsYXR0ZXIg anVzdCByZXR1cm4gYSBjb3B5IG9mIGFsbCBmZW5jZXMgaW4gdGhlIGZlbmNlLgo+Pj4+PiBJJ20g bm90IHNvIHN1cmUgYWJvdXQgYWRkaW5nIHRob3NlIHR3bywgc3BlYWNpYWxseSAuZ2V0X2ZlbmNl cygpLiBXaGF0IGRvIHlvdQo+Pj4+PiB0aGluaz8KPj4+PiBDbGVhcmx5IG5vdCBhIGdvb2QgaWRl YSB0byBhZGQgdGhpcyBhIGZlbmNlIG9wcywgY2F1c2UgdGhvc2UgYXJlIHNwZWNpYWxpemVkCj4+ Pj4gZnVuY3Rpb25zIGZvciBvbmx5IGEgY2VydGFpbiBmZW5jZSBpbXBsZW1lbnRhdGlvbiAodGhl IGZlbmNlX2FycmF5KS4KPj4+IEFyZSB5b3UgcmVmZXJpbmcgb25seSB0byAuZ2V0X2ZlbmNlcygp Pwo+PiBUaGF0IGNvbW1lbnQgd2FzIG9ubHkgZm9yIHRoZSBnZXRfZmVuY2VzKCkgb3BlcmF0aW9u LCBidXQgdGhlIHRlYXJkb3duKCkKPj4gY2FsbGJhY2sgbG9va3MgdmVyeSBzdXNwaWNpb3VzIHRv IG1lIGFzIHdlbGwuCj4+Cj4+IENhbiB5b3UgZXhwbGFpbiBvbmNlIG1vcmUgd2h5IHRoYXQgc2hv dWxkIGJlIG5lY2Vzc2FyeT8KPiBXaGVuIHRoZSBzeW5jX2ZpbGUgb3duZXIgZXhpdHMgd2UgbmVl ZCB0byBjbGVhbiB1cCBpdCBhbmQgdGhhdCBtZWFucyByZWxlYXNpbmcKPiB0aGUgZmVuY2UgdG9v LCBob3dldmVyIHdpdGggZmVuY2VfYXJyYXkgd2UgY2FuJ3QganVzdCBjYWxsIGZlbmNlX3B1dCgp Cj4gYXMgYSBleHRyYSByZWZlcmVuY2UgdG8gYXJyYXktPmJhc2UgZm9yIGVhY2ggZmVuY2UgaXMg aGVsZCB3aGVuIGVuYWJsaW5nCj4gc2lnbmFsbGluZy4gVGh1cyB3ZSBuZWVkIGEgcHJpb3Igc3Rl cCwgdGhhdCBJIGNhbGxlZCB0ZWFyZG93bigpLCB0bwo+IHJlbW92ZSB0aGUgY2FsbGJhY2sgZm9y IG5vdCBzaWduYWxlZCBmZW5jZXMgYW5kIHB1dCB0aGUgZXh0cmEKPiByZWZlcmVuY2VzLgo+Cj4g QW5vdGhlciB3YXkgdG8gZG8gdGhpcyB3b3VsZCBiZToKPgo+IAlpZiAoZmVuY2VfaXNfYXJyYXko c3luY19maWxlLT5mZW5jZSkpCj4gCQlmZW5jZV9hcnJheV9kZXN0cm95KHRvX2ZlbmNlX2FycmF5 KHN5bmNfZmlsZS0+ZmVuY2UpKTsKPiAJZWxzZQo+IAkJZmVuY2VfcHV0KHN5bmNfZmlsZV9mZW5j ZSk7Cj4KPiBUaGlzIHdvdWxkIGF2b2lkIHRoZSBleHRyYSBvcHMsIG1heWJlIHdlIHNob3VsZCBn byB0aGlzIHdheS4KCk5BSyBvbiBib3RoIGFwcHJvYWNoZXMuIFRoZSBmZW5jZSBhcnJheSBncmFi cyBhbm90aGVyIHJlZmVyZW5jZSBvbiAKaXRzZWxmIGZvciBlYWNoIGNhbGxiYWNrIGl0IHJlZ2lz dGVycywgc28gdGhpcyBpc24ndCBuZWNlc3Nhcnk6Cgo+ICAgICAgICAgZm9yIChpID0gMDsgaSA8 IGFycmF5LT5udW1fZmVuY2VzOyArK2kpIHsKPiAgICAgICAgICAgICAgICAgY2JbaV0uYXJyYXkg PSBhcnJheTsKPiAgICAgICAgICAgICAgICAgLyoKPiAgICAgICAgICAgICAgICAgICogQXMgd2Ug bWF5IHJlcG9ydCB0aGF0IHRoZSBmZW5jZSBpcyBzaWduYWxlZCBiZWZvcmUgYWxsCj4gICAgICAg ICAgICAgICAgICAqIGNhbGxiYWNrcyBhcmUgY29tcGxldGUsIHdlIG5lZWQgdG8gdGFrZSBhbiBh ZGRpdGlvbmFsCj4gICAgICAgICAgICAgICAgICAqIHJlZmVyZW5jZSBjb3VudCBvbiB0aGUgYXJy YXkgc28gdGhhdCB3ZSBkbyBub3QgZnJlZSAKPiBpdCB0b28KPiAgICAgICAgICAgICAgICAgICog ZWFybHkuIFRoZSBjb3JlIGZlbmNlIGhhbmRsaW5nIHdpbGwgb25seSBob2xkIHRoZSAKPiByZWZl cmVuY2UKPiAgICAgICAgICAgICAgICAgICogdW50aWwgd2Ugc2lnbmFsIHRoZSBhcnJheSBhcyBj b21wbGV0ZSAoYnV0IHRoYXQgaXMgbm93Cj4gICAgICAgICAgICAgICAgICAqIGluc3VmZmljaWVu dCkuCj4gICAgICAgICAgICAgICAgICAqLwo+ICAgICAgICAgICAgICAgICBmZW5jZV9nZXQoJmFy cmF5LT5iYXNlKTsKPiAgICAgICAgICAgICAgICAgaWYgKGZlbmNlX2FkZF9jYWxsYmFjayhhcnJh eS0+ZmVuY2VzW2ldLCAmY2JbaV0uY2IsCj4gICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgZmVuY2VfYXJyYXlfY2JfZnVuYykpIHsKPiAgICAgICAgICAgICAgICAgICAgICAg ICBmZW5jZV9wdXQoJmFycmF5LT5iYXNlKTsKPiAgICAgICAgICAgICAgICAgICAgICAgICBpZiAo YXRvbWljX2RlY19hbmRfdGVzdCgmYXJyYXktPm51bV9wZW5kaW5nKSkKPiAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgIHJldHVybiBmYWxzZTsKPiAgICAgICAgICAgICAgICAgfQo+ICAg ICAgICAgfQoKU28geW91IGNhbiBqdXN0IHVzZSBmZW5jZV9yZW1vdmVfY2FsbGJhY2soKSBhbmQg dGhlbiBmZW5jZV9wdXQoKSB3aXRob3V0IAp3b3JyeWluZyBhYm91dCB0aGUgcmVmZXJlbmNlLgoK UmVnYXJkcywKQ2hyaXN0aWFuLgoKPgo+IAlHdXN0YXZvCgpfX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fXwpkcmktZGV2ZWwgbWFpbGluZyBsaXN0CmRyaS1kZXZl bEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFp bG1hbi9saXN0aW5mby9kcmktZGV2ZWwK