ksummit.lists.linux.dev archive mirror
 help / color / mirror / Atom feed
From: Jiri Kosina <jkosina@suse.com>
To: ksummit-discuss@lists.linuxfoundation.org
Subject: [Ksummit-discuss] [TECH TOPIC] giving freezer well-defined semantics
Date: Tue, 7 Jul 2015 22:42:36 +0200 (CEST)	[thread overview]
Message-ID: <alpine.LNX.2.00.1507071340480.10183@pobox.suse.cz> (raw)

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).

The long and complicated history of freezer resulted in the current state, 
where many kernel threads are marked freezable "just because", with 
freezing points randomly sprinkled all over the place, and it really 
complicates any efforts in cleaning up kthreads (which is why I am 
interested in it -- we'd love to have kthreads cleaned up to make live 
patching easier).

There are various possible aproaches to this. One is basically ad-hoc, 
going over all the existing kthreads one by one, and fixing (probably just 
removing, in most cases) its usage of freezer. The obivous disadvantage is 
that this doesn't prevent anyone to abusing it again the same way in the 
future.

Tejun came up with a different aproach at [1] -- basically getting rid of 
the freezer completely, and rather annotating those I/O requests which are 
needed for writing the hibernation image out, so that they make it through 
all the affected subsystems, while other I/O requests would be frozen.

This would be rather dramatic change both in a way how kthreads work, how 
hibernation works, and it'd be necessary to have means to mark I/O 
requests as "needed for hibernation", therefore I think this would be a 
good cross-subsystem topic.

[1] http://lkml.kernel.org/r/20150613232222.GB346@mtj.duckdns.org

-- 
Jiri Kosina
SUSE Labs

             reply	other threads:[~2015-07-07 20:42 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-07 20:42 Jiri Kosina [this message]
2015-07-07 21:32 ` Rafael J. Wysocki
2015-07-07 21:13   ` Jiri Kosina
2015-07-07 23:12     ` Rafael J. Wysocki
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=alpine.LNX.2.00.1507071340480.10183@pobox.suse.cz \
    --to=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