[PATCH] Revert "Make sys.c use unaligned access functions provided in compiler."

Tiago Vignatti tiago.vignatti at nokia.com
Tue Dec 29 05:42:09 PST 2009


On Mon, Dec 28, 2009 at 08:02:02PM +0100, ext Matt Turner wrote:
> On Mon, Dec 28, 2009 at 11:00 AM, Tiago Vignatti
> <tiago.vignatti at nokia.com> wrote:
> > Sorry to not scream before about it, when the patch came to xorg-devel review.
> >
> > Now I could try to fix the issue of the patch, but I'm not happy with its
> > approach: the patch is using a header from the server inside x86emu. There's
> > no conceptual problem on it but it will complicates when we try to remove
> > x86emu from the server (my ongoing work - using lrmi).
> >
> > So I'd be happy to just revert such patch now.

...


> The fix here isn't to add back in a bunch of duplicated Alpha assembly.
> 
> It this needs its own unaligned access code, then copy the platform
> agnostic code from compiler.h. That is, the code like
> 
> static __inline__ uint32_t ldl_u(uint32_t *p)
> {
>     const struct __una_u32 *ptr = (const struct __una_u32 *) p;
>     return ptr->x;
> }
> 
> But please, do not add back in the Alpha assembly.
> 
> I understand that ARM-specific nastiness in compiler.h (lines
> compiler.h:1017-1019) causes sys.c to not compile--there is an
> appropriate fix and this is not it.
> 
> Maybe move ARM away from out{b,w,l} to the new pci access routines ajax wrote?

Okay okay okay... but you missed my argument why I'm against your approach. 

So again: compiler.h would be the first header from X inside x86emu. By nature
x86emu should not be inside X. So, adding a header from Xorg inside x86emu is
just coupling even more the entire server. We're more than never trying to
decouple modules, components and os routines in our X implementation. 

Please, revert it.

            Tiago


More information about the xorg-devel mailing list