Messages in this thread | | | Subject | Re: [PATCH 1/2] pci: Block config access during BIST | From | Benjamin Herrenschmidt <> | Date | Sat, 20 Nov 2004 09:46:42 +1100 |
| |
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.
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |