[PATCH 2/2] radeon: optimize allocation for depth w/o stencil and stencil w/o depth on EG

2012-07-30 Thread Marek Olšák
On Mon, Jul 30, 2012 at 4:56 PM, Jerome Glisse wrote: > On Sun, Jul 29, 2012 at 1:04 PM, Marek Ol??k wrote: >> If we don't need stencil, don't allocate it. >> If we need only stencil (like PIPE_FORMAT_S8_UINT), don't allocate depth. >> >> v2: actually do it correctly > > Big NAK > > We need to

[PATCH 2/2] radeon: optimize allocation for depth w/o stencil and stencil w/o depth on EG

2012-07-30 Thread Roland Scheidegger
Am 30.07.2012 16:56, schrieb Jerome Glisse: > On Sun, Jul 29, 2012 at 1:04 PM, Marek Ol??k wrote: >> If we don't need stencil, don't allocate it. >> If we need only stencil (like PIPE_FORMAT_S8_UINT), don't allocate depth. >> >> v2: actually do it correctly > > Big NAK > > We need to allocate

[PATCH 2/2] radeon: optimize allocation for depth w/o stencil and stencil w/o depth on EG

2012-07-30 Thread Christian König
On 30.07.2012 16:56, Jerome Glisse wrote: > On Sun, Jul 29, 2012 at 1:04 PM, Marek Ol??k wrote: >> If we don't need stencil, don't allocate it. >> If we need only stencil (like PIPE_FORMAT_S8_UINT), don't allocate depth. >> >> v2: actually do it correctly > Big NAK > > We need to allocate stencil

[PATCH 2/2] radeon: optimize allocation for depth w/o stencil and stencil w/o depth on EG

2012-07-30 Thread Jerome Glisse
On Mon, Jul 30, 2012 at 1:03 PM, Alex Deucher wrote: > On Mon, Jul 30, 2012 at 12:52 PM, Jerome Glisse wrote: >> On Mon, Jul 30, 2012 at 12:16 PM, Alex Deucher >> wrote: >>> On Mon, Jul 30, 2012 at 12:03 PM, Alex Deucher >>> wrote: On Mon, Jul 30, 2012 at 11:48 AM, Marek Ol??k wrote:

[PATCH 2/2] radeon: optimize allocation for depth w/o stencil and stencil w/o depth on EG

2012-07-30 Thread Alex Deucher
On Mon, Jul 30, 2012 at 12:52 PM, Jerome Glisse wrote: > On Mon, Jul 30, 2012 at 12:16 PM, Alex Deucher > wrote: >> On Mon, Jul 30, 2012 at 12:03 PM, Alex Deucher >> wrote: >>> On Mon, Jul 30, 2012 at 11:48 AM, Marek Ol??k wrote: On Mon, Jul 30, 2012 at 4:56 PM, Jerome Glisse

[PATCH 2/2] radeon: optimize allocation for depth w/o stencil and stencil w/o depth on EG

2012-07-30 Thread Jerome Glisse
On Mon, Jul 30, 2012 at 12:16 PM, Alex Deucher wrote: > On Mon, Jul 30, 2012 at 12:03 PM, Alex Deucher > wrote: >> On Mon, Jul 30, 2012 at 11:48 AM, Marek Ol??k wrote: >>> On Mon, Jul 30, 2012 at 4:56 PM, Jerome Glisse >>> wrote: On Sun, Jul 29, 2012 at 1:04 PM, Marek Ol??k wrote:

[PATCH 2/2] radeon: optimize allocation for depth w/o stencil and stencil w/o depth on EG

2012-07-30 Thread Alex Deucher
On Mon, Jul 30, 2012 at 12:03 PM, Alex Deucher wrote: > On Mon, Jul 30, 2012 at 11:48 AM, Marek Ol??k wrote: >> On Mon, Jul 30, 2012 at 4:56 PM, Jerome Glisse wrote: >>> On Sun, Jul 29, 2012 at 1:04 PM, Marek Ol??k wrote: If we don't need stencil, don't allocate it. If we need only

[PATCH 2/2] radeon: optimize allocation for depth w/o stencil and stencil w/o depth on EG

2012-07-30 Thread Alex Deucher
On Mon, Jul 30, 2012 at 11:48 AM, Marek Ol??k wrote: > On Mon, Jul 30, 2012 at 4:56 PM, Jerome Glisse wrote: >> On Sun, Jul 29, 2012 at 1:04 PM, Marek Ol??k wrote: >>> If we don't need stencil, don't allocate it. >>> If we need only stencil (like PIPE_FORMAT_S8_UINT), don't allocate depth. >>>

[PATCH 2/2] radeon: optimize allocation for depth w/o stencil and stencil w/o depth on EG

2012-07-30 Thread Jerome Glisse
On Mon, Jul 30, 2012 at 11:17 AM, Christian K?nig wrote: > On 30.07.2012 16:56, Jerome Glisse wrote: >> >> On Sun, Jul 29, 2012 at 1:04 PM, Marek Ol??k wrote: >>> >>> If we don't need stencil, don't allocate it. >>> If we need only stencil (like PIPE_FORMAT_S8_UINT), don't allocate depth. >>>

[PATCH 2/2] radeon: optimize allocation for depth w/o stencil and stencil w/o depth on EG

2012-07-30 Thread Jerome Glisse
On Sun, Jul 29, 2012 at 1:04 PM, Marek Ol??k wrote: > If we don't need stencil, don't allocate it. > If we need only stencil (like PIPE_FORMAT_S8_UINT), don't allocate depth. > > v2: actually do it correctly Big NAK We need to allocate stencil and depth no matter what. Otherwise it will lockup.

Re: [PATCH 2/2] radeon: optimize allocation for depth w/o stencil and stencil w/o depth on EG

2012-07-30 Thread Jerome Glisse
On Sun, Jul 29, 2012 at 1:04 PM, Marek Olšák mar...@gmail.com wrote: If we don't need stencil, don't allocate it. If we need only stencil (like PIPE_FORMAT_S8_UINT), don't allocate depth. v2: actually do it correctly Big NAK We need to allocate stencil and depth no matter what. Otherwise it

Re: [PATCH 2/2] radeon: optimize allocation for depth w/o stencil and stencil w/o depth on EG

2012-07-30 Thread Christian König
On 30.07.2012 16:56, Jerome Glisse wrote: On Sun, Jul 29, 2012 at 1:04 PM, Marek Olšák mar...@gmail.com wrote: If we don't need stencil, don't allocate it. If we need only stencil (like PIPE_FORMAT_S8_UINT), don't allocate depth. v2: actually do it correctly Big NAK We need to allocate

Re: [PATCH 2/2] radeon: optimize allocation for depth w/o stencil and stencil w/o depth on EG

2012-07-30 Thread Marek Olšák
On Mon, Jul 30, 2012 at 4:56 PM, Jerome Glisse j.gli...@gmail.com wrote: On Sun, Jul 29, 2012 at 1:04 PM, Marek Olšák mar...@gmail.com wrote: If we don't need stencil, don't allocate it. If we need only stencil (like PIPE_FORMAT_S8_UINT), don't allocate depth. v2: actually do it correctly

Re: [PATCH 2/2] radeon: optimize allocation for depth w/o stencil and stencil w/o depth on EG

2012-07-30 Thread Alex Deucher
On Mon, Jul 30, 2012 at 11:48 AM, Marek Olšák mar...@gmail.com wrote: On Mon, Jul 30, 2012 at 4:56 PM, Jerome Glisse j.gli...@gmail.com wrote: On Sun, Jul 29, 2012 at 1:04 PM, Marek Olšák mar...@gmail.com wrote: If we don't need stencil, don't allocate it. If we need only stencil (like

Re: [PATCH 2/2] radeon: optimize allocation for depth w/o stencil and stencil w/o depth on EG

2012-07-30 Thread Alex Deucher
On Mon, Jul 30, 2012 at 12:03 PM, Alex Deucher alexdeuc...@gmail.com wrote: On Mon, Jul 30, 2012 at 11:48 AM, Marek Olšák mar...@gmail.com wrote: On Mon, Jul 30, 2012 at 4:56 PM, Jerome Glisse j.gli...@gmail.com wrote: On Sun, Jul 29, 2012 at 1:04 PM, Marek Olšák mar...@gmail.com wrote: If we

Re: [PATCH 2/2] radeon: optimize allocation for depth w/o stencil and stencil w/o depth on EG

2012-07-30 Thread Jerome Glisse
On Mon, Jul 30, 2012 at 12:16 PM, Alex Deucher alexdeuc...@gmail.com wrote: On Mon, Jul 30, 2012 at 12:03 PM, Alex Deucher alexdeuc...@gmail.com wrote: On Mon, Jul 30, 2012 at 11:48 AM, Marek Olšák mar...@gmail.com wrote: On Mon, Jul 30, 2012 at 4:56 PM, Jerome Glisse j.gli...@gmail.com wrote:

Re: [PATCH 2/2] radeon: optimize allocation for depth w/o stencil and stencil w/o depth on EG

2012-07-30 Thread Alex Deucher
On Mon, Jul 30, 2012 at 12:52 PM, Jerome Glisse j.gli...@gmail.com wrote: On Mon, Jul 30, 2012 at 12:16 PM, Alex Deucher alexdeuc...@gmail.com wrote: On Mon, Jul 30, 2012 at 12:03 PM, Alex Deucher alexdeuc...@gmail.com wrote: On Mon, Jul 30, 2012 at 11:48 AM, Marek Olšák mar...@gmail.com wrote:

Re: [PATCH 2/2] radeon: optimize allocation for depth w/o stencil and stencil w/o depth on EG

2012-07-30 Thread Jerome Glisse
On Mon, Jul 30, 2012 at 1:03 PM, Alex Deucher alexdeuc...@gmail.com wrote: On Mon, Jul 30, 2012 at 12:52 PM, Jerome Glisse j.gli...@gmail.com wrote: On Mon, Jul 30, 2012 at 12:16 PM, Alex Deucher alexdeuc...@gmail.com wrote: On Mon, Jul 30, 2012 at 12:03 PM, Alex Deucher alexdeuc...@gmail.com

[PATCH 2/2] radeon: optimize allocation for depth w/o stencil and stencil w/o depth on EG

2012-07-29 Thread Marek Olšák
If we don't need stencil, don't allocate it. If we need only stencil (like PIPE_FORMAT_S8_UINT), don't allocate depth. v2: actually do it correctly --- radeon/radeon_surface.c | 23 --- 1 file changed, 8 insertions(+), 15 deletions(-) diff --git a/radeon/radeon_surface.c

[PATCH 2/2] radeon: optimize allocation for depth w/o stencil and stencil w/o depth on EG

2012-07-29 Thread Marek Olšák
If we don't need stencil, don't allocate it. If we need only stencil (like PIPE_FORMAT_S8_UINT), don't allocate depth. --- radeon/radeon_surface.c | 19 +-- 1 file changed, 5 insertions(+), 14 deletions(-) diff --git a/radeon/radeon_surface.c b/radeon/radeon_surface.c index

[PATCH 2/2] radeon: optimize allocation for depth w/o stencil and stencil w/o depth on EG

2012-07-29 Thread Marek Olšák
If we don't need stencil, don't allocate it. If we need only stencil (like PIPE_FORMAT_S8_UINT), don't allocate depth. --- radeon/radeon_surface.c | 19 +-- 1 file changed, 5 insertions(+), 14 deletions(-) diff --git a/radeon/radeon_surface.c b/radeon/radeon_surface.c index

[PATCH 2/2] radeon: optimize allocation for depth w/o stencil and stencil w/o depth on EG

2012-07-29 Thread Marek Olšák
If we don't need stencil, don't allocate it. If we need only stencil (like PIPE_FORMAT_S8_UINT), don't allocate depth. v2: actually do it correctly --- radeon/radeon_surface.c | 23 --- 1 file changed, 8 insertions(+), 15 deletions(-) diff --git a/radeon/radeon_surface.c