Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751034AbcCXOkD (ORCPT ); Thu, 24 Mar 2016 10:40:03 -0400 Received: from mail-yw0-f176.google.com ([209.85.161.176]:35231 "EHLO mail-yw0-f176.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752012AbcCXOjg convert rfc822-to-8bit (ORCPT ); Thu, 24 Mar 2016 10:39:36 -0400 Date: Thu, 24 Mar 2016 11:39:13 -0300 From: Gustavo Padovan To: Inki Dae Cc: dri-devel@lists.freedesktop.org, Daniel Stone , Daniel Vetter , Arve =?iso-8859-1?B?SGr4bm5lduVn?= , linux-kernel@vger.kernel.org, Riley Andrews , Gustavo Padovan , John Harrison Subject: Re: [RFC 0/6] drm/fences: add in-fences to DRM Message-ID: <20160324143913.GB4781@joana> Mail-Followup-To: Gustavo Padovan , Inki Dae , dri-devel@lists.freedesktop.org, Daniel Stone , Daniel Vetter , Arve =?iso-8859-1?B?SGr4bm5lduVn?= , linux-kernel@vger.kernel.org, Riley Andrews , Gustavo Padovan , John Harrison 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-Disposition: inline Content-Transfer-Encoding: 8BIT In-Reply-To: <56F3A2DC.8080507@samsung.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1630 Lines: 33 Hi Inki, 2016-03-24 Inki Dae : > 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? Because we want explicit fencing as the Linux standard in the future to be able to do smart scheduling, e.g., send async jobs to the gpu and at the same time send async atomic commits with sync_file fd attached so they can wait the GPU to finish and we don't block in userspace anymore, quite similar to what Android does. This would still use dma-buf fences in the driver level, but it has a lot more advantages than implicit fencing. Gustavo