[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