Re: [Mesa-dev] [PATCH] aubinator: implement a rolling window of programs

2017-09-01 Thread Lionel Landwerlin
On 01/09/17 15:37, Eric Engestrom wrote: >8 With that, you can keep the assert (although it would only fire it someone were to manually increment `num_programs`) and the MIN() becomes unnecessary. Actually I just realized the MIN() doesn't work once num_programs goes back to 0... Th

Re: [Mesa-dev] [PATCH] aubinator: implement a rolling window of programs

2017-09-01 Thread Eric Engestrom
On Friday, 2017-09-01 15:45:37 +0100, Lionel Landwerlin wrote: > On 01/09/17 15:37, Eric Engestrom wrote: > > On Friday, 2017-09-01 10:38:33 +0100, Lionel Landwerlin wrote: > > > If we have more programs than what we can store, > > > aubinator_error_decode will assert. Instead let's have a rolling

Re: [Mesa-dev] [PATCH] aubinator: implement a rolling window of programs

2017-09-01 Thread Lionel Landwerlin
On 01/09/17 15:37, Eric Engestrom wrote: On Friday, 2017-09-01 10:38:33 +0100, Lionel Landwerlin wrote: If we have more programs than what we can store, aubinator_error_decode will assert. Instead let's have a rolling window of programs. Signed-off-by: Lionel Landwerlin --- src/intel/tools/a

Re: [Mesa-dev] [PATCH] aubinator: implement a rolling window of programs

2017-09-01 Thread Eric Engestrom
On Friday, 2017-09-01 10:38:33 +0100, Lionel Landwerlin wrote: > If we have more programs than what we can store, > aubinator_error_decode will assert. Instead let's have a rolling > window of programs. > > Signed-off-by: Lionel Landwerlin > --- > src/intel/tools/aubinator_error_decode.c | 16 ++

[Mesa-dev] [PATCH] aubinator: implement a rolling window of programs

2017-09-01 Thread Lionel Landwerlin
If we have more programs than what we can store, aubinator_error_decode will assert. Instead let's have a rolling window of programs. Signed-off-by: Lionel Landwerlin --- src/intel/tools/aubinator_error_decode.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a