Re: [Piglit] [PATCH] arb_shader_atomic_counter_ops: declare counters to fix undefined behaviour.

2017-10-10 Thread Ian Romanick
Oops. Reviewed-by: Ian Romanick On 10/10/2017 05:18 PM, Dave Airlie wrote: > From: Dave Airlie > > It's undefined (including crashing) behaviour to use atomic counters > without having bound enough storage for them. This fixes this test > to bind the storage for the counters it uses. > > Sign

Re: [Piglit] [PATCH] arb_shader_atomic_counter_ops: declare counters to fix undefined behaviour.

2017-10-10 Thread Timothy Arceri
Reviewed-by: Timothy Arceri On 11/10/17 11:18, Dave Airlie wrote: From: Dave Airlie It's undefined (including crashing) behaviour to use atomic counters without having bound enough storage for them. This fixes this test to bind the storage for the counters it uses. Signed-off-by: Dave Airlie

[Piglit] [PATCH] arb_shader_atomic_counter_ops: declare counters to fix undefined behaviour.

2017-10-10 Thread Dave Airlie
From: Dave Airlie It's undefined (including crashing) behaviour to use atomic counters without having bound enough storage for them. This fixes this test to bind the storage for the counters it uses. Signed-off-by: Dave Airlie --- .../arb_shader_atomic_counter_ops/execution/all_touch_test.shad