Re: [Bug #15124] PCI host bridge windows ignored (works with pci=use_crs)

2010-01-28 Thread Rafael J. Wysocki
On Thursday 28 January 2010, Yinghai Lu wrote: On 01/27/2010 08:26 PM, Bjorn Helgaas wrote: On Wed, 2010-01-27 at 15:34 -0800, Yinghai Lu wrote: On 01/27/2010 01:03 PM, Bjorn Helgaas wrote: On Wednesday 27 January 2010 01:50:12 pm Linus Torvalds wrote: On Wed, 27 Jan 2010, Bjorn Helgaas

Re: [Bug #15124] PCI host bridge windows ignored (works with pci=use_crs)

2010-01-28 Thread Jesse Barnes
On Wed, 27 Jan 2010 22:02:26 -0600 j...@jgarrett.org (Jeff Garrett) wrote: On Wed, Jan 27, 2010 at 07:24:09PM -0800, Jesse Barnes wrote: On Wed, 27 Jan 2010 17:50:17 -0800 (PST) Linus Torvalds torva...@linux-foundation.org wrote: On Tue, 26 Jan 2010, Yinghai Lu wrote: [PATCH]

Re: [Bug #15124] PCI host bridge windows ignored (works with pci=use_crs)

2010-01-28 Thread Yinghai Lu
On 01/28/2010 08:09 AM, Bjorn Helgaas wrote: On Wednesday 27 January 2010 10:53:51 pm Yinghai Lu wrote: On 01/27/2010 08:26 PM, Bjorn Helgaas wrote: On Wed, 2010-01-27 at 15:34 -0800, Yinghai Lu wrote: 2. how about when apci is disabled? When ACPI is disabled, I think we just have to

[PATCH] x86/pci: print ioh resources only

2010-01-28 Thread Yinghai Lu
don't use them for peer pci root bus resource yet. so could cross check _CRS results Signed-off-by: Yinghai Lu ying...@kernel.org --- arch/x86/pci/intel_bus.c | 24 1 file changed, 8 insertions(+), 16 deletions(-) Index: linux-2.6/arch/x86/pci/intel_bus.c

Re: [PATCH] x86/pci: print ioh resources only

2010-01-28 Thread Linus Torvalds
On Thu, 28 Jan 2010, Yinghai Lu wrote: @@ -43,7 +43,7 @@ static void __devinit pci_root_bus_res(s { u16 word; u32 dword; - struct pci_root_info *info; + struct pci_root_info info; That structure is something like a kilobyte in size, please don't put those things on

Re: [Bug #15124] PCI host bridge windows ignored (works with pci=use_crs)

2010-01-28 Thread Jesse Barnes
On Thu, 28 Jan 2010 10:20:04 -0800 Yinghai Lu ying...@kernel.org wrote: On 01/28/2010 08:09 AM, Bjorn Helgaas wrote: On Wednesday 27 January 2010 10:53:51 pm Yinghai Lu wrote: On 01/27/2010 08:26 PM, Bjorn Helgaas wrote: On Wed, 2010-01-27 at 15:34 -0800, Yinghai Lu wrote: 2. how

[PATCH -v2] x86/pci: print ioh resources only

2010-01-28 Thread Yinghai Lu
don't use them for peer pci root bus resource yet. so could cross check _CRS results -v2: dont put info struct in stack according to Linus. because that is kbytes big Signed-off-by: Yinghai Lu ying...@kernel.org --- arch/x86/pci/intel_bus.c |8 ++-- 1 file changed, 2

Re: [PATCH -v2] x86/pci: print ioh resources only

2010-01-28 Thread Olivier Galibert
On Thu, Jan 28, 2010 at 11:10:14AM -0800, Yinghai Lu wrote: don't use them for peer pci root bus resource yet. so could cross check _CRS results -v2: dont put info struct in stack according to Linus. because that is kbytes big No kfree? OG. -- To unsubscribe from this list: send

Re: [Bug #15124] PCI host bridge windows ignored (works with pci=use_crs)

2010-01-28 Thread Bjorn Helgaas
On Thursday 28 January 2010 11:20:04 am Yinghai Lu wrote: On 01/28/2010 08:09 AM, Bjorn Helgaas wrote: On Wednesday 27 January 2010 10:53:51 pm Yinghai Lu wrote: On 01/27/2010 08:26 PM, Bjorn Helgaas wrote: On Wed, 2010-01-27 at 15:34 -0800, Yinghai Lu wrote: 2. how about when apci is

Re: [Bug #15124] PCI host bridge windows ignored (works with pci=use_crs)

2010-01-28 Thread Rafael J. Wysocki
On Thursday 28 January 2010, Jesse Barnes wrote: On Thu, 28 Jan 2010 10:20:04 -0800 Yinghai Lu ying...@kernel.org wrote: On 01/28/2010 08:09 AM, Bjorn Helgaas wrote: On Wednesday 27 January 2010 10:53:51 pm Yinghai Lu wrote: On 01/27/2010 08:26 PM, Bjorn Helgaas wrote: On Wed,

Re: [Bug #15124] PCI host bridge windows ignored (works with pci=use_crs)

2010-01-28 Thread Bjorn Helgaas
On Thursday 28 January 2010 11:20:04 am Yinghai Lu wrote: On 01/28/2010 08:09 AM, Bjorn Helgaas wrote: On Wednesday 27 January 2010 10:53:51 pm Yinghai Lu wrote: We should fix that problem rather than just covering it up by disabling ACPI. Can you provide any details? that is not covering