tech-kern archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: xorg pci probing



>>> pci_device_is_boot_vga()
>> [...]
> While X is probing the pci devices the driver enables all pci vga
> devices.  That is why checking for the 'firmware enabled' one fails.

But checking for "the" firmware-enabled one is a broken idea.  There
may fewer or more than one such, for one thing - nothing says the
firmware has to initialize any displays; nothing says it has to
initialize no more than one.

And it's a check X shouldn't be doing anyway.  Doing bus enumeration in
userland is insane; that's what we've got kernels for.  X should be
using the device - or devices - it's told to use; if not told anything
about what device to use, it should be picking a sensible default, like
the console (not necessarily boot) device, without caring about others.
I can, sort of, see doing direct access to the display as a PCI device
(_of course_ every display will be a PCI device!), but in no
circumstance can I see any excuse for poking at devices it isn't going
to use or walking all of any bus.

/~\ The ASCII                             Mouse
\ / Ribbon Campaign
 X  Against HTML                mouse%rodents-montreal.org@localhost
/ \ Email!           7D C8 61 52 5D E7 2D 39  4E F1 31 3E E8 B3 27 4B


Home | Main Index | Thread Index | Old Index