Re: [Mesa-dev] [PATCH] nv50: allocate more offset space for occlusion queries

2015-04-04 Thread Samuel Pitoiset
Reviewed-by: Samuel Pitoiset But please, fix the comment in the nv50_query struct, now it is "/* base + i * 32 */". On 04/04/2015 06:00 AM, Ilia Mirkin wrote: Commit 1a170980a09 started writing to q->data[4]/[5] but kept the per-query space at 16, which meant that in some cases we would writ

[Mesa-dev] [PATCH] nv50: allocate more offset space for occlusion queries

2015-04-03 Thread Ilia Mirkin
Commit 1a170980a09 started writing to q->data[4]/[5] but kept the per-query space at 16, which meant that in some cases we would write past the end of the buffer. Rotate by 32, like nvc0 does. Signed-off-by: Ilia Mirkin Tested-by: Nick Tenney Cc: "10.4 10.5" --- src/gallium/drivers/nouveau/nv5