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

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 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

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, y

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 DÃnzer | Debian (powerpc), X and DRI developer Libre

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. > >

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 #ifde