Re: [Mesa-dev] [PATCH] gallium: add pipe_grid_info::partial_block

2019-01-08 Thread Marek Olšák
On Tue, Jan 8, 2019, 7:55 PM Ilia Mirkin On Tue, Jan 8, 2019 at 7:26 PM Marek Olšák wrote: > > > > On Tue, Jan 8, 2019 at 7:18 PM Ilia Mirkin wrote: > >> > >> On Tue, Jan 8, 2019 at 6:21 PM Marek Olšák wrote: > >> > > >> > On Tue, Jan 8, 2019 at 5:25 PM Ilia Mirkin > wrote: > >> >> > >> >>

Re: [Mesa-dev] [PATCH] gallium: add pipe_grid_info::partial_block

2019-01-08 Thread Ilia Mirkin
On Tue, Jan 8, 2019 at 7:26 PM Marek Olšák wrote: > > On Tue, Jan 8, 2019 at 7:18 PM Ilia Mirkin wrote: >> >> On Tue, Jan 8, 2019 at 6:21 PM Marek Olšák wrote: >> > >> > On Tue, Jan 8, 2019 at 5:25 PM Ilia Mirkin wrote: >> >> >> >> Why does this need to be in p_state? And who is responsible

Re: [Mesa-dev] [PATCH] gallium: add pipe_grid_info::partial_block

2019-01-08 Thread Marek Olšák
On Tue, Jan 8, 2019 at 7:18 PM Ilia Mirkin wrote: > On Tue, Jan 8, 2019 at 6:21 PM Marek Olšák wrote: > > > > On Tue, Jan 8, 2019 at 5:25 PM Ilia Mirkin wrote: > >> > >> Why does this need to be in p_state? And who is responsible for > >> setting it (and how will it be set)? > > > > > > Oh

Re: [Mesa-dev] [PATCH] gallium: add pipe_grid_info::partial_block

2019-01-08 Thread Ilia Mirkin
On Tue, Jan 8, 2019 at 6:21 PM Marek Olšák wrote: > > On Tue, Jan 8, 2019 at 5:25 PM Ilia Mirkin wrote: >> >> Why does this need to be in p_state? And who is responsible for >> setting it (and how will it be set)? > > > Oh right, there is a way to get it out of p_state.h if needed. > > It should

Re: [Mesa-dev] [PATCH] gallium: add pipe_grid_info::partial_block

2019-01-08 Thread Marek Olšák
On Tue, Jan 8, 2019 at 5:25 PM Ilia Mirkin wrote: > Why does this need to be in p_state? And who is responsible for > setting it (and how will it be set)? > Oh right, there is a way to get it out of p_state.h if needed. It should be set to 0 by default. If your thread block is 8x8x1, but you

Re: [Mesa-dev] [PATCH] gallium: add pipe_grid_info::partial_block

2019-01-08 Thread Ilia Mirkin
Why does this need to be in p_state? And who is responsible for setting it (and how will it be set)? On Tue, Jan 8, 2019 at 2:47 PM Jiang, Sonny wrote: > > and add radeonsi support. This will be used by radeonsi internally. > > Signed-off-by: Sonny Jiang > --- >

[Mesa-dev] [PATCH] gallium: add pipe_grid_info::partial_block

2019-01-08 Thread Jiang, Sonny
and add radeonsi support. This will be used by radeonsi internally. Signed-off-by: Sonny Jiang --- src/gallium/drivers/radeonsi/si_compute.c | 33 +++ src/gallium/include/pipe/p_state.h| 7 + 2 files changed, 35 insertions(+), 5 deletions(-) diff --git