[PULL] bus cleanup, darwin hw/xfree86, build regression fixes
Jeremy Huddleston
jeremyhu at apple.com
Sat Sep 24 12:35:30 PDT 2011
On Sep 24, 2011, at 09:31, Matthieu Herrb wrote:
>> IOADDRESS is used massively throughout the drivers. I really don't
>> see why it needs to be removed. Leaving the typedef in xf86Pci.h will
>> save you/us a lot of work. At that point you might as well drop the
>> changesto remove IOADDRESS usage in the int10 module.
>
> I agree with Mark on this.
Ok. Then how about leaving the PCITAG and IOADDRESS typedefs in place but deprecate their use. I also think we should change them from unsigned long to CARD32 for better compatibility with pciaccess (yes, ABI break, but we know that).
>> Still only a partial review; looking into the changes to the
>> interfaces that map video memory now.
>
> I wanted to look, and more importantly test them, but so far the
> IOADRESS fallout (plus a few other stuff induced by the usual
> autotools churn) has caused the build of drivers to fail. I'll restart
> the build with the IOADDRES removal reverted and see how it goes.
>
> One 1st comment though: apparently the intension is to remove the old
> MTRR support backend completely (since its now done by libpciaccess
> directly) but some bits of it have been left in place in
> bsd/i386_video.c. Is that intentional ?
Just in case it wasn't clear already, I'm doing this so that I can turn *off* bits that require pciaccess in my builds on osx. I can do minimal build fix failures across the board but I can only test with what I have. I have a ppc and intel linux box and can test vesa, vmware, nv, nouveau, ati, and intel (with 1 pci domain) which covers the common cases. I am brand new to the hw/xfree86 and libpciaccess codebase, and these changes are fairly extensive, so I'm glad to see these changes getting solid attention and reviews (thanks).
I really wish ajax would speak up about these changes but he's been mostly silent. It's entirely possible that he just missed that MTRR code in bsd/i386_video.c.
More information about the xorg-devel
mailing list