[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <878u6hjea5.fsf@notabene.neil.brown.name>
Date: Mon, 02 Nov 2015 14:01:06 +1100
From: Neil Brown <nfbrown@...ell.com>
To: Oliver Neukum <oneukum@...e.com>, Jiri Kosina <jikos@...nel.org>
Cc: "Rafael J. Wysocki" <rafael.j.wysocki@...el.com>,
Dave Chinner <david@...morbit.com>, Jan Kara <jack@...e.cz>,
Christoph Hellwig <hch@....de>,
Linus Torvalds <torvalds@...ux-foundation.org>,
Al Viro <viro@...IV.linux.org.uk>, Tejun Heo <tj@...nel.org>,
Pavel Machek <pavel@....cz>, linux-kernel@...r.kernel.org,
linux-fsdevel@...r.kernel.org, linux-pm@...r.kernel.org
Subject: Re: [PATCH 1/3] power, vfs: move away from PF_KTHREAD freezing in favor of fs freezing
On Sat, Oct 31 2015, Oliver Neukum wrote:
> On Fri, 2015-10-30 at 14:47 +0100, Jiri Kosina wrote:
>> Basically the main argument why kthread freezer is not needed boils
>> down to
>> this: the only facility that is needed during suspend: "no persistent
>> fs
>> changes are allowed from now on".
>
> Is that true? Drivers of character devices also may assume
> that IO and suspend() wouldn't race (except in fairly clear exceptions)
Anything that is a device can hook in to the suspend using the standard
call back and make sure no problematic races happen.
Filesystems are different partly because the "know" in memory some
details of what is persisting on storage, and partly because they aren't
devices.
Maybe filesystems should be devices .... though that probably wouldn't
help much.
I would suggest that "the only facility that is needed during suspend"
is really "well defined states and well defined transitions".
Filesystems need them as well as devices, and frozen kthreads might have
been a kludge to try to provide them.
NeilBrown
Download attachment "signature.asc" of type "application/pgp-signature" (819 bytes)
Powered by blists - more mailing lists