[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <1183441706.10386.73.camel@localhost.localdomain>
Date: Tue, 03 Jul 2007 15:48:26 +1000
From: Benjamin Herrenschmidt <benh@...nel.crashing.org>
To: Nigel Cunningham <nigel@...el.suspend2.net>
Cc: Matthew Garrett <mjg59@...f.ucam.org>,
linux-kernel@...r.kernel.org, linux-pm@...ts.linux-foundation.org
Subject: Re: [PATCH] Remove process freezer from suspend to RAM pathway
> Note, though, that this won't help at all when people use the "suspend-to-ram
> instead of powering down after writing a hibernation image" feature in
> (uswsusp | tuxonice). Fuse is just a broken idea in the first place, but
> given that it exists, we still need to find the underlying cause.
No, Fuse is not a broken idea in the first place. It's the freezer that
is a totally broken idea. It has proven many times to be racy by design
and cannot be made right. Ther usermode helper mess is just part of
that, fuse is another example, etc etc ...
So I think Matthew is totally right. In fact, the presence of the
freezer is the main reason why Paulus so far NACKed Johannes attempts at
merging the PPC PM code with the generic code in kernel/power.c
We've been doing fine without it so far and intend to continue to do so.
As for suspend-to-disk, I refer you to the discussions we had in the
past with Linus, where he explains I think quite clearly how wrong the
current implementation of STR is :-)
Thing is, if you're going to do snapshots, you should probably not sync
after you have "frozen" anyway.
Cheers,
Ben.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists