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

Benjamin Herrenschmidt
2007-12-28T05:28:58Z

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*.