[lkml]   [2004]   [Nov]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH 1/2] pci: Block config access during BIST
    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.


    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2005-03-22 14:08    [W:0.022 / U:36.520 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site