PCI rework

Jesse Barnes jbarnes at virtuousgeek.org
Fri Apr 28 14:10:55 PDT 2006


On Thursday, April 27, 2006 11:02 am, Ian Romanick wrote:
> I've just posted my first set of patches for my PCI rework.  Right now
> this just patches the core server and the MGA driver.  These patches
> require that libpciaccess be built and installed like all the other X
> libs.
>
> http://users.freedesktop.org/~idr/pci_cleanup/

Nice.  I especially like all the code removal.

> The MGA driver currently just blanks the screen when UseFBDev is not
> used.  I had hoped to track that issue down before posting these
> patches, but I've been busy putting out other fires.

There are a few places where you call pci_device_map_region(dev, n, 
TRUE).  I wonder if those should be pci_device_map_region(dev, 
pMga->(io_bar|framebuffer_bar|whatever), TRUE) instead?  Or did you 
intend to hardcode them?

> I'm trying to understand how all of "resource" stuff works and how
> much more of it I can gut.  It seems like most of it can go.  The only
> resources that we should have to track and deal with conflicts is ISA,
> right?

If we assume ISA pnp and that the kernel will be managing resources, even 
the ISA resource management could go probably, but that might require 
another library (libisaaccess? :).

> The other issue is determining which release this will eventually
> target.  7.2?  8.0?  7.1 is obviously right out. :)

My vote (fwiw given that I don't have an ObBug at the bottom of this 
mail) is ASAP.

Thanks,
Jesse



More information about the xorg mailing list