[lkml]   [2006]   [Oct]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] Block on access to temporarily unavailable pci device
    Ar Mer, 2006-10-18 am 08:51 -0600, ysgrifennodd Matthew Wilcox:
    > The existing implementation of pci_block_user_cfg_access() was recently
    > criticised for providing out of date information and for returning errors
    > on write, which applications won't be expecting.

    It was also favoured by some of us as well. In addition this whole issue
    was extensively debated in the past to select the current approach. That
    said I do like the approach of a short wait *specifically* on power
    transitions, its bounded in time, its neat and it makes sense.

    We must be very very sure its never triggered in the real world any
    other way (eg your >block testing must be impossible)

    So unless you distinguish between "back in a moment", "back someday" and
    "not coming back" it isn't useful. Thus your patch is incomplete as it
    does not provide the cache that is also needed. At least I don't think X
    hanging forever mid mode switch is terribly useful...

    > I also addressed the potential issue with nested attempts to block.
    > Now pci_block_user_cfg_access() can return -EBUSY if it's already blocked,
    > and pci_unblock_user_cfg_access() will WARN if you try to unblock an
    > already unblocked device.

    Gak that is IMHO a mistake. Just keep a counter. You've just added
    another "what the hell do I do if this returns -EBUSY" problem for all
    the driver authors.

    > Signed-off-by: Matthew Wilcox <>

    NAK for all the above reasons. This is a cure looking for a disease, and
    its a cure which is worse than the theoretical disease it wants to fix.

    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: 2006-10-18 17:53    [W:0.020 / U:7.176 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site