Re: Back to the future.
From: Rafael J. Wysocki
Date: Sat Apr 28 2007 - 13:47:17 EST
On Saturday, 28 April 2007 18:28, Linus Torvalds wrote:
> On Sat, 28 Apr 2007, Pavel Machek wrote:
> > We do not want kernel threads running:
> > a) they may hold some locks and deadlock suspend
> > b) they may do some writes to disk, leading to corruption
> You're really just making both of those up.
> If a kernel thread holds a lock and deadlocks suspend, that would deadlock
> anythign else _too_. Suspend isn't *that* special. Everything it does are
> things other people do too.
> And no, kernel threads do not write to disk on their own. Name one.
xfssyncd , or at least it seems so at a quick look.
> They help *others* write to disk, but those disk writes need to happen.
> The freezer has *caused* those deadlocks (eg by stopping threads that were
> needed for the suspend writeouts to succeed!), not solved them.
I can't remember anything like this, but I believe you have a specific test
case in mind.
> So stop making these totally bogus arguments up.
Well, they may be bogus, but there's something else.
I have reviewed some kernel threads used by device drivers that currently are
frozen to see if it would be safe not to freeze them, and I'm worried.
What, for example, if such a thread schedules a timeout and waits for
something to happen (eg. the airo driver does something like this), but instead
the hibernation/suspend happens and the device is frozen/suspended under it?
Shouldn't the thread be notified by the driver's freeze/suspend callback?
Moreover, what if after the restore the device is not present (for example, it
may be a pcmcia card that the user has removed) and the thread is scheduled
before the device's unfreeze callback has a chance to run? Shouldn't the
thread check that the device is present? In that case it would have to be
notified by someone that the check is necessary, but who can do that?
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/