Re: radeon big endian patch

2005-02-13 Thread Benjamin Herrenschmidt
On Sat, 2005-02-12 at 23:56 -0500, Michel Dänzer wrote: On Sun, 2005-02-13 at 15:16 +1100, Benjamin Herrenschmidt wrote: Those are still incorrect as they totally lack memory barriers... INREG() doesn't (or does it?), and it's the only one used by the 3D drivers. Hrm.. in fact, you are

Re: radeon big endian patch

2005-02-13 Thread Michel Dänzer
On Mon, 2005-02-14 at 09:07 +1100, Benjamin Herrenschmidt wrote: On Sat, 2005-02-12 at 23:56 -0500, Michel Dnzer wrote: On Sun, 2005-02-13 at 15:16 +1100, Benjamin Herrenschmidt wrote: Those are still incorrect as they totally lack memory barriers... INREG() doesn't (or does it?),

Re: radeon big endian patch

2005-02-13 Thread Benjamin Herrenschmidt
On Sun, 2005-02-13 at 19:25 -0500, Michel Dänzer wrote: On Mon, 2005-02-14 at 09:07 +1100, Benjamin Herrenschmidt wrote: On Sat, 2005-02-12 at 23:56 -0500, Michel Dänzer wrote: On Sun, 2005-02-13 at 15:16 +1100, Benjamin Herrenschmidt wrote: Those are still incorrect as they totally

radeon big endian patch

2005-02-12 Thread Jerome Glisse
Hi, Could anyone apply this patch if it looks good. This correct the problem many ppc user have using the Mesa-dri cvs which does not do the swapping for bendian. The code came from Michel, i am only doing the patch and the #ifdef things :) Jerome Glisse mesa_bendian Description: Binary data

Re: radeon big endian patch

2005-02-12 Thread Michel Dänzer
On Sat, 2005-02-12 at 13:03 +0100, Jerome Glisse wrote: Could anyone apply this patch if it looks good. This correct the problem many ppc user have using the Mesa-dri cvs which does not do the swapping for bendian. The code came from Michel, i am only doing the patch and the #ifdef

Re: radeon big endian patch

2005-02-12 Thread Benjamin Herrenschmidt
On Sat, 2005-02-12 at 12:42 -0500, Michel Dänzer wrote: On Sat, 2005-02-12 at 13:03 +0100, Jerome Glisse wrote: Could anyone apply this patch if it looks good. This correct the problem many ppc user have using the Mesa-dri cvs which does not do the swapping for bendian. The code

Re: radeon big endian patch

2005-02-12 Thread Michel Dänzer
On Sun, 2005-02-13 at 15:16 +1100, Benjamin Herrenschmidt wrote: Those are still incorrect as they totally lack memory barriers... INREG() doesn't (or does it?), and it's the only one used by the 3D drivers. -- Earthling Michel Dnzer | Debian (powerpc), X and DRI developer Libre