[Mesa-dev] [PATCH v2 1/3] st/glsl_to_tgsi: move nir detection earlier - bisected

[Timothy Arceri]

On 07/02/18 19:17, Dieter Nützel wrote:
> Am 06.02.2018 00:23, schrieb Timothy Arceri:
>> On 05/02/18 15:04, Dieter Nützel wrote:
>>> Am 02.02.2018 10:24, schrieb Timothy Arceri:
>>>> On 02/02/18 19:26, Dieter Nützel wrote:
>>>>> Hello Tim,
>>>>>
>>>>> _this_ version brake UH, UV, mpv, blender 2.79 (some test files not 
>>>>> all).
>>>>> Must be something with the cache file(s).
>>>>
>>>> The cache currently needs to be deleted when switching between nir and
>>>> tgsi. I'm not sure it I should try to avoid this or not ... I guess it
>>>> will probably save some bug reports so I'll try send a follow up
>>>> patch.
>>>
>>> Hi Tim,
>>>
>>> it is NOT your fault.
>>> I tracked it down to Marek's commit commit 
>>> be973ed21f6e456ebd753f26a99151d9ea6e765c
>>
>> This should fix things for now:
>>
>> https://patchwork.freedesktop.org/patch/202759/
> 
> Apart that it landed already:
> 
> Tested-by: Dieter Nützel <Dieter at nuetzel-hh.de>
> 
> But I get some severe hangs with current git code on Polaris 20.
> Steam (Linux only, NOT  Wine), UH and UV hang full system.
> I could remotely log in but nothing in the logs.
> Have to bisect, again...

I'm seeing deadlocks in piglit caused by the following patch, could be 
what you are seeing.

commit 6a651b6b77b68db71a027c826abccc843ace88ef (HEAD)
Author: Tapani Pälli <tapani.palli at intel.com>
Date:   Mon Jan 22 11:55:06 2018 +0200

     disk cache: initialize cache path and index only when used

     This patch makes disk_cache initialize path and index lazily so
     that we can utilize disk_cache without a path using callback
     functionality introduced by next patch.

     v2: unmap mmap and destroy queue only if index_mmap exists

     Signed-off-by: Tapani Pälli <tapani.palli at intel.com>
     Reviewed-by: Jordan Justen <jordan.l.justen at intel.com>
     Reviewed-by: Emil Velikov <emil.velikov at collabora.com>


> 
> Sadly, I've a bad cold.
> 
> Thanks,
> Dieter