Re: [Mesa-dev] [PATCH v2] i965: do not emit empty surface state

2018-08-14 Thread Erik Faye-Lund
Quoting the original patch: ---8<--- This fixes an assert that triggers for me when running dEQP-GLES31.functional.shaders.opaque_type_indexing.ssbo.const_literal_vertex on top of a debug-build of mesa on top of i965. Since setting up a virgl setup is rather convoluted, here's a piglit shader-tes

Re: [Mesa-dev] [PATCH v2] i965: do not emit empty surface state

2018-08-14 Thread Lionel Landwerlin
Hey Erik, Out of curiosity, what app/test managed to run into this assert? We could have some test added :) Thanks, - Lionel On 08/08/18 09:34, Lionel Landwerlin wrote: On 07/08/18 20:31, Erik Faye-Lund wrote: If called with an empty size, brw_emit_buffer_surface_state asserts. We already ha

Re: [Mesa-dev] [PATCH v2] i965: do not emit empty surface state

2018-08-08 Thread Lionel Landwerlin
On 07/08/18 20:31, Erik Faye-Lund wrote: If called with an empty size, brw_emit_buffer_surface_state asserts. We already have a dedicated helper for uploading nothing, so let's use that instead. Signed-off-by: Erik Faye-Lund Looks good to me. Maybe Ken can confirm? Reviewed-by: Lionel Landwe

[Mesa-dev] [PATCH v2] i965: do not emit empty surface state

2018-08-07 Thread Erik Faye-Lund
If called with an empty size, brw_emit_buffer_surface_state asserts. We already have a dedicated helper for uploading nothing, so let's use that instead. Signed-off-by: Erik Faye-Lund --- Here's an updated patch. v2: call emit_null_surface_state to ensure out_offset is initialized properly.