From mboxrd@z Thu Jan 1 00:00:00 1970 From: Inki Dae Subject: Re: [RFC 0/6] drm/fences: add in-fences to DRM Date: Fri, 25 Mar 2016 08:49:21 +0900 Message-ID: <56F47D01.7040508@samsung.com> References: <1458758847-21170-1-git-send-email-gustavo@padovan.org> <56F3A2DC.8080507@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mailout1.samsung.com (mailout1.samsung.com [203.254.224.24]) by gabe.freedesktop.org (Postfix) with ESMTPS id A7BA66E1C5 for ; Thu, 24 Mar 2016 23:49:24 +0000 (UTC) Received: from epcpsbgr1.samsung.com (u141.gpu120.samsung.co.kr [203.254.230.141]) by mailout1.samsung.com (Oracle Communications Messaging Server 7.0.5.31.0 64bit (built May 5 2014)) with ESMTP id <0O4K028BSI69ACB0@mailout1.samsung.com> for dri-devel@lists.freedesktop.org; Fri, 25 Mar 2016 08:49:21 +0900 (KST) In-reply-to: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Rob Clark Cc: Daniel Stone , Linux Kernel Mailing List , "dri-devel@lists.freedesktop.org" , =?UTF-8?B?QXJ2ZSBIasO4bm5ldsOlZw==?= , Daniel Vetter , Riley Andrews , Gustavo Padovan , John Harrison List-Id: dri-devel@lists.freedesktop.org CgoyMDE264WEIDAz7JuUIDI17J28IDAwOjQw7JeQIFJvYiBDbGFyayDsnbQo6rCAKSDsk7Qg6riA Ogo+IE9uIFRodSwgTWFyIDI0LCAyMDE2IGF0IDQ6MTggQU0sIElua2kgRGFlIDxpbmtpLmRhZUBz YW1zdW5nLmNvbT4gd3JvdGU6Cj4+IEhpLAo+Pgo+PiAyMDE264WEIDAz7JuUIDI07J28IDAzOjQ3 7JeQIEd1c3Rhdm8gUGFkb3ZhbiDsnbQo6rCAKSDsk7Qg6riAOgo+Pj4gRnJvbTogR3VzdGF2byBQ YWRvdmFuIDxndXN0YXZvLnBhZG92YW5AY29sbGFib3JhLmNvLnVrPgo+Pj4KPj4+IEhpLAo+Pj4K Pj4+IFRoaXMgaXMgYSBmaXJzdCBwcm9wb3NhbCB0byBkaXNjdXNzIHRoZSBhZGRpdGlvbiBvZiBp bi1mZW5jZXMgc3VwcG9ydAo+Pj4gdG8gRFJNLiBJdCBhZGRzIGEgbmV3IHN0cnVjdCB0byBmZW5j ZS5jIHRvIGFic3RyYWN0IHRoZSB1c2Ugb2Ygc3luY19maWxlCj4+PiBpbiBEUk0gZHJpdmVycy4g VGhlIG5ldyBzdHJ1Y3QgZmVuY2VfY29sbGVjdGlvbiBjb250YWlucyBhIGFycmF5IHdpdGggYWxs Cj4+PiBmZW5jZXMgdGhhdCBhIGF0b21pYyBjb21taXQgbmVlZHMgdG8gd2FpdCBvbgo+Pgo+PiBB cyBJIG1lbnRpb25lZCBhbHJlYWR5IGxpa2UgYmVsb3csCj4+IGh0dHA6Ly93d3cuc3Bpbmljcy5u ZXQvbGlzdHMvZHJpLWRldmVsL21zZzEwMzIyNS5odG1sCj4+Cj4+IEkgZG9uJ3Qgc2VlIHdoeSBB bmRyb2lkIHNwZWNpZmljIHRoaW5nIGlzIHRyaWVkIHRvIHByb3BhZ2F0ZSB0byBMaW51eCBEUk0u IEluIExpbnV4IG1haW5saW5lLCBpdCBoYXMgYWxyZWFkeSBpbXBsaWNpdCBzeW5jIGludGVyZmFj ZXMgZm9yIERNQSBkZXZpY2VzIGNhbGxlZCBkbWEgZmVuY2Ugd2hpY2ggZm9yY2VzIHJlZ2lzdGVy aW5nIGEgZmVuY2Ugb2JlamN0IHRvIERNQUJVRiB0aHJvdWdoIGEgcmVzZXJ2YXRpb24gb2JlamN0 IHdoZW4gYSBkbWFidWYgb2JqZWN0IGlzIGNyZWF0ZWQuIEhvd2V2ZXIsIEFuZHJvaWQgc3luYyBk cml2ZXIgY3JlYXRlcyBhIG5ldyBmaWxlIGZvciBhIHN5bmMgb2JqZWN0IGFuZCB0aGlzIHdvdWxk IGhhdmUgZGlmZmVyZW50IHBvaW50IG9mIHZpZXcuCj4+Cj4+IElzIHRoZXJlIGFueW9uZSB3aG8g Y2FuIGV4cGxhbiB3aHkgQW5kcm9pZCBzcGVjaWZpYyB0aGluZyBpcyB0cmllZCB0byBzcHJlYWQg aW50byBMaW51eCBEUk0/IFdhcyB0aGVyZSBhbnkgY29uc2Vuc3VzIHRvIHVzZSBBbmRyb2lkIHN5 bmMgZHJpdmVyIC0gd2hpY2ggdXNlcyBleHBsaWNpdCBzeW5jIGludGVyZmFjZXMgLSBhcyBMaW51 eCBzdGFuZGFyZD8KPj4KPiAKPiBidHcsIHRoZXJlIGlzIGFscmVhZHkgcGxhbmVfc3RhdGUtPmZl bmNlIC4uIHdoaWNoIEkgZG9uJ3QgdGhpbmsgaGFzCj4gYW55IHVzZXJzIHlldCwgYnV0IEkgc3Rh cnQgdG8gdXNlIGl0IGluIG15IHBhdGNoc2V0IHRoYXQgY29udmVydHMKPiBkcm0vbXNtIHRvICdz dHJ1Y3QgZmVuY2UnCgpZZXMsIEV4eW5vcyBhbHNvIHN0YXJ0ZWQgaXQuCgo+IAo+IFRoYXQgc2Fp ZCwgd2UgZG8gbmVlZCBzeW5jcHQgYXMgdGhlIHdheSB0byBleHBvc2UgZmVuY2VzIHRvIHVzZXJz cGFjZQo+IGZvciBleHBsaWNpdCBzeW5jaHJvbml6YXRpb24sIGJ1dCBJJ20gbm90IGVudGlyZWx5 IHN1cmUgdGhhdCB0aGUKCkl0J3MgZGVmaW5pdGVseSBkaWZmZXJlbnQgY2FzZS4gVGhpcyB0cmll cyB0byBhZGQgbmV3IHVzZXItc3BhY2UgaW50ZXJmYWNlcyB0byBleHBvc2UgZmVuY2VzIHRvIHVz ZXItc3BhY2UuIEF0IGxlYXN0LCBpbXBsaWNpdCBpbnRlcmZhY2VzIGFyZSBlbWJlZGRlZCBpbnRv IGRyaXZlcnMuClNvIEknZCBsaWtlIHRvIGdpdmUgeW91IGEgcXVlc3Rpb24uIFdoeSBleHBvc2lu ZyBmZW5jZXMgdG8gdXNlci1zcGFjZSBpcyByZXF1aXJlZD8gVG8gcHJvdmlkZSBlYXN5LXRvLWRl YnVnIHNvbHV0aW9uIHRvIHJlbmRlcmluZyBwaXBsZWxpbmU/IFRvIHByb3ZpZGUgbWVyZ2UgZmVu Y2UgZmVhdHVyZT8gCgpBbmQgaWYgd2UgbmVlZCByZWFsbHkgdG8gZXhwb3NlIGZlbmNlcyB0byB1 c2VyLXNwYWNlIGFuZCB0aGVyZSBpcyByZWFsbHkgcmVhbCB1c2VyLCB0aGVuIHdlIGhhdmUgYWxy ZWFkeSBnb29kIGNhbmRpZGF0ZXMsIERNQS1CVUYtSU9DVEwtU1lOQyBvciBtYXliZSBmY250bCBz eXN0ZW0gY2FsbCBiZWNhdXNlIHdlIHNoYXJlIGFscmVhZHkgRE1BIGJ1ZmZlciBiZXR3ZWVuIENQ VSA8LT4gRE1BIGFuZCBETUEgPC0+IERNQSB1c2luZyBETUFCVUYuCkZvciBETUEtQlVGLUlPQ1RM LVNZTkMsIEkgdGhpbmsgeW91IHJlbWVtYmVyIHRoYXQgd2FzIHdoYXQgSSB0cmllZCBsb25nIHRp bWUgYWdvIGJlY2F1c2UgeW91IHdhcyB0aGVyZS4gU2V2ZXJhbCB5ZWFycyBhZ28sIEkgdHJpZWQg dG8gY291cGxlIGV4cG9zaW5nIHRoZSBmZW5jZXMgdG8gdXNlci1zcGFjZSB3aXRoIGNhY2hlIG9w ZXJhdGlvbiBhbHRob3VnaCBhdCB0aGF0IHRpbWUsIEkgcmVhbGx5IG1pc2xlYWRlZCB0aGUgZmVu Y2UgbWFjaG5pc20uIE15IHRyeWluZyB3YXMgYWxzbyBmb3IgdGhlIHBvdGVudGlhbCB1c2Vycy4K CkFueXdheSwgbXkgb3BpbmlvbiBpcyB0aGF0IHdlIGNvdWxkIGV4cG9zZSB0aGUgZmVuY2VzIGhp ZGVkIGJ5IERNQUJVRiB0byB1c2VyLXNwYWNlIHVzaW5nIGludGVyZmFjZXMgaXQgZXhpc3RzIGFs cmVhZHkgYXJvdW5kIHVzLiBBbmQgZm9yIHRoaXMsIGJlbG93IENocm9taXVtIHNvbHV0aW9uIHdv dWxkIGFsc28gZ2l2ZSB1cyBzb21lIGhlbHBzLApodHRwczovL2Nocm9taXVtLmdvb2dsZXNvdXJj ZS5jb20vY2hyb21pdW1vcy90aGlyZF9wYXJ0eS9rZXJuZWwvKy9jaHJvbWVvcy0zLjE4L2RyaXZl cnMvZ3B1L2RybS9kcm1fc3luY19oZWxwZXIuYwoKQW5kIGluIC9kcml2ZXIvZG1hLWJ1Zi8sIHRo ZXJlIGFyZSBETUFCVUYtY2VudHJpYyBtb2R1bGVzIHNvIGxvb2tzIHN0cmFuZ2Ugc3luY19maWxl IG1vZHVsZSBvZiBBbmRyb2lkIGlzIHBsYWNlZCBpbiB0aGF0IGRpcmVjdG9yeSAtIEFuZHJvaWQg c3luYyBkcml2ZXIgZG9lc24ndCB1c2UgcmVhbGx5IERNQUJVRiBidXQgY3JlYXRlcyBuZXcgZmls ZSBmb3IgdGhlaXIgc3luYyBmZW5jZSBpbnN0ZWFkLiAKRm9yIGltcGxpY2l0IHN5bmMgaW50ZXJm YWNlcyBmb3IgRE1BIGRldmljZXMsIHdlIHVzZSBETUFCVUYgYW5kIGZvciBleHBsaWN0IHN5bmMg aW50ZXJmYWNlcyBmb3IgdXNlci1zcGFjZSwgd2UgdXNlIHN5bmNfZmlsZSBub3QgRE1BQlVGPyBU aGF0IGRvZXNuJ3QgbWFrZSBzZW5zZS4gCgpJIGxvdmUgcmVhbGx5IEFuZHJvaWQgYnV0IEkgZmVl bCBhcyBpZiB3ZSB0cnkgdG8gZ2l2ZSBhIHNlYXQgYXZhaWxhYmxlIHRvIEFuZHJvaWQgc29tZWhv dy4KClRoYW5rcywKSW5raSBEYWUKCj4gdmFyaW91cyBkcml2ZXJzIGV2ZXIgbmVlZCB0byBzZWUg dGhhdCAodnMganVzdCBzdHJ1Y3QgZmVuY2UpLCBhdCBsZWFzdAo+IG9uIHRoZSBrbXMgc2lkZSBv ZiB0aGluZ3MuCj4gCj4gQlIsCj4gLVIKPiAKPiAKPj4gVGhhbmtzLAo+PiBJbmtpIERhZQo+Pgo+ Pj4KPj4+IC8qKgo+Pj4gICogc3RydWN0IGZlbmNlX2NvbGxlY3Rpb24gLSBhZ2dyZWdhdGUgZmVu Y2VzIHRvZ2V0aGVyCj4+PiAgKiBAbnVtX2ZlbmNlczogbnVtYmVyIG9mIGZlbmNlIGluIHRoZSBj b2xsZWN0aW9uLgo+Pj4gICogQHVzZXJfZGF0YTogdXNlciBkYXRhLgo+Pj4gICogQGZ1bmM6IHVz ZXIgY2FsbGJhY2sgdG8gcHV0IHVzZXIgZGF0YS4KPj4+ICAqIEBmZW5jZXM6IGFycmF5IG9mIEBu dW1fZmVuY2VzIGZlbmNlcy4KPj4+ICAqLwo+Pj4gc3RydWN0IGZlbmNlX2NvbGxlY3Rpb24gewo+ Pj4gICAgICAgIGludCBudW1fZmVuY2VzOwo+Pj4gICAgICAgIHZvaWQgKnVzZXJfZGF0YTsKPj4+ ICAgICAgICBjb2xsZWN0aW9uX3B1dF9mdW5jX3QgZnVuYzsKPj4+ICAgICAgICBzdHJ1Y3QgZmVu Y2UgKmZlbmNlc1tdOwo+Pj4gfTsKPj4+Cj4+Pgo+Pj4gVGhlIGZlbmNlX2NvbGxlY3Rpb24gaXMg YWxsb2NhdGVkIGFuZCBmaWxsZWQgYnkgc3luY19maWxlX2ZlbmNlc19nZXQoKSBhbmQKPj4+IGF0 b21pY19jb21taXQgaGVscGVycyBjYW4gdXNlIGZlbmNlX2NvbGxlY3Rpb25fd2FpdCgpIHRvIHdh aXQgdGhlIGZlbmNlcyB0bwo+Pj4gc2lnbmFsLgo+Pj4KPj4+IFRoZXNlIHBhdGNoZXMgZGVwZW5k cyBvbiB0aGUgc3luYyBBQkkgcmV3b3JrOgo+Pj4KPj4+IGh0dHBzOi8vd3d3LnNwaW5pY3MubmV0 L2xpc3RzL2RyaS1kZXZlbC9tc2cxMDI3OTUuaHRtbAo+Pj4KPj4+IGFuZCB0aGUgcGF0Y2ggdG8g ZGUtc3RhZ2UgdGhlIHN5bmMgZnJhbWV3b3JrOgo+Pj4KPj4+IGh0dHBzOi8vd3d3LnNwaW5pY3Mu bmV0L2xpc3RzL2RyaS1kZXZlbC9tc2cxMDI3OTkuaHRtbAo+Pj4KPj4+Cj4+PiBJIGFsc28gaGFj a2VkIHRvZ2V0aGVyIHNvbWUgc3luYyBzdXBwb3J0IGludG8gbW9kZXRlc3QgZm9yIHRlc3Rpbmc6 Cj4+Pgo+Pj4gaHR0cHM6Ly9naXQuY29sbGFib3JhLmNvbS9jZ2l0L3VzZXIvcGFkb3Zhbi9saWJk cm0uZ2l0L2xvZy8/aD1hdG9taWMKPj4+Cj4+Pgo+Pj4gICAgICAgR3VzdGF2bwo+Pj4KPj4+Cj4+ PiBHdXN0YXZvIFBhZG92YW4gKDYpOgo+Pj4gICBkcm0vZmVuY2U6IGFkZCBGRU5DRV9GRCBwcm9w ZXJ0eSB0byBwbGFuZXMKPj4+ICAgZG1hLWJ1Zi9mZW5jZTogYWRkIHN0cnVjdCBmZW5jZV9jb2xs ZWN0aW9uCj4+PiAgIGRtYS1idWYvc3luY19maWxlOiBhZGQgc3luY19maWxlX2ZlbmNlc19nZXQo KQo+Pj4gICBkbWEtYnVmL2ZlbmNlOiBhZGQgZmVuY2VfY29sbGVjdGlvbl9wdXQoKQo+Pj4gICBk bWEtYnVmL2ZlbmNlOiBhZGQgZmVuY2VfY29sbGVjdGlvbl93YWl0KCkKPj4+ICAgZHJtL2ZlbmNl OiBzdXBwb3J0IGZlbmNlX2NvbGxlY3Rpb24gb24gYXRvbWljIGNvbW1pdAo+Pj4KPj4+ICBkcml2 ZXJzL2RtYS1idWYvZmVuY2UuYyAgICAgICAgICAgICB8IDMzICsrKysrKysrKysrKysrKysrKysr KysrKysrKysrKysrKwo+Pj4gIGRyaXZlcnMvZG1hLWJ1Zi9zeW5jX2ZpbGUuYyAgICAgICAgIHwg MzYgKysrKysrKysrKysrKysrKysrKysrKysrKysrKysrKysrKysrCj4+PiAgZHJpdmVycy9ncHUv ZHJtL2RybV9hdG9taWMuYyAgICAgICAgfCAxMyArKysrKysrKysrKysrCj4+PiAgZHJpdmVycy9n cHUvZHJtL2RybV9hdG9taWNfaGVscGVyLmMgfCAxMCArKysrKystLS0tCj4+PiAgZHJpdmVycy9n cHUvZHJtL2RybV9jcnRjLmMgICAgICAgICAgfCAgNyArKysrKysrCj4+PiAgaW5jbHVkZS9kcm0v ZHJtX2NydGMuaCAgICAgICAgICAgICAgfCAgNSArKysrLQo+Pj4gIGluY2x1ZGUvbGludXgvZmVu Y2UuaCAgICAgICAgICAgICAgIHwgMTkgKysrKysrKysrKysrKysrKysrKwo+Pj4gIGluY2x1ZGUv bGludXgvc3luY19maWxlLmggICAgICAgICAgIHwgIDggKysrKysrKysKPj4+ICA4IGZpbGVzIGNo YW5nZWQsIDEyNiBpbnNlcnRpb25zKCspLCA1IGRlbGV0aW9ucygtKQo+Pj4KPj4gX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KPj4gZHJpLWRldmVsIG1haWxp bmcgbGlzdAo+PiBkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCj4+IGh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCj4gCj4gCl9fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmRyaS1kZXZlbCBtYWls aW5nIGxpc3QKZHJpLWRldmVsQGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZy ZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2RyaS1kZXZlbAo= From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752017AbcCXXt2 (ORCPT ); Thu, 24 Mar 2016 19:49:28 -0400 Received: from mailout1.samsung.com ([203.254.224.24]:33020 "EHLO mailout1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750859AbcCXXtY (ORCPT ); Thu, 24 Mar 2016 19:49:24 -0400 MIME-version: 1.0 Content-type: text/plain; charset=utf-8 X-AuditID: cbfee68d-f79646d000001355-46-56f47d01d9d4 Content-transfer-encoding: 8BIT Message-id: <56F47D01.7040508@samsung.com> Date: Fri, 25 Mar 2016 08:49:21 +0900 From: Inki Dae User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 To: Rob Clark Cc: Gustavo Padovan , "dri-devel@lists.freedesktop.org" , Daniel Stone , Daniel Vetter , =?UTF-8?B?QXJ2ZSBIasO4bm5ldsOlZw==?= , Linux Kernel Mailing List , Riley Andrews , Gustavo Padovan , John Harrison Subject: Re: [RFC 0/6] drm/fences: add in-fences to DRM References: <1458758847-21170-1-git-send-email-gustavo@padovan.org> <56F3A2DC.8080507@samsung.com> In-reply-to: X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFjrAIsWRmVeSWpSXmKPExsWyRsSkRJex9kuYwas15hbv/95ns1j48C6z xab+rewWV76+Z7P4tLqV3WLng13sFpOePmCzuLxrDpvF601/GS2eL/zB7MDlsW33NlaPv8+v s3jsuLuE0WPvtwUsHjtn3WX3WLznJZPH/e7jTB47J+1l8vi8SS6AM4rLJiU1J7MstUjfLoEr Y/+5bewFh3Uq7v3oZ2lgbFHpYuTkkBAwkbiybxoThC0mceHeerYuRi4OIYEVjBJt944zdjFy gBVdmuwKUiMksJRRYkdbFojNKyAo8WPyPRaQEmYBeYkjl7IhTHWJKVNyIaY8YJSYvOcWO0S5 lsSOlZcZQWwWAVWJY/POgcXZgOyJK+6zgfSKCkRIdJ+oBAmLCChLrNq6nwXEZhY4xCxxdVYk iC0sYC7R+7CTFWL+akaJ06t6wa7kFAiW+LI8CCQuIdDKIXHl3Q4WiF0CEt8mH2KB+ERWYtMB ZohvJSUOrrjBMoFRbBaSZ2YhPDML4ZkFjMyrGEVTC5ILipPSiwz1ihNzi0vz0vWS83M3MQIj 9vS/Z707GG8fsD7EKMDBqMTD6+D+JUyINbGsuDL3EKMp0A0TmaVEk/OBaSGvJN7Q2MzIwtTE 1NjI3NJMSZxXUepnsJBAemJJanZqakFqUXxRaU5q8SFGJg5OqQbGJM0D55TO1F8pXhlv/dxl xtqgn/J2k358umt5+G+Oe8W2FEettjWve15en/bjaW33KT+3iaduTFfRduyoKJ4i/Ou7zsTX fBZOPvuVJk4I+ZLZlKWjzvFF4GhO6PcYmYfpXw9MCPP0mFi+JvlL30a72E1NRmesru9/35d0 Mm7DWfvLblHWZoutlViKMxINtZiLihMB9vSzbNMCAAA= X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFuplleLIzCtJLcpLzFFi42I5/e+xgC5j7Zcwg4k79Sze/73PZrHw4V1m i039W9ktrnx9z2bxaXUru8XOB7vYLSY9fcBmcXnXHDaL15v+Mlo8X/iD2YHLY9vubawef59f Z/HYcXcJo8febwtYPHbOusvusXjPSyaP+93HmTx2TtrL5PF5k1wAZ1QDo01GamJKapFCal5y fkpmXrqtkndwvHO8qZmBoa6hpYW5kkJeYm6qrZKLT4CuW2YO0KlKCmWJOaVAoYDE4mIlfTtM E0JD3HQtYBojdH1DguB6jAzQQMIaxoz957axFxzWqbj3o5+lgbFFpYuRg0NCwETi0mTXLkZO IFNM4sK99WwgtpDAUkaJHW1ZIDavgKDEj8n3WEDKmQXkJY5cyoYw1SWmTMntYuQCqn7AKDF5 zy12iHItiR0rLzOC2CwCqhLH5p0Di7MB2RNX3GcD6RUViJDoPlEJEhYRUJZYtXU/C4jNLHCI WeLqrEgQW1jAXKL3YScrxPzVjBKnV/UygvRyCgRLfFkeNIFRYBaS42YhHDcL4bgFjMyrGCVS C5ILipPScw3zUsv1ihNzi0vz0vWS83M3MYKTwjOpHYwHd7kfYhTgYFTi4XVw/xImxJpYVlyZ e4hRgoNZSYRXKxooxJuSWFmVWpQfX1Sak1p8iNEU6LuJzFKiyfnAhJVXEm9obGJmZGlkbmhh ZGyuJM77+P+6MCGB9MSS1OzU1ILUIpg+Jg5OqQbGhltFS2TWbqoz77E0X71RdW9WTpfgvk09 k1vWihyYyf3r2d0PHyuLTgRWPDKOufznRUvozP/9SRf5ipzNckL3R0jqRx/QMlQz0K0//c/h 4o7od41z2urcjL/tefn1286zTLzKfCKKR8PvvP7jVC2zWU3q9QbV74trrFc21JjduPBUYXev QnSJEktxRqKhFnNRcSIAjiqcGyADAAA= DLP-Filter: Pass X-MTR: 20000000000000000@CPGS X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 2016년 03월 25일 00:40에 Rob Clark 이(가) 쓴 글: > On Thu, Mar 24, 2016 at 4:18 AM, Inki Dae wrote: >> Hi, >> >> 2016년 03월 24일 03:47에 Gustavo Padovan 이(가) 쓴 글: >>> From: Gustavo Padovan >>> >>> Hi, >>> >>> This is a first proposal to discuss the addition of in-fences support >>> to DRM. It adds a new struct to fence.c to abstract the use of sync_file >>> in DRM drivers. The new struct fence_collection contains a array with all >>> fences that a atomic commit needs to wait on >> >> As I mentioned already like below, >> http://www.spinics.net/lists/dri-devel/msg103225.html >> >> I don't see why Android specific thing is tried to propagate to Linux DRM. In Linux mainline, it has already implicit sync interfaces for DMA devices called dma fence which forces registering a fence obejct to DMABUF through a reservation obejct when a dmabuf object is created. However, Android sync driver creates a new file for a sync object and this would have different point of view. >> >> Is there anyone who can explan why Android specific thing is tried to spread into Linux DRM? Was there any consensus to use Android sync driver - which uses explicit sync interfaces - as Linux standard? >> > > btw, there is already plane_state->fence .. which I don't think has > any users yet, but I start to use it in my patchset that converts > drm/msm to 'struct fence' Yes, Exynos also started it. > > That said, we do need syncpt as the way to expose fences to userspace > for explicit synchronization, but I'm not entirely sure that the It's definitely different case. This tries to add new user-space interfaces to expose fences to user-space. At least, implicit interfaces are embedded into drivers. So I'd like to give you a question. Why exposing fences to user-space is required? To provide easy-to-debug solution to rendering pipleline? To provide merge fence feature? And if we need really to expose fences to user-space and there is really real user, then we have already good candidates, DMA-BUF-IOCTL-SYNC or maybe fcntl system call because we share already DMA buffer between CPU <-> DMA and DMA <-> DMA using DMABUF. For DMA-BUF-IOCTL-SYNC, I think you remember that was what I tried long time ago because you was there. Several years ago, I tried to couple exposing the fences to user-space with cache operation although at that time, I really misleaded the fence machnism. My trying was also for the potential users. Anyway, my opinion is that we could expose the fences hided by DMABUF to user-space using interfaces it exists already around us. And for this, below Chromium solution would also give us some helps, https://chromium.googlesource.com/chromiumos/third_party/kernel/+/chromeos-3.18/drivers/gpu/drm/drm_sync_helper.c And in /driver/dma-buf/, there are DMABUF-centric modules so looks strange sync_file module of Android is placed in that directory - Android sync driver doesn't use really DMABUF but creates new file for their sync fence instead. For implicit sync interfaces for DMA devices, we use DMABUF and for explict sync interfaces for user-space, we use sync_file not DMABUF? That doesn't make sense. I love really Android but I feel as if we try to give a seat available to Android somehow. Thanks, Inki Dae > various drivers ever need to see that (vs just struct fence), at least > on the kms side of things. > > BR, > -R > > >> Thanks, >> Inki Dae >> >>> >>> /** >>> * struct fence_collection - aggregate fences together >>> * @num_fences: number of fence in the collection. >>> * @user_data: user data. >>> * @func: user callback to put user data. >>> * @fences: array of @num_fences fences. >>> */ >>> struct fence_collection { >>> int num_fences; >>> void *user_data; >>> collection_put_func_t func; >>> struct fence *fences[]; >>> }; >>> >>> >>> The fence_collection is allocated and filled by sync_file_fences_get() and >>> atomic_commit helpers can use fence_collection_wait() to wait the fences to >>> signal. >>> >>> These patches depends on the sync ABI rework: >>> >>> https://www.spinics.net/lists/dri-devel/msg102795.html >>> >>> and the patch to de-stage the sync framework: >>> >>> https://www.spinics.net/lists/dri-devel/msg102799.html >>> >>> >>> I also hacked together some sync support into modetest for testing: >>> >>> https://git.collabora.com/cgit/user/padovan/libdrm.git/log/?h=atomic >>> >>> >>> Gustavo >>> >>> >>> Gustavo Padovan (6): >>> drm/fence: add FENCE_FD property to planes >>> dma-buf/fence: add struct fence_collection >>> dma-buf/sync_file: add sync_file_fences_get() >>> dma-buf/fence: add fence_collection_put() >>> dma-buf/fence: add fence_collection_wait() >>> drm/fence: support fence_collection on atomic commit >>> >>> drivers/dma-buf/fence.c | 33 +++++++++++++++++++++++++++++++++ >>> drivers/dma-buf/sync_file.c | 36 ++++++++++++++++++++++++++++++++++++ >>> drivers/gpu/drm/drm_atomic.c | 13 +++++++++++++ >>> drivers/gpu/drm/drm_atomic_helper.c | 10 ++++++---- >>> drivers/gpu/drm/drm_crtc.c | 7 +++++++ >>> include/drm/drm_crtc.h | 5 ++++- >>> include/linux/fence.h | 19 +++++++++++++++++++ >>> include/linux/sync_file.h | 8 ++++++++ >>> 8 files changed, 126 insertions(+), 5 deletions(-) >>> >> _______________________________________________ >> dri-devel mailing list >> dri-devel@lists.freedesktop.org >> https://lists.freedesktop.org/mailman/listinfo/dri-devel > >