Re: [Mesa-dev] [PATCH] r600g: order atom emission

2012-09-07 Thread Jerome Glisse
On Thu, Sep 6, 2012 at 11:32 AM, Alex Deucher wrote: > On Thu, Sep 6, 2012 at 10:54 AM, Jerome Glisse wrote: >> On Thu, Sep 6, 2012 at 6:20 AM, Dave Airlie wrote: >>> On Thu, Sep 6, 2012 at 5:21 PM, Philipp Klaus Krause wrote: On 06.09.2012 07:35, j.gli...@gmail.com wrote: > From: Jero

Re: [Mesa-dev] [PATCH] r600g: order atom emission v2

2012-09-06 Thread Jerome Glisse
On Thu, Sep 6, 2012 at 8:32 PM, Dave Airlie wrote: > On Fri, Sep 7, 2012 at 10:03 AM, Marek Olšák wrote: >> On Fri, Sep 7, 2012 at 12:05 AM, Jerome Glisse wrote: >>> On Thu, Sep 6, 2012 at 4:10 PM, Marek Olšák wrote: On Thu, Sep 6, 2012 at 8:34 PM, Jerome Glisse wrote: > On Thu, Sep 6

Re: [Mesa-dev] [PATCH] r600g: order atom emission v2

2012-09-06 Thread Dave Airlie
On Fri, Sep 7, 2012 at 10:03 AM, Marek Olšák wrote: > On Fri, Sep 7, 2012 at 12:05 AM, Jerome Glisse wrote: >> On Thu, Sep 6, 2012 at 4:10 PM, Marek Olšák wrote: >>> On Thu, Sep 6, 2012 at 8:34 PM, Jerome Glisse wrote: On Thu, Sep 6, 2012 at 2:29 PM, Marek Olšák wrote: > This looks go

Re: [Mesa-dev] [PATCH] r600g: order atom emission v2

2012-09-06 Thread Marek Olšák
On Fri, Sep 7, 2012 at 12:05 AM, Jerome Glisse wrote: > On Thu, Sep 6, 2012 at 4:10 PM, Marek Olšák wrote: >> On Thu, Sep 6, 2012 at 8:34 PM, Jerome Glisse wrote: >>> On Thu, Sep 6, 2012 at 2:29 PM, Marek Olšák wrote: This looks good to me. It's funny to see the r300g architecture being >>

Re: [Mesa-dev] [PATCH] r600g: order atom emission v2

2012-09-06 Thread Jerome Glisse
On Thu, Sep 6, 2012 at 4:10 PM, Marek Olšák wrote: > On Thu, Sep 6, 2012 at 8:34 PM, Jerome Glisse wrote: >> On Thu, Sep 6, 2012 at 2:29 PM, Marek Olšák wrote: >>> This looks good to me. It's funny to see the r300g architecture being >>> re-implemented in r600g. :) >>> >>> There's one optimizati

Re: [Mesa-dev] [PATCH] r600g: order atom emission v2

2012-09-06 Thread Marek Olšák
On Thu, Sep 6, 2012 at 8:34 PM, Jerome Glisse wrote: > On Thu, Sep 6, 2012 at 2:29 PM, Marek Olšák wrote: >> This looks good to me. It's funny to see the r300g architecture being >> re-implemented in r600g. :) >> >> There's one optimization that r300g has that this patch doesn't. r300g >> keeps t

Re: [Mesa-dev] [PATCH] r600g: order atom emission v2

2012-09-06 Thread Jerome Glisse
On Thu, Sep 6, 2012 at 2:29 PM, Marek Olšák wrote: > This looks good to me. It's funny to see the r300g architecture being > re-implemented in r600g. :) > > There's one optimization that r300g has that this patch doesn't. r300g > keeps the index of the first and the last dirty atom and the loops >

Re: [Mesa-dev] [PATCH] r600g: order atom emission v2

2012-09-06 Thread Marek Olšák
This looks good to me. It's funny to see the r300g architecture being re-implemented in r600g. :) There's one optimization that r300g has that this patch doesn't. r300g keeps the index of the first and the last dirty atom and the loops over the list of atoms look like this: for (i = first_dirty; i

Re: [Mesa-dev] [PATCH] r600g: order atom emission

2012-09-06 Thread Roland Scheidegger
Am 06.09.2012 16:54, schrieb Jerome Glisse: > On Thu, Sep 6, 2012 at 6:20 AM, Dave Airlie wrote: >> On Thu, Sep 6, 2012 at 5:21 PM, Philipp Klaus Krause wrote: >>> On 06.09.2012 07:35, j.gli...@gmail.com wrote: From: Jerome Glisse To avoid GPU lockup registers must be emited in a

Re: [Mesa-dev] [PATCH] r600g: order atom emission

2012-09-06 Thread Christian König
On 06.09.2012 18:06, Jerome Glisse wrote: On Thu, Sep 6, 2012 at 11:32 AM, Alex Deucher wrote: On Thu, Sep 6, 2012 at 10:54 AM, Jerome Glisse wrote: On Thu, Sep 6, 2012 at 6:20 AM, Dave Airlie wrote: On Thu, Sep 6, 2012 at 5:21 PM, Philipp Klaus Krause wrote: On 06.09.2012 07:35, j.gli...

Re: [Mesa-dev] [PATCH] r600g: order atom emission

2012-09-06 Thread Jerome Glisse
On Thu, Sep 6, 2012 at 11:32 AM, Alex Deucher wrote: > On Thu, Sep 6, 2012 at 10:54 AM, Jerome Glisse wrote: >> On Thu, Sep 6, 2012 at 6:20 AM, Dave Airlie wrote: >>> On Thu, Sep 6, 2012 at 5:21 PM, Philipp Klaus Krause wrote: On 06.09.2012 07:35, j.gli...@gmail.com wrote: > From: Jero

Re: [Mesa-dev] [PATCH] r600g: order atom emission

2012-09-06 Thread Alex Deucher
On Thu, Sep 6, 2012 at 10:54 AM, Jerome Glisse wrote: > On Thu, Sep 6, 2012 at 6:20 AM, Dave Airlie wrote: >> On Thu, Sep 6, 2012 at 5:21 PM, Philipp Klaus Krause wrote: >>> On 06.09.2012 07:35, j.gli...@gmail.com wrote: From: Jerome Glisse To avoid GPU lockup registers must be e

[Mesa-dev] [PATCH] r600g: order atom emission v2

2012-09-06 Thread j . glisse
From: Jerome Glisse To avoid GPU lockup registers must be emited in a specific order (no kidding ...). This patch rework atom emission so order in which atom are emited in respect to each other is always the same. We don't have any informations on what is the correct order so order will need to b

Re: [Mesa-dev] [PATCH] r600g: order atom emission

2012-09-06 Thread Jerome Glisse
On Thu, Sep 6, 2012 at 6:20 AM, Dave Airlie wrote: > On Thu, Sep 6, 2012 at 5:21 PM, Philipp Klaus Krause wrote: >> On 06.09.2012 07:35, j.gli...@gmail.com wrote: >>> From: Jerome Glisse >>> >>> To avoid GPU lockup registers must be emited in a specific order >>> (no kidding ...). This patch rew

Re: [Mesa-dev] [PATCH] r600g: order atom emission

2012-09-06 Thread Dave Airlie
On Thu, Sep 6, 2012 at 5:21 PM, Philipp Klaus Krause wrote: > On 06.09.2012 07:35, j.gli...@gmail.com wrote: >> From: Jerome Glisse >> >> To avoid GPU lockup registers must be emited in a specific order >> (no kidding ...). This patch rework atom emission so order in which >> atom are emited in r

Re: [Mesa-dev] [PATCH] r600g: order atom emission

2012-09-06 Thread Philipp Klaus Krause
On 06.09.2012 07:35, j.gli...@gmail.com wrote: > From: Jerome Glisse > > To avoid GPU lockup registers must be emited in a specific order > (no kidding ...). This patch rework atom emission so order in which > atom are emited in respect to each other is always the same. We > don't have any inform

[Mesa-dev] [PATCH] r600g: order atom emission

2012-09-05 Thread j . glisse
From: Jerome Glisse To avoid GPU lockup registers must be emited in a specific order (no kidding ...). This patch rework atom emission so order in which atom are emited in respect to each other is always the same. We don't have any informations on what is the correct order so order will need to b