From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jesse Barnes Subject: Re: [Bug #15124] PCI host bridge windows ignored (works with pci=use_crs) Date: Thu, 28 Jan 2010 12:12:16 -0800 Message-ID: <20100128121216.717ca66a@jbarnes-piketon> References: <201001261348.59508.rjw@sisk.pl> <201001261032.37053.bjorn.helgaas@hp.com> <201001261902.13911.rjw@sisk.pl> <20100126101752.78196900@jbarnes-piketon> <4B5F735B.2040308@kernel.org> <20100127192409.79af96db@jbarnes-piketon> <20100128040226.GB8964@jgarrett.org> <20100128082432.05a97ebc@jbarnes-piketon> <4B61D3C6.1050104@kernel.org> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <4B61D3C6.1050104-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> Sender: kernel-testers-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-ID: Content-Type: text/plain; charset="us-ascii" To: Yinghai Lu Cc: Jeff Garrett , Linus Torvalds , "Rafael J. Wysocki" , Bjorn Helgaas , Linux Kernel Mailing List , Kernel Testers List , Linux PCI , Myron Stowe , Matthew Garrett , Ingo Molnar On Thu, 28 Jan 2010 10:13:26 -0800 Yinghai Lu wrote: > On 01/28/2010 08:24 AM, Jesse Barnes wrote: > > On Wed, 27 Jan 2010 22:02:26 -0600 > > jeff-184d8XiO+QJg9hUCZPvPmw@public.gmane.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 wrote: > >>>> On Tue, 26 Jan 2010, Yinghai Lu wrote: > >>>>> > >>>>> [PATCH] x86/pci: don't use ioh resource if only have one ioh > >>>> > >>>> Please, no. > >>>> > >>>> This patch is too ugly to live. > >>>> > >>>> And it's totally unacceptable to probe every single possible PCI > >>>> device for something like this. > >>>> > >>>> If we don't know enough about the hardware workings of those > >>>> Intel bridges to know when they are active and how they decode > >>>> things, then please let's just disable intel_bus.c entirely. > >>>> > >>>> There's no excuse for hacky tests like this. > >>> > >>> Ok, we'll just kill it entirely then. I'll send a patch tomorrow > >>> unless Yinghai beats me to it. > >> > >> What about something like this (works for me, without pci=use_crs)? > >> > >> --- > >> Remove intel_bus.c Intel-specific PCI/IOH logic > >> > >> Signed-off-by: Jeff Garrett > > > > Yeah, looks good. I'll push to Linus today. > > > > please don't. will send you another patch, to keep the print out so > we can cross check the _CRS. I don't think there's much point due to the points we discussed earlier, I'd rather just get rid of it. -- Jesse Barnes, Intel Open Source Technology Center