From: "Rafael J. Wysocki" <rjw@rjwysocki.net>
To: Jiri Kosina <jkosina@suse.com>
Cc: ksummit-discuss@lists.linuxfoundation.org
Subject: Re: [Ksummit-discuss] [TECH TOPIC] giving freezer well-defined semantics
Date: Wed, 08 Jul 2015 01:12:33 +0200 [thread overview]
Message-ID: <9412727.0bKgPF9YKr@vostro.rjw.lan> (raw)
In-Reply-To: <alpine.LNX.2.00.1507072307110.10183@pobox.suse.cz>
On Tuesday, July 07, 2015 11:13:01 PM Jiri Kosina wrote:
> On Tue, 7 Jul 2015, Rafael J. Wysocki wrote:
>
> > > Currently, the freezer has rather random semantics and there is no
> > > rigorous definition that would provide clear guidance which kernel threads
> > > should be freezable (and what rules they have to follow if they are).
> >
> > That's with respect to kernel threads, right?
>
> Yeah, sorry for not being verbose enough. I have been buried in kthread
> interactions with the rest of the kernel for too long, so my world view is
> now a bit distorted.
>
> Yes, this is purely related to try_to_freeze() in !PF_NOFREEZE kernel
> threads.
>
> > The primary purpose of the freezer is to get user space out of the way
> > and at least in principle there should be no fundamental need to freeze
> > any kernel threads.
>
> Except those which are needed for hibernation image I/O.
I don't quite see why there's a need to freeze those particular ones.
OK, it is necessary to ensure that the contents of the image will be consistent
with the state of filesystems on the storage media, so everything that may
change that state should be "frozen" before the image is created, but "frozen"
in terms of "no persistent state changes from now on" rather than in terms of
"no forward progress from now on".
Thanks,
Rafael
next prev parent reply other threads:[~2015-07-07 22:46 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-07-07 20:42 Jiri Kosina
2015-07-07 21:32 ` Rafael J. Wysocki
2015-07-07 21:13 ` Jiri Kosina
2015-07-07 23:12 ` Rafael J. Wysocki [this message]
2015-07-08 8:16 ` Jiri Kosina
2015-07-08 21:55 ` Rafael J. Wysocki
2015-07-09 11:25 ` Jan Kara
2015-07-10 0:18 ` Rafael J. Wysocki
2015-07-11 5:22 ` Benjamin Herrenschmidt
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=9412727.0bKgPF9YKr@vostro.rjw.lan \
--to=rjw@rjwysocki.net \
--cc=jkosina@suse.com \
--cc=ksummit-discuss@lists.linuxfoundation.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox