MIME-Version: 1.0
In-Reply-To: <CAPj87rOZMLs3oQHm1vWaA3i_uLXQ59ZwV3Nkp5ArPp5NVGShEA@mail.gmail.com>
References: <1458758847-21170-1-git-send-email-gustavo@padovan.org>
 <56F3A2DC.8080507@samsung.com> <CAF6AEGu13bahQWK5yBDnsUoKdmLxJT=fj=S-VNWKjjNqLG2gEg@mail.gmail.com>
 <56F47D01.7040508@samsung.com> <CAF6AEGtesj5hDQtBQgTAJFnMi0euY+Xm+wbUupMc660VPVtmvg@mail.gmail.com>
 <CAPj87rMpNOwbvv1+jfKfE=Bte7S3wpH6WpZUpHv6uvKQ5DeY=Q@mail.gmail.com>
 <56F88828.5050304@samsung.com> <CAPj87rPJj0E_WvKCfGbANoXT9w6ns2k3mnNWvo5LZFfqtN1RGQ@mail.gmail.com>
 <56F9E613.1030902@samsung.com> <CAF6AEGt3OOfvtSVtueNKdVFo5Pnm=mL8K+dPPJZ8BjYXSi+0ww@mail.gmail.com>
 <56FCD5A3.4040700@samsung.com> <CAPj87rPjoBiF_TuBERM4sqL6sb_73CMirRav+JCYFiWzpp+FHw@mail.gmail.com>
 <56FCF67A.8090109@samsung.com> <CAPj87rOZMLs3oQHm1vWaA3i_uLXQ59ZwV3Nkp5ArPp5NVGShEA@mail.gmail.com>
From: Inki Dae <daeinki@gmail.com>
Date: Thu, 31 Mar 2016 20:26:45 +0900
Message-ID: <CAAQKjZOh-DxraZYo_uht+jALY9F8V6wkHzQ76Tn_L4btbdd4dA@mail.gmail.com>
Subject: Re: [RFC 0/6] drm/fences: add in-fences to DRM
To: Daniel Stone <daniel@fooishbar.org>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>,
        "dri-devel@lists.freedesktop.org" <dri-devel@lists.freedesktop.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Riley Andrews <riandrews@android.com>,
        =?UTF-8?B?QXJ2ZSBIasO4bm5ldsOlZw==?= <arve@android.com>,
        Gustavo Padovan <gustavo.padovan@collabora.co.uk>,
        John Harrison <John.C.Harrison@intel.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8BIT
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1896
Lines: 40

Hi Daniel,

2016-03-31 19:56 GMT+09:00 Daniel Stone <daniel@fooishbar.org>:
> Hi Inki,
>
> On 31 March 2016 at 11:05, Inki Dae <inki.dae@samsung.com> wrote:
>> 2016년 03월 31일 18:35에 Daniel Stone 이(가) 쓴 글:
>>> On 31 March 2016 at 08:45, Inki Dae <inki.dae@samsung.com> wrote:
>>>> As of now, it seems that this wouldn't be optional but mandatory if explicit fence support is added to the atomic helper framework. This would definitely be duplication and it seems not clear enough even if one of them is just skipped in runtime.
>>>
>>> Drivers would have to opt in to explicit fencing support, and part of
>>> that would be ensuring that the driver does not wait on implicit
>>> fences when the user has requested explicit fencing be used.
>>>
>>
>> Then, existing drivers would need additional works for explicit fencing support. This wouldn't be really what the drivers have to but should be handled with this patch series because this would affect exising device drivers which use implicit fencing.
>
> Well, yes. Anyone implementing their own atomic commit would need to
> ensure that the commit works properly for fences. The helpers could
> also add it, but the helpers are not mandatory, and you are not
> required to use every part of the helper to use one part of the
> helper. There is no magic wand you can wave that instantly makes it
> work for every driver

I meant there are already several DRM drivers which work properly for
implicit fence. So if atomic helper framework of DRM core is
considered only for the explicit fence, then fencing operation would
affect the existing DRM drivers. So I hope this trying could consider
existing implicit fence users.

Thanks,
Inki Dae
.
>
> Cheers,
> Daniel
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel