Re: [Mesa-dev] [PATCH 1/7] nvc0: reduce the number of PUSH_SPACE in draw path

2016-11-07 Thread Samuel Pitoiset
On 11/07/2016 04:32 AM, Ilia Mirkin wrote: On Wed, Oct 26, 2016 at 4:14 AM, Samuel Pitoiset wrote: On 10/25/2016 09:49 PM, Ilia Mirkin wrote: What if instance_count = 1M? (It can happen.) We allocate a giant space in the pushbuf in one shot. Well, anyways this is not the optimization o

Re: [Mesa-dev] [PATCH 1/7] nvc0: reduce the number of PUSH_SPACE in draw path

2016-11-06 Thread Ilia Mirkin
On Wed, Oct 26, 2016 at 4:14 AM, Samuel Pitoiset wrote: > > > On 10/25/2016 09:49 PM, Ilia Mirkin wrote: >> >> What if instance_count = 1M? (It can happen.) > > > We allocate a giant space in the pushbuf in one shot. Well, anyways this is > not the optimization of the year, so I can drop it. :-)

Re: [Mesa-dev] [PATCH 1/7] nvc0: reduce the number of PUSH_SPACE in draw path

2016-10-26 Thread Samuel Pitoiset
On 10/25/2016 09:49 PM, Ilia Mirkin wrote: What if instance_count = 1M? (It can happen.) We allocate a giant space in the pushbuf in one shot. Well, anyways this is not the optimization of the year, so I can drop it. :-) On Tue, Oct 25, 2016 at 3:41 PM, Samuel Pitoiset wrote: This migh

Re: [Mesa-dev] [PATCH 1/7] nvc0: reduce the number of PUSH_SPACE in draw path

2016-10-25 Thread Ilia Mirkin
What if instance_count = 1M? (It can happen.) On Tue, Oct 25, 2016 at 3:41 PM, Samuel Pitoiset wrote: > This might help CPU-bounds applications but should not have > any real effects for GPU-bounds ones. > > Signed-off-by: Samuel Pitoiset > --- > src/gallium/drivers/nouveau/nvc0/nvc0_vbo.c | 8

[Mesa-dev] [PATCH 1/7] nvc0: reduce the number of PUSH_SPACE in draw path

2016-10-25 Thread Samuel Pitoiset
This might help CPU-bounds applications but should not have any real effects for GPU-bounds ones. Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/nouveau/nvc0/nvc0_vbo.c | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_vbo.c