login
Header Space

 
 

Re: Why such a big difference in init-time PCI resource call-paths (x86 vs x86_64) ?

Previous thread: [PATCH] v4l/dvb: Add linux/version.h include as KERNEL_VERSION macro is used. by Andres Freund on Thursday, May 1, 2008 - 2:05 pm. (7 messages)

Next thread: Re:Re: kernel BUG at mm/rmap.c:669 (during localegen) by devzero on Thursday, May 1, 2008 - 2:19 pm. (2 messages)
To: <linux-pci@...>
Cc: TJ <linux@...>, Andi Kleen <andi@...>, Ingo Molnar <mingo@...>, Thomas Gleixner <tglx@...>, H. Peter Anvin <hpa@...>, linux-kernel <linux-kernel@...>
Date: Thursday, May 1, 2008 - 2:16 pm

Well e820_setup_gap is a bit more descriptive, but if you're factoring out 
common code in the x86 tree, it may as well be even more descriptive, e.g. 

Andi may remember some of the history here.  It seems like we should be 
reserving those regions early on like the x86_64 code does, but it's possible 
that will create more conflicts later on for some platforms (I don't know of 

For future patches and x86 specific questions, you should probably cc lkml and 
the x86 maintainers, they may be able to provide additional feedback.

Thanks,
Jesse
--
To: Jesse Barnes <jbarnes@...>
Cc: <linux-pci@...>, TJ <linux@...>, Andi Kleen <andi@...>, Ingo Molnar <mingo@...>, Thomas Gleixner <tglx@...>, H. Peter Anvin <hpa@...>, linux-kernel <linux-kernel@...>
Date: Thursday, May 1, 2008 - 4:11 pm

I tried that some time ago and it turned out that some systems have
mappings in holes and don't boot anymore when you fill the holes too much. 
But that was only considering e820. if you do this it might work if you 

I don't remember why it is different. Probably wasn't intentional.
But in general x86-64 is less fragile than i386 here because it has the e820
allocator and can deal better with conflicts.

-Andi
--
To: Andi Kleen <andi@...>
Cc: Jesse Barnes <jbarnes@...>, <linux-pci@...>, TJ <linux@...>, Ingo Molnar <mingo@...>, Thomas Gleixner <tglx@...>, H. Peter Anvin <hpa@...>, linux-kernel <linux-kernel@...>
Date: Thursday, May 1, 2008 - 5:10 pm

wonder if using holes in MTRR AND e820 could help...

YH
--
To: Yinghai Lu <yhlu.kernel@...>
Cc: Andi Kleen <andi@...>, Jesse Barnes <jbarnes@...>, <linux-pci@...>, TJ <linux@...>, Ingo Molnar <mingo@...>, Thomas Gleixner <tglx@...>, linux-kernel <linux-kernel@...>
Date: Thursday, May 1, 2008 - 5:11 pm

Typically not, since the MTRRs won't tell you what is free address space 
and what is occupied by non-BAR I/O devices.

	-hpa
--
To: Andi Kleen <andi@...>
Cc: Jesse Barnes <jbarnes@...>, <linux-pci@...>, TJ <linux@...>, Ingo Molnar <mingo@...>, Thomas Gleixner <tglx@...>, linux-kernel <linux-kernel@...>
Date: Thursday, May 1, 2008 - 4:11 pm

Yes, considering all possible reservation schemes is really critical 

Yes, and that's definitely the model we want on both sides.  Working on 
this is way high on my personal priority list at the moment.

	-hpa

--
To: H. Peter Anvin <hpa@...>
Cc: Andi Kleen <andi@...>, <linux-pci@...>, TJ <linux@...>, Ingo Molnar <mingo@...>, Thomas Gleixner <tglx@...>, linux-kernel <linux-kernel@...>
Date: Thursday, May 1, 2008 - 4:29 pm

Yeah, good point.  There's code to check for ACPI reserved regions when 
verifying MCFG space, so at least we're starting to do this...

Thanks,
Jesse

--
Previous thread: [PATCH] v4l/dvb: Add linux/version.h include as KERNEL_VERSION macro is used. by Andres Freund on Thursday, May 1, 2008 - 2:05 pm. (7 messages)

Next thread: Re:Re: kernel BUG at mm/rmap.c:669 (during localegen) by devzero on Thursday, May 1, 2008 - 2:19 pm. (2 messages)
speck-geostationary