Re: [Patch v2] Make PCI extended config space (MMCONFIG) a driveropt-in

From: Benjamin Herrenschmidt
Date: Fri Dec 28 2007 - 00:29:08 EST



On Thu, 2007-12-27 at 13:37 -0800, Linus Torvalds wrote:
>
> > Does anybody knows what was the original rational to call
> > pci_enable_crs() by default?
>
> .. another good question. I don't think anybody expected it to be
> broken,
> but if this turns out to be the thing that triggers it, I think we
> should
> disable CRS by default.

Some sane archs need CRS and do a lot of HW.. However, just testing for
vendor ID being 0x0001 instead of testing all bits might be a useable
workaround.

> The code doesn't actually do what CRS is supposed to help with (ie go
> on
> to probe another device and then come back to the slow one later), so
> right now it's pretty much useless *anyway*.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/