[SeaBIOS] [PATCH 0/9] Misc boot related cleanups
gleb at redhat.com
Wed Dec 29 21:29:57 CET 2010
On Wed, Dec 29, 2010 at 02:10:16PM -0500, Kevin O'Connor wrote:
> On Wed, Dec 29, 2010 at 09:00:36PM +0200, Gleb Natapov wrote:
> > On Wed, Dec 29, 2010 at 12:46:45PM -0500, Kevin O'Connor wrote:
> > > Some cleanups I came across while reviewing the boot code.
> > >
> > > Gleb - patch 7 conflicts with your last patch - I hope it isn't a big
> > > deal to fixup though. (I'm still reviewing your last patch.)
> > >
> > The remaining patch is pretty independent. Should be very easy to rebase.
> > Do you want me to do it?
> It wouldn't hurt.
> BTW, I was thinking of ways to try and simplify this code. What if we
> tried doing an exact string match for the device, but if that fails
> and a BDF is known for the device then do a prefix string search for
> the PCI device?
I don't see why this would be simpler. Also how can you do exact string
match if BDF is not knows? You can't create device path without knowing
BDF (well DF). My code works like OpenFirmware device path was designed
to work: match device path one node at a time. This also allows for
greater flexibility in qemu<->seabios interface.
More information about the SeaBIOS