[lkml]   [2007]   [Apr]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] powerpc pseries eeh: Convert to kthread API
    On Sun, Apr 22, 2007 at 01:31:55PM +0100, Christoph Hellwig wrote:
    > On Thu, Apr 19, 2007 at 01:58:45AM -0600, Eric W. Biederman wrote:
    > > From: Eric W. Biederman <>
    > >
    > > This patch modifies the startup of eehd to use kthread_run
    > > not a combination of kernel_thread and daemonize. Making
    > > the code slightly simpler and more maintainable.
    > This one has the same scheme as the various s390 drivers where a thread
    > is spawned using a workqueue on demand. I think we should not blindly
    > convert it but think a litte more about it.
    > The first question is obviously, is this really something we want?
    > spawning kernel thread on demand without reaping them properly seems
    > quite dangerous.

    I'm not quite sure what the intent of this patch really is, being
    at most a somewhat passing and casual user of kernel threads.

    Some background may be useful: (this in reply to some comments from
    Andrew Morton)

    EEH events are supposed to be very rare, as they correspond to
    hardware failures, typically PCI bus parity errors, but also
    things like wild DMA's. The code that generates these will limit
    them to no more than 6 per hour per pci device. Any more than that,
    and the PCI device is permanently disabled (the sysadmin would
    need to do something to recover).

    The only reason for using threads here is to get the error recovery
    out of an interrupt context (where errors may be detected), and then,
    an hour later, decrement a counter (which is how we limit these to
    6 per hour). Thread reaping is "trivial", the thread just exits
    after an hour.

    Since these are events rare, I've no particular concern about
    performance or resource consumption. The current code seems
    to work just fine. :-)

    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: 2007-04-23 22:53    [W:0.207 / U:7.776 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site