Re: [PATCH 1/2] pci: Block config access during BIST

Benjamin Herrenschmidt
2004-11-19T22:50:19Z

On Fri, 2004-11-19 at 16:25 -0600, Brian King wrote:

> I thought about that when writing up this patch, but decided against it.
> I figured it was overkill and was going to make the patch more complicated
> than it needed to be to solve the main problem I have seen, which is
> userspace code, usually hotplug/coldplug scripts, reading config space
> when an adapter is running BIST.

How so ? Why would it be more complicated to do the workaround in
drivers/pci/access.c macros instead and not touch all the wrappers ? It
would actually make a much smaller patch...

> If you think there are usages of the pci_bus_* functions in the
> kernel after the adapter device driver gets loaded, from callers other
> than adapter device drivers and userspace APIs, I would have to agree
> with you. I was hoping to keep this patch as simple as possible.
>
> Having to protect the pci_bus_* functions requires a lookup in these
> functions to find the pci_dev to get the saved_config_space, which
> I was hoping to avoid.
>
> Ben - do you have any concerns with this limitation for the use you have
> for this set of APIs?

If we ever endup rescanning the bus segment, indeed... I'd rather play
safe, it's easy to move the blocking to the bus access functions and
have the BIST function use the low level bus callbacks directly.

Ben.