Re: [Mesa-dev] [PATCH 00/18] anv: Rework the allocation data structures

2017-05-05 Thread Jason Ekstrand
Sorry about that. It only fails if you *don't* build against valgrind headers. Since my dev machine and our CI are both set up with valgrind support, I didn't catch it. I've sent a patch to the list to fix this. --Jason On Fri, May 5, 2017 at 5:36 AM, Mike Lothian wrote: > I'm seeing the fol

Re: [Mesa-dev] [PATCH 00/18] anv: Rework the allocation data structures

2017-05-05 Thread Mike Lothian
I'm seeing the following failure with GCC 7.1: /var/tmp/portage/media-libs/mesa-/work/mesa-/src/intel/vulkan/anv_allocator.c: In function ‘anv_state_stream_alloc’: /var/tmp/portage/media-libs/mesa-/work/mesa-/src/intel/vulkan/anv_allocator.c:930:28: error: ‘struct anv_state_stream_

Re: [Mesa-dev] [PATCH 00/18] anv: Rework the allocation data structures

2017-05-04 Thread Juan A. Suarez Romero
On Tue, 2017-05-02 at 06:44 -0700, Jason Ekstrand wrote: > Juan, > > Were you planning to review the rest of the series? Just wondering. > Sorry, it took a bit more to review the remaining patches. Now it is done. J.A. > --Jason > > > On April 26, 2017 7:35:29 AM Jason Ekstrand

Re: [Mesa-dev] [PATCH 00/18] anv: Rework the allocation data structures

2017-05-02 Thread Jason Ekstrand
Juan, Were you planning to review the rest of the series? Just wondering. --Jason On April 26, 2017 7:35:29 AM Jason Ekstrand wrote: This absurdly long series does something fairly simple: It pulls the block_pool into the state_pool and makes the state pool capable of allocating states la

[Mesa-dev] [PATCH 00/18] anv: Rework the allocation data structures

2017-04-26 Thread Jason Ekstrand
This absurdly long series does something fairly simple: It pulls the block_pool into the state_pool and makes the state pool capable of allocating states larger than the block size. Unfortunately, there was no good way to do it in less than 18 patches. :-( Cc: Juan A. Suarez Romero Jason Ekstr