[Mesa-dev] freedreno: 'Unhandled NIR tex src type: 11' on A3XX

[Jonathan Marek]

On 6/10/19 10:00 PM, Brian Masney wrote:
> On Mon, Jun 10, 2019 at 09:53:25PM -0400, Jonathan Marek wrote:
>>>>> This error doesn't happen on X11 using the mesa master branch. Instead,
>>>>> I get the following error on that branch:
>>>>>
>>>>> ../src/gallium/drivers/freedreno/freedreno_batch.c:424:fd_batch_add_dep: Assertion `!batch_depends_on(dep, batch)' failed.
>>>>>
>>>>> Full disclosure though: I rebuilt the mesa package using the
>>>>> postmarketOS packaging yesterday and it includes a few extra patches for
>>>>> musl libc.
>>>>>
>>>>> https://gitlab.com/postmarketOS/pmaports/tree/master/temp/mesa
>>>>
>>>>
>>>> I don't see anything obvious in those patches that would be related..
>>>> but I suspect this type of error is going to be timing related.
>>>> (Which could ofc be due to musl or something else)
>>>>
>>>> but a bit surprised debug_assert() is enabled in debug builds.. it
>>>> would probably be a "harmless" situation if asserts were not enabled.
>>>>
>>>> (note that I do most of my testing with debug builds with asserts
>>>> enabled.. this is the type of thing that I want to see and fix.. but
>>>> probably shouldn't matter to end users)
>>>
>>> I recompiled the master branch of mesa in pmOS with '-Db_ndebug=true'
>>> and X11 is now working properly on the Nexus 5. glxgears averages about
>>> 59.5 FPS. I'll add a bug report with pmOS to have them add that flag to
>>> their mesa build. Fedora added that flag to their builds:
>>> https://bugzilla.redhat.com/show_bug.cgi?id=1692426
>>>
>>> 19.1.0-rc5 still doesn't work for me due to the original error.
>>>
>>> Brian
>>>
>>
>> You probably want '--buildtype=release' instead of '-Db_ndebug=true'
> 
> According to:
> https://gitlab.freedesktop.org/mesa/mesa/blob/master/docs/meson.html#L321
> 
>      -Db_ndebug - This option controls assertions in meson projects. When
>      set to false (the default) assertions are enabled, when set to true
>      they are disabled. This is unrelated to the buildtype; setting the
>      latter to release will not turn off assertions.
> 
> Brian
> 

I always thought release == no assertions, I guess meson has different 
ideas. You will want '--buildtype=release' anyway for optimizations, etc.