On Thu, 28 Jan 2010 10:13:26 -0800 Yinghai Lu <yinghai@xxxxxxxxxx> wrote: > On 01/28/2010 08:24 AM, Jesse Barnes wrote: > > On Wed, 27 Jan 2010 22:02:26 -0600 > > jeff@xxxxxxxxxxxx (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 <torvalds@xxxxxxxxxxxxxxxxxxxx> 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 <jeff@xxxxxxxxxxxx> > > > > 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 -- 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
