On Sun, Jan 03, 2010 at 07:04:40PM -0800, Yinghai Lu wrote: > it is above 0x100, so if mmconf is not enable, need to skip it > > Reported-by: Jens Axboe <jens.axboe@xxxxxxxxxx> > Signed-off-by: Yinghai Lu <yinghai@xxxxxxxxxx> > --- > arch/x86/pci/intel_bus.c | 4 ++++ > 1 files changed, 4 insertions(+), 0 deletions(-) > > diff --git a/arch/x86/pci/intel_bus.c b/arch/x86/pci/intel_bus.c > index b7a55dc..baf283a 100644 > --- a/arch/x86/pci/intel_bus.c > +++ b/arch/x86/pci/intel_bus.c > @@ -49,6 +49,10 @@ static void __devinit pci_root_bus_res(struct pci_dev *dev) > u64 mmioh_base, mmioh_end; > int bus_base, bus_end; > > + /* some sys doesn't get mmconf enabled */ > + if (dev->cfg_size < 0x200) Here we check under the 512 index.. but the comment states that the check should be above 256 index? Should the comment say "it is above 0x200, so if mmconf.."? -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
