From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gustavo Padovan Subject: Re: [PATCH] dma-buf/sync-file: Avoid enable fence signaling if poll(.timeout=0) Date: Wed, 21 Sep 2016 10:26:25 +0300 Message-ID: <20160921072625.GA3473@joana> References: <20160829070834.22296-11-chris@chris-wilson.co.uk> <20160829181613.30722-1-chris@chris-wilson.co.uk> <20160915000041.rrh3qtur5bjp56sb@nadine2.fso.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mail-lf0-f65.google.com (mail-lf0-f65.google.com [209.85.215.65]) by gabe.freedesktop.org (Postfix) with ESMTPS id F28E46E7B8 for ; Wed, 21 Sep 2016 07:26:28 +0000 (UTC) Received: by mail-lf0-f65.google.com with SMTP id l131so1964379lfl.0 for ; Wed, 21 Sep 2016 00:26:28 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20160915000041.rrh3qtur5bjp56sb@nadine2.fso.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Rafael Antognolli Cc: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org SGkgUmFmYWVsLAoKMjAxNi0wOS0xNCBSYWZhZWwgQW50b2dub2xsaSA8cmFmYWVsLmFudG9nbm9s bGlAaW50ZWwuY29tPjoKCj4gSGkgQ2hyaXMgYW5kIEd1c3Rhdm8sCj4gCj4gT24gTW9uLCBBdWcg MjksIDIwMTYgYXQgMDc6MTY6MTNQTSArMDEwMCwgQ2hyaXMgV2lsc29uIHdyb3RlOgo+ID4gSWYg d2UgYmVpbmcgcG9sbGVkIHdpdGggYSB0aW1lb3V0IG9mIHplcm8sIGEgbm9uYmxvY2tpbmcgYnVz eSBxdWVyeSwKPiA+IHdlIGRvbid0IG5lZWQgdG8gaW5zdGFsbCBhbnkgZmVuY2UgY2FsbGJhY2tz IGFzIHdlIHdpbGwgbm90IGJlIHdhaXRpbmcuCj4gPiBBcyB3ZSBvbmx5IGluc3RhbGwgdGhlIGNh bGxiYWNrIG9uY2UsIHRoZSBvdmVyaGVhZCBjb21lcyBmcm9tIHRoZSBhdG9taWMKPiA+IGJpdCB0 ZXN0IHRoYXQgYWxzbyBjYXVzZXMgc2VyaWFsaXNhdGlvbiBiZXR3ZWVuIHRocmVhZHMuCj4gPiAK PiA+IFNpZ25lZC1vZmYtYnk6IENocmlzIFdpbHNvbiA8Y2hyaXNAY2hyaXMtd2lsc29uLmNvLnVr Pgo+ID4gQ2M6IFN1bWl0IFNlbXdhbCA8c3VtaXQuc2Vtd2FsQGxpbmFyby5vcmc+Cj4gPiBDYzog R3VzdGF2byBQYWRvdmFuIDxndXN0YXZvQHBhZG92YW4ub3JnPgo+ID4gQ2M6IGxpbnV4LW1lZGlh QHZnZXIua2VybmVsLm9yZwo+ID4gQ2M6IGRyaS1kZXZlbEBsaXN0cy5mcmVlZGVza3RvcC5vcmcK PiA+IENjOiBsaW5hcm8tbW0tc2lnQGxpc3RzLmxpbmFyby5vcmcKPiA+IC0tLQo+ID4gIGRyaXZl cnMvZG1hLWJ1Zi9zeW5jX2ZpbGUuYyB8IDMgKystCj4gPiAgMSBmaWxlIGNoYW5nZWQsIDIgaW5z ZXJ0aW9ucygrKSwgMSBkZWxldGlvbigtKQo+ID4gCj4gPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9k bWEtYnVmL3N5bmNfZmlsZS5jIGIvZHJpdmVycy9kbWEtYnVmL3N5bmNfZmlsZS5jCj4gPiBpbmRl eCA0ODZkMjljMWE4MzAuLmFiYjVmZGFiNzVmZCAxMDA2NDQKPiA+IC0tLSBhL2RyaXZlcnMvZG1h LWJ1Zi9zeW5jX2ZpbGUuYwo+ID4gKysrIGIvZHJpdmVycy9kbWEtYnVmL3N5bmNfZmlsZS5jCj4g PiBAQCAtMzA2LDcgKzMwNiw4IEBAIHN0YXRpYyB1bnNpZ25lZCBpbnQgc3luY19maWxlX3BvbGwo c3RydWN0IGZpbGUgKmZpbGUsIHBvbGxfdGFibGUgKndhaXQpCj4gPiAgCj4gPiAgCXBvbGxfd2Fp dChmaWxlLCAmc3luY19maWxlLT53cSwgd2FpdCk7Cj4gPiAgCj4gPiAtCWlmICghdGVzdF9hbmRf c2V0X2JpdChQT0xMX0VOQUJMRUQsICZzeW5jX2ZpbGUtPmZlbmNlLT5mbGFncykpIHsKPiA+ICsJ aWYgKCFwb2xsX2RvZXNfbm90X3dhaXQod2FpdCkgJiYKPiA+ICsJICAgICF0ZXN0X2FuZF9zZXRf Yml0KFBPTExfRU5BQkxFRCwgJnN5bmNfZmlsZS0+ZmVuY2UtPmZsYWdzKSkgewo+ID4gIAkJaWYg KGZlbmNlX2FkZF9jYWxsYmFjayhzeW5jX2ZpbGUtPmZlbmNlLCAmc3luY19maWxlLT5jYiwKPiA+ ICAJCQkJICAgICAgIGZlbmNlX2NoZWNrX2NiX2Z1bmMpIDwgMCkKPiA+ICAJCQl3YWtlX3VwX2Fs bCgmc3luY19maWxlLT53cSk7Cj4gCj4gVGhpcyBjb21taXQgaXMgY2F1c2luZyBhbiBlcnJvciBv biBvbmUgb2YgdGhlIHRlc3RzIHRoYXQgUm9iZXJ0IEZvc3MKPiBzdWJtaXR0ZWQgZm9yIGktZy10 LiBUaGUgb25lIHRoYXQgZG9lcyByYW5kb20gbWVyZ2Ugb2YgZmVuY2VzIGZyb20KPiBkaWZmZXJl bnQgdGltZWxpbmVzLiBBIHNpbXBsZSB2ZXJzaW9uIG9mIHRoZSB0ZXN0IHRoYXQgc3RpbGwgdHJp Z2dlcnMKPiB0aGlzIGlzOgo+IAo+IHN0YXRpYyB2b2lkIHRlc3Rfc3luY19zaW1wbGVfbWVyZ2Uo dm9pZCkKPiB7Cj4gICAgICAgICBpbnQgZmVuY2UxLCBmZW5jZTIsIGZlbmNlX21lcmdlLCB0aW1l bGluZTEsIHRpbWVsaW5lMjsKPiAgICAgICAgIGludCByZXQ7Cj4gCj4gICAgICAgICB0aW1lbGlu ZTEgPSBzd19zeW5jX3RpbWVsaW5lX2NyZWF0ZSgpOwo+ICAgICAgICAgdGltZWxpbmUyID0gc3df c3luY190aW1lbGluZV9jcmVhdGUoKTsKPiAgICAgICAgIGZlbmNlMSA9IHN3X3N5bmNfZmVuY2Vf Y3JlYXRlKHRpbWVsaW5lMSwgMSk7Cj4gICAgICAgICBmZW5jZTIgPSBzd19zeW5jX2ZlbmNlX2Ny ZWF0ZSh0aW1lbGluZTIsIDIpOwo+ICAgICAgICAgZmVuY2VfbWVyZ2UgPSBzd19zeW5jX21lcmdl KGZlbmNlMSwgZmVuY2UyKTsKPiAgICAgICAgIHN3X3N5bmNfdGltZWxpbmVfaW5jKHRpbWVsaW5l MSwgNSk7Cj4gICAgICAgICBzd19zeW5jX3RpbWVsaW5lX2luYyh0aW1lbGluZTIsIDUpOwo+IAo+ ICAgICAgICAgcmV0ID0gc3dfc3luY193YWl0KGZlbmNlX21lcmdlLCAwKTsKPiAgICAgICAgIGln dF9hc3NlcnRfZihyZXQgPiAwLCAiRmFpbHVyZSB0cmlnZ2VyaW5nIGZlbmNlXG4iKTsKPiAKPiAg ICAgICAgIHN3X3N5bmNfZmVuY2VfZGVzdHJveShmZW5jZV9tZXJnZSk7Cj4gICAgICAgICBzd19z eW5jX2ZlbmNlX2Rlc3Ryb3koZmVuY2UxKTsKPiAgICAgICAgIHN3X3N5bmNfZmVuY2VfZGVzdHJv eShmZW5jZTIpOwo+ICAgICAgICAgc3dfc3luY190aW1lbGluZV9kZXN0cm95KHRpbWVsaW5lMSk7 Cj4gICAgICAgICBzd19zeW5jX3RpbWVsaW5lX2Rlc3Ryb3kodGltZWxpbmUyKTsKPiB9Cj4gCj4g SXQgbG9va3MgbGlrZSB5b3UgY2Fubm90IHRydXN0IGZlbmNlX2lzX3NpZ25hbGVkKCkgd2l0aG91 dCBhCj4gZmVuY2VfYWRkX2NhbGxiYWNrKCkuIEkgdGhpbmsgdGhlIGZlbmNlX2FycmF5LT5udW1f cGVuZGluZyB3b24ndCBnZXQKPiB1cGRhdGVkLiBBbHRob3VnaCBJIGNvdWxkbid0IGZpZ3VyZSBv dXQgd2h5IGl0IG9ubHkgaGFwcGVucyBpZiB5b3UgbWVyZ2UKPiBmZW5jZXMgZnJvbSBkaWZmZXJl bnQgdGltZWxpbmVzLgoKWWVzLCBudW1fcGVuZGluZyBpcyBvbmx5IHVwZGF0ZWQgd2hlbiBzaWdu YWxpbmcgaXMgZW5hYmxlZC4gSXQgb25seQpoYXBwZW5zIHdpdGggZGlmZmVyZW50IHRpbWVsaW5l cyBiZWNhdXNlIHdoZW4geW91IG1lcmdlIGZlbmNlcyB0aGF0IGFyZQpvbiB0aGUgc2FtZSB0aW1l bGluZSB5b3VyIGZpbmFsIHN5bmNfZmlsZSBoYXMgb25seSBvbmUgZmVuY2UgYW5kIHRodXMgCmEg ZmVuY2VfYXJyYXkgaXMgbm90IGNyZWF0ZWQuCgpJZiB3ZSB3YW50IHRvIGtlZXAgdGhlIHBvbGxf ZG9lc19ub3Rfd2FpdCBvcHRpbWl6YXRpb24gd2UgbmVlZCBhIHdheQp0byBjb3VudCB0aGUgcGVu ZGluZyBmZW5jZXMgZHVyaW5nIGZlbmNlX2lzX3NpZ25hbGVkKCkuIEknZCBwcm9wb3NlCnNvbWV0 aGluZyBsaWtlIHRoaXM6CgoKQXV0aG9yOiBHdXN0YXZvIFBhZG92YW4gPGd1c3Rhdm8ucGFkb3Zh bkBjb2xsYWJvcmEuY28udWs+CkRhdGU6ICAgVHVlIFNlcCAyMCAxNjo0MzowNiAyMDE2ICswMjAw CgogICAgZG1hLWJ1Zi9mZW5jZS1hcnJheTogZ2V0IHNpZ25hbGVkIHN0YXRlIHdoZW4gc2lnbmFs aW5nIGlzIGRpc2FibGVkCiAgICAKICAgIElmIHRoZSBmZW5jZXMgaW4gdGhlIGZlbmNlX2FycmF5 IHNpZ25hbCBvbiB0aGUgZmVuY2VfYXJyYXkgZG9lcyBub3QgaGF2ZQogICAgc2lnbmFsbGluZyBl bmFibGVkIG51bV9wZW5kaW5nIHdpbGwgbm90IGJlIHVwZGF0ZWQgYWNjb3JkaW5nbHkuCiAgICAK ICAgIFNvIHdoZW4gc2lnbmFsaW5nIGlzIGRpc2FibGVkIGNoZWNrIHRoZSBzaWduYWwgb2YgZXZl cnkgZmVuY2Ugd2l0aAogICAgZmVuY2VfaXNfc2lnbmFsZWQoKSBhbmQgdGhlbiBjb21wYXJlIHdp dGggbnVtX3BlbmRpbmcgdG8gbGVhcm4gaWYgdGhlCiAgICBmZW5jZV9hcnJheSB3YXMgc2lnbmFs bGVkIG9yIG5vdC4KICAgIAogICAgSWYgd2Ugd2FudCB0byBrZWVwIHRoZSBwb2xsX2RvZXNfbm90 X3dhaXQgb3B0aW1pemF0aW9uIEkgdGhpbmsgd2UgbmVlZAogICAgc29tZXRoaW5nIGxpa2UgdGhp cy4gSXQga2VlcHMgdGhlIHNhbWUgYmVoYXZpb3VyIGlmIHNpZ25hbGxpbmcgaXMgZW5hYmxlZAog ICAgYnV0IHRyaWVzIHRvIGNhbGN1bGF0ZWQgdGhlIHN0YXRlIG90aGVyd2lzZS4KICAgIAogICAg U2lnbmVkLW9mZi1ieTogR3VzdGF2byBQYWRvdmFuIDxndXN0YXZvLnBhZG92YW5AY29sbGFib3Jh LmNvLnVrPgoKZGlmZiAtLWdpdCBhL2RyaXZlcnMvZG1hLWJ1Zi9mZW5jZS1hcnJheS5jIGIvZHJp dmVycy9kbWEtYnVmL2ZlbmNlLWFycmF5LmMKaW5kZXggZjE5ODlmYy4uMzRjOTIwOSAxMDA2NDQK LS0tIGEvZHJpdmVycy9kbWEtYnVmL2ZlbmNlLWFycmF5LmMKKysrIGIvZHJpdmVycy9kbWEtYnVm L2ZlbmNlLWFycmF5LmMKQEAgLTc1LDggKzc1LDE4IEBAIHN0YXRpYyBib29sIGZlbmNlX2FycmF5 X2VuYWJsZV9zaWduYWxpbmcoc3RydWN0IGZlbmNlICpmZW5jZSkKIHN0YXRpYyBib29sIGZlbmNl X2FycmF5X3NpZ25hbGVkKHN0cnVjdCBmZW5jZSAqZmVuY2UpCiB7CiAgICAgICAgc3RydWN0IGZl bmNlX2FycmF5ICphcnJheSA9IHRvX2ZlbmNlX2FycmF5KGZlbmNlKTsKKyAgICAgICBpbnQgaSwg bnVtX3BlbmRpbmc7CiAKLSAgICAgICByZXR1cm4gYXRvbWljX3JlYWQoJmFycmF5LT5udW1fcGVu ZGluZykgPD0gMDsKKyAgICAgICBudW1fcGVuZGluZyA9IGF0b21pY19yZWFkKCZhcnJheS0+bnVt X3BlbmRpbmcpOworCisgICAgICAgaWYgKCF0ZXN0X2JpdChGRU5DRV9GTEFHX0VOQUJMRV9TSUdO QUxfQklULCAmZmVuY2UtPmZsYWdzKSkgeworICAgICAgICAgICAgICAgZm9yIChpID0gMCA7IGkg PCBhcnJheS0+bnVtX2ZlbmNlczsgKytpKSB7CisgICAgICAgICAgICAgICAgICAgICAgIGlmIChm ZW5jZV9pc19zaWduYWxlZChhcnJheS0+ZmVuY2VzW2ldKSkKKyAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICBudW1fcGVuZGluZy0tOworICAgICAgICAgICAgICAgfQorICAgICAgIH0KKwor ICAgICAgIHJldHVybiBudW1fcGVuZGluZyA8PSAwOwogfQogCiBzdGF0aWMgdm9pZCBmZW5jZV9h cnJheV9yZWxlYXNlKHN0cnVjdCBmZW5jZSAqZmVuY2UpCgoKR3VzdGF2bwpfX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpkcmktZGV2ZWwgbWFpbGluZyBsaXN0 CmRyaS1kZXZlbEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3Rv cC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwK