[Mesa-dev] [PATCH 00/10] gallium: normalize CONST file accesses to 2D - 2. try without image for the list
Nicolai Hähnle
nhaehnle at gmail.com
Fri Aug 25 07:13:13 UTC 2017
Hi Dieter,
sorry for the churn -- do these issues also occur with the latest
addition to https://cgit.freedesktop.org/~nh/mesa/log/?h=tgsi-const-2d?
Cheers,
Nicolai
On 25.08.2017 08:19, Dieter Nützel wrote:
> Am 25.08.2017 07:38, schrieb Dieter Nützel:
>> Am 23.08.2017 18:41, schrieb Nicolai Hähnle:
>>> Hi all,
>>>
>>> Following the discussion on Timothy's std430 packing series, here's
>>> a quick proposal to just always use 2D accesses to the CONST file
>>> in TGSI.
>>>
>>> The first patch should be sufficient for all drivers to accept
>>> those 2D accesses. It seems that most older drivers simply ignore
>>> the dimension, and newer ones should handle it directly.
>>>
>>> Subsequent patches modify the producers of TGSI to always use 2D
>>> constant references. This is mostly done by changing ureg.
>>>
>>> Finally, the last patch adds an assertion to radeonsi to make
>>> sure all constant references are really 2D. It has survived my
>>> very superficial initial testing.
>
> Addendum:
>
> glmark2 (2017.07) threw assertion.
> System hang followed so sadly only mobile screenshot appended.
>
> Goodnight! ;-)
>
> Dieter
>
>>> What needs to be tested is:
>>> - some more drivers
>>> - Nine
>>
>> Sorry Nicolai,
>>
>> but Nine corruption with Wine (LS2017 / FarmingSimulator2017) on
>> RX580, here.
>>
>> After KDE relogin partially window/screen corruption (window boarder
>> pixel flickering).
>>
>> Dieter
>>
>>
>>> - TGSI-to-NIR
>>>
>>> You can find the series here:
>>> https://cgit.freedesktop.org/~nh/mesa/log/?h=tgsi-const-2d
>>>
>>> Please comment/review!
>>> Thanks,
>>> Nicolai
>>> --
>>> src/gallium/auxiliary/hud/hud_context.c | 8 +-
>>> src/gallium/auxiliary/nir/tgsi_to_nir.c | 2 +-
>>> src/gallium/auxiliary/postprocess/pp_mlaa.h | 20 +--
>>> src/gallium/auxiliary/tgsi/tgsi_ureg.c | 22 +--
>>> src/gallium/auxiliary/util/u_tests.c | 4 +-
>>> src/gallium/docs/source/screen.rst | 11 +-
>>> src/gallium/drivers/radeon/r600_query.c | 36 ++--
>>> src/gallium/drivers/radeonsi/si_shader.c | 1 +
>>> src/gallium/state_trackers/nine/nine_ff.c | 2 +-
>>> .../state_trackers/nine/nine_shader.c | 10 +-
>>> .../tests/graw/fragment-shader/frag-cb-1d.sh | 8 +-
>>> .../tests/graw/vertex-shader/vert-cb-1d.sh | 8 +-
>>> src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 153 +++++++++--------
>>> 13 files changed, 136 insertions(+), 149 deletions(-)
>>>
>>> _______________________________________________
>>> mesa-dev mailing list
>>> mesa-dev at lists.freedesktop.org
>>> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>> _______________________________________________
>> mesa-dev mailing list
>> mesa-dev at lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
--
Lerne, wie die Welt wirklich ist,
Aber vergiss niemals, wie sie sein sollte.
More information about the mesa-dev
mailing list