[Mesa-dev] [PATCH 3/3] radeonsi/gfx9: implement primitive binning

2017-08-31 Thread Marek Olšák
From: Marek Olšák This increases performance, but it was tuned for Raven, not Vega. We don't know yet how Vega will perform, hopefully not worse. --- src/gallium/drivers/radeon/r600_pipe_common.c | 2 + src/gallium/drivers/radeon/r600_pipe_common.h | 2 + src/gallium/drivers/radeonsi/Mak

Re: [Mesa-dev] [PATCH 3/3] radeonsi/gfx9: implement primitive binning

2017-09-04 Thread Nicolai Hähnle
On 01.09.2017 02:57, Marek Olšák wrote: From: Marek Olšák This increases performance, but it was tuned for Raven, not Vega. We don't know yet how Vega will perform, hopefully not worse. --- src/gallium/drivers/radeon/r600_pipe_common.c | 2 + src/gallium/drivers/radeon/r600_pipe_common.h

Re: [Mesa-dev] [PATCH 3/3] radeonsi/gfx9: implement primitive binning

2017-09-04 Thread Marek Olšák
On Mon, Sep 4, 2017 at 3:56 PM, Nicolai Hähnle wrote: > On 01.09.2017 02:57, Marek Olšák wrote: >> >> From: Marek Olšák >> >> This increases performance, but it was tuned for Raven, not Vega. >> We don't know yet how Vega will perform, hopefully not worse. >> --- >> src/gallium/drivers/radeon/r

Re: [Mesa-dev] [PATCH 3/3] radeonsi/gfx9: implement primitive binning

2017-09-04 Thread Marek Olšák
I actually made a mistake while porting the code. All UINT_MAX occurences should stay, and UINT_MAX should be the terminator, so I'm adding this: diff --git a/src/gallium/drivers/radeonsi/si_state_binning.c b/src/gallium/drivers/radeonsi/si_state_binning.c index 56bcdc8..d75e86e 100644 --- a/src/g

Re: [Mesa-dev] [PATCH 3/3] radeonsi/gfx9: implement primitive binning

2017-09-04 Thread Nicolai Hähnle
On 04.09.2017 20:04, Marek Olšák wrote: I actually made a mistake while porting the code. All UINT_MAX occurences should stay, and UINT_MAX should be the terminator, so I'm adding this: Does it really make a difference? Let's take the 1 RB / 1 SE case. With sum == 16, both loops run until i ==