Re: [Mesa-dev] [PATCH] mesa: count uniform against storage when its bindless

2017-08-17 Thread Timothy Arceri
On 18/08/17 11:00, Kenneth Graunke wrote: On Thursday, August 17, 2017 4:41:52 PM PDT Timothy Arceri wrote: On 18/08/17 08:30, Kenneth Graunke wrote: On Tuesday, August 15, 2017 3:42:29 AM PDT Timothy Arceri wrote: Gallium drivers use this code path so we need to account for bindless after

Re: [Mesa-dev] [PATCH] mesa: count uniform against storage when its bindless

2017-08-17 Thread Kenneth Graunke
On Thursday, August 17, 2017 4:41:52 PM PDT Timothy Arceri wrote: > On 18/08/17 08:30, Kenneth Graunke wrote: > > On Tuesday, August 15, 2017 3:42:29 AM PDT Timothy Arceri wrote: > >> Gallium drivers use this code path so we need to account for > >> bindless after all. > > > > Why do Gallium

Re: [Mesa-dev] [PATCH] mesa: count uniform against storage when its bindless

2017-08-17 Thread Timothy Arceri
On 18/08/17 08:30, Kenneth Graunke wrote: On Tuesday, August 15, 2017 3:42:29 AM PDT Timothy Arceri wrote: Gallium drivers use this code path so we need to account for bindless after all. Why do Gallium drivers use ir_to_mesa? That seems like a misfeature. Because Gallium drivers still use

Re: [Mesa-dev] [PATCH] mesa: count uniform against storage when its bindless

2017-08-17 Thread Kenneth Graunke
On Tuesday, August 15, 2017 3:42:29 AM PDT Timothy Arceri wrote: > Gallium drivers use this code path so we need to account for > bindless after all. Why do Gallium drivers use ir_to_mesa? That seems like a misfeature. i965 stopped using it years ago. --Ken signature.asc Description: This is

Re: [Mesa-dev] [PATCH] mesa: count uniform against storage when its bindless

2017-08-15 Thread Timothy Arceri
On 15/08/17 23:07, Marek Olšák wrote: Reviewed-by: Marek Olšák Marek Pushed, Thanks. ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH] mesa: count uniform against storage when its bindless

2017-08-15 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Tue, Aug 15, 2017 at 12:42 PM, Timothy Arceri wrote: > Gallium drivers use this code path so we need to account for > bindless after all. > > Fixes: 365d34540f33 ("mesa: correctly calculate the storage offset for