[lkml]   [2008]   [May]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: regression fixed by using pci=rom
On Fri, May 09, 2008 at 08:44:03AM +0200, Ingo Molnar wrote:
> * Linus Torvalds <> wrote:
> > On Thu, 8 May 2008, Jesse Barnes wrote:
> > >
> > > Hm, yeah in many cases we definitely *do* want to try to get the
> > > expansion ROM space allocated. But maybe it should be a lower
> > > priority than other BARs... Gary?
> >
> > The thing is, a lot of these things have been done this way because
> > not doing them that way breaks.
> >
> > We want to allocate expansion ROM space - even if we don't enable it -
> > because not doing so will screw up bus sizing etc, and can make it
> > impossible to allocate later.
> >
> > In general, changing PCI allocation strategy is really _really_
> > dangerous, even when it is "right", because it tends to expose a lot
> > of issues where something worked just because it was perhaps
> > indirectly causing a layout that worked.
> i tend to believe that the best strategy would be to generally do what
> other OSs do for PCI BAR sizing and PCI resource setup [on desktop
> systems that would be Windows in particular] - and i believe there are
> still a number of gratuitous-looking differences in our code that seem
> unnecessary. Especially on the myriads of desktop systems Windows is
> what gets tested primarily, and by deviating from that legacy layout we
> just set up ourselves for unnecessary failures.

Ingo, Would you (or others listening to this discussion) know
exactly what Windows does with respect to BIOS unassigned
expansion ROMs? Does Windows attempt to obtain space for the
expansion ROMs at boot-time or does it perhaps have some sort
of strategy where space is allocated on an "as needed" basis
at run-time?


Gary Hade
System x Enablement
IBM Linux Technology Center
503-578-4503 IBM T/L: 775-4503

 \ /
  Last update: 2008-05-09 18:49    [W:0.132 / U:1.412 seconds]
©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site