[Mesa-dev] [PATCH v6.2] egl: Allow creation of per surface out fence

[Tomasz Figa]

On Thu, Aug 31, 2017 at 2:18 AM, Marathe, Yogesh
<yogesh.marathe at intel.com> wrote:
>> -----Original Message-----
>> From: mesa-dev [mailto:mesa-dev-bounces at lists.freedesktop.org] On Behalf
>> Of Emil Velikov
>> Sent: Wednesday, August 30, 2017 9:44 PM
>> To: Marathe, Yogesh <yogesh.marathe at intel.com>
>> Cc: Gao, Shuo <shuo.gao at intel.com>; Liu, Zhiquan <zhiquan.liu at intel.com>;
>> daniels at collabora.com; nicolai.haehnle at amd.com; Antognolli, Rafael
>> <rafael.antognolli at intel.com>; eric at engestrom.ch; Wu, Zhongmin
>> <zhongmin.wu at intel.com>; tfiga at chromium.org; kenneth at whitecape.org;
>> Kondapally, Kalyan <kalyan.kondapally at intel.com>; fernetmenta at online.de;
>> mesa-dev at lists.freedesktop.org; tarceri at itsqueeze.com;
>> varad.gautam at collabora.com
>> Subject: Re: [Mesa-dev] [PATCH v6.2] egl: Allow creation of per surface out
>> fence
>>
>> On 30 August 2017 at 15:39, Marathe, Yogesh <yogesh.marathe at intel.com>
>> wrote:
>>
>> >
>> > Thank you, Tomasz and all involved for the help and guidance.
>> >
>> Our excitement was short lived - see commit
>> 8c9df0daf20206fafb7df77b1edcbc41b8e91372.
>>
>> Seems the patch was not run through the Intel CI, though I'm should not have
>> assumed that you're aware of if.
>> Please get in touch with Mark Janes (Cc'd here, janesma on IRC). He can set you
>> up and/or run a branch for you.
>>
>
> No problem. I will contact Mark.
>
> Primarily looks like platform / kernel version issue. intel_get_boolean() for
> I915_PARAM_HAS_EXEC_FENCE is false, but I see following in
> i915_drv.c:915_getparam in kernel, no clue why that would come false
> in UMD.
>
> ...
>         case I915_PARAM_HAS_EXEC_FENCE:
>                 /* For the time being all of these are always true;
>                  * if some supported hardware does not have one of these
>                  * features this value needs to be provided from
>                  * INTEL_INFO(), a feature macro, or similar.
>                  */
>                 value = 1;
>                 break;
> ...

Which kernel are you looking at? Remember that not everyone uses
current upstream master. There is a number of upstream stable releases
and downstream forks. Grepping for I915_PARAM_HAS_EXEC_FENCE on
http://elixir.free-electrons.com, shows that it was only added in
Linux 4.12.

Best regards,
Tomasz