From mboxrd@z Thu Jan 1 00:00:00 1970 From: Maarten Lankhorst Subject: Re: [PATCH 00/11] sync framework de-staging: part 1 - clean up Date: Mon, 25 Jan 2016 12:23:52 +0100 Message-ID: <56A605C8.8020703@linux.intel.com> References: <1453380564-3328-1-git-send-email-gustavo@padovan.org> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by gabe.freedesktop.org (Postfix) with ESMTP id 4D50B6E184 for ; Mon, 25 Jan 2016 03:23:58 -0800 (PST) In-Reply-To: <1453380564-3328-1-git-send-email-gustavo@padovan.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Gustavo Padovan , Greg Kroah-Hartman Cc: devel@driverdev.osuosl.org, Daniel Stone , Daniel Vetter , Riley Andrews , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, =?UTF-8?Q?Arve_Hj=c3=b8nnev=c3=a5g?= , Gustavo Padovan , John Harrison List-Id: dri-devel@lists.freedesktop.org SGV5LAoKT3AgMjEtMDEtMTYgb20gMTM6NDkgc2NocmVlZiBHdXN0YXZvIFBhZG92YW46Cj4gRnJv bTogR3VzdGF2byBQYWRvdmFuIDxndXN0YXZvLnBhZG92YW5AY29sbGFib3JhLmNvLnVrPgo+Cj4g SGksCj4KPiBUaGUgZm9sbG93aW5nIHBhdGNoZXMgYXJlIHNvbWUgY2xlYW4gdXBzIG9uIHRoZSBz eW5jIGZyYW1ld29yayBiZWZvcmUKPiB3ZSBzdGFydCB0aGUgYWN0dWFsIGRlLXN0YWdpbmcuIFRo ZSBtYWluIGNoYW5nZXMgaGVyZSBhcmUgdGhlIG1vdmUgb2YKPiBTV19TWU5DX1VTRVIgdG8gZGVi dWdmcy4gUmVtb3ZhbCBvZiBzdHJ1Y3Qgc3luY19wdCBpbiBmYXZvciBvZiBkaXJlY3QKPiB1c2Ug b2Ygc3RydWN0IGZlbmNlLiBBbmQgdGhlIHJlbmFtZSBvZiBzeW5jX2ZlbmNlIHRvIHN5bmNfZmls ZSwgYSBuYW1lCj4gdG8gYmV0dGVyIHJlZmxlY3Qgd2hhdCBpdCBpcyBmb3IsIGEgc3RydWN0IHRo YXQgY29ubmVjdHMgc3RydWN0IGZlbmNlKHMpCj4gdG8gYSBmaWxlLgo+Cj4gUGxlYXNlIHJldmll dy4gVGhhbmtzLgo+Cj4gR3VzdGF2byBQYWRvdmFuICgxMSk6Cj4gICBzdGFnaW5nL2FuZHJvaWQ6 IGZpeCBzeW5jIGZyYW1ld29yayBkb2N1bWVudGF0aW9uCj4gICBzdGFnaW5nL2FuZHJvaWQ6IHN5 bmM6IHJlbW92ZSBpbnRlcmZhY2VzIHRoYXQgYXJlIG5vdCB1c2VkCj4gICBzdGFnaW5nL2FuZHJv aWQ6IHJlbW92ZSBub3QgdXNlZCBzeW5jX3RpbWVsaW5lIG9wcwo+ICAgc3RhZ2luZy9hbmRyb2lk OiBjcmVhdGUgYSAnc3luYycgZGlyIGZvciBkZWJ1Z2ZzIGluZm9ybWF0aW9uCj4gICBzdGFnaW5n L2FuZHJvaWQ6IG1vdmUgU1dfU1lOQ19VU0VSIHRvIGEgZGVidWdmcyBmaWxlCj4gICBzdGFnaW5n L2FuZHJvaWQ6IHJlbmFtZSBzeW5jX2ZlbmNlIHRvIHN5bmNfZmlsZQo+ICAgc3RhZ2luZy9hbmRy b2lkOiByZW5hbWUgJ3N5bmNfcHQnIHRvICdmZW5jZScgaW4gc3RydWN0IHN5bmNfZmVuY2VfY2IK PiAgIHN0YWdpbmcvYW5kcm9pZDogcmVtb3ZlIHN0cnVjdCBzeW5jX3B0Cj4gICBzdGFnaW5nL2Fu ZHJvaWQ6IHJlbW92ZSB1bnVzZWQgdmFyIGZyb20gc3luY190aW1lbGluZV9zaWduYWwoKQo+ICAg c3RhZ2luZy9hbmRyb2lkOiByZW1vdmUgcG9pbnRsZXNzIHN5bmNfdGltZWxpbmVfc2lnbmFsIGF0 IGRlc3Ryb3kKPiAgICAgcGhhc2UKPiAgIHN0YWdpbmcvYW5kcm9pZDogcmVtb3ZlIHN5bmNfZmVu Y2VfY3JlYXRlX2RtYSgpCj4KRm9yIHdob2xlIHNlcmllczoKUmV2aWV3ZWQtYnk6IE1hYXJ0ZW4g TGFua2hvcnN0IDxtYWFydGVuLmxhbmtob3JzdEBsaW51eC5pbnRlbC5jb20+CgpSZW5hbWluZyBz eW5jX2ZlbmNlIHRvIHN5bmNfZmlsZSBtYWtlcyBzZW5zZSwgaXQgc2hvdWxkIGhvcGVmdWxseSBy ZWR1Y2UgY29uZnVzaW9uLgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fXwpkcmktZGV2ZWwgbWFpbGluZyBsaXN0CmRyaS1kZXZlbEBsaXN0cy5mcmVlZGVza3Rv cC5vcmcKaHR0cDovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2RyaS1k ZXZlbAo= From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756916AbcAYLYE (ORCPT ); Mon, 25 Jan 2016 06:24:04 -0500 Received: from mga09.intel.com ([134.134.136.24]:43842 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756691AbcAYLX7 (ORCPT ); Mon, 25 Jan 2016 06:23:59 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.22,344,1449561600"; d="scan'208";a="888685440" Subject: Re: [PATCH 00/11] sync framework de-staging: part 1 - clean up To: Gustavo Padovan , Greg Kroah-Hartman References: <1453380564-3328-1-git-send-email-gustavo@padovan.org> Cc: linux-kernel@vger.kernel.org, devel@driverdev.osuosl.org, dri-devel@lists.freedesktop.org, Daniel Stone , =?UTF-8?Q?Arve_Hj=c3=b8nnev=c3=a5g?= , Riley Andrews , Daniel Vetter , Rob Clark , Greg Hackmann , John Harrison , Gustavo Padovan From: Maarten Lankhorst Message-ID: <56A605C8.8020703@linux.intel.com> Date: Mon, 25 Jan 2016 12:23:52 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.1.0 MIME-Version: 1.0 In-Reply-To: <1453380564-3328-1-git-send-email-gustavo@padovan.org> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hey, Op 21-01-16 om 13:49 schreef Gustavo Padovan: > From: Gustavo Padovan > > Hi, > > The following patches are some clean ups on the sync framework before > we start the actual de-staging. The main changes here are the move of > SW_SYNC_USER to debugfs. Removal of struct sync_pt in favor of direct > use of struct fence. And the rename of sync_fence to sync_file, a name > to better reflect what it is for, a struct that connects struct fence(s) > to a file. > > Please review. Thanks. > > Gustavo Padovan (11): > staging/android: fix sync framework documentation > staging/android: sync: remove interfaces that are not used > staging/android: remove not used sync_timeline ops > staging/android: create a 'sync' dir for debugfs information > staging/android: move SW_SYNC_USER to a debugfs file > staging/android: rename sync_fence to sync_file > staging/android: rename 'sync_pt' to 'fence' in struct sync_fence_cb > staging/android: remove struct sync_pt > staging/android: remove unused var from sync_timeline_signal() > staging/android: remove pointless sync_timeline_signal at destroy > phase > staging/android: remove sync_fence_create_dma() > For whole series: Reviewed-by: Maarten Lankhorst Renaming sync_fence to sync_file makes sense, it should hopefully reduce confusion.