mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@elte.hu>
To: Andrew Morton <akpm@osdl.org>
Cc: paulmck@linux.vnet.ibm.com, dipankar@in.ibm.com,
	Gautham Shenoy <ego@linux.vnet.ibm.com>,
	linux-kernel@vger.kernel.org
Subject: Re: Fw: Re: [mm PATCH 4/6] RCU: (now) CPU hotplug
Date: Mon, 29 Jan 2007 20:12:41 +0100	[thread overview]
Message-ID: <20070129191241.GB14353@elte.hu> (raw)
In-Reply-To: <20070128153005.f320b834.akpm@osdl.org>


* Andrew Morton <akpm@osdl.org> wrote:

> > The idea being to essentially suspend the system to RAM, remove the 
> > CPU and then unsuspend it?  Seems like quite high overhead -- or am 
> > I misunderstanding the proposal?
> 
> The process freezer basically wakes up all threads in the machine and 
> makes them go to sleep in a specific place, so they're all in a known 
> state. kernel threads are also captured, via their open-coded polling 
> call to try_to_freeze().
> 
> The machine suspend code uses the process freezer, as does kprobes.  
> The freezer isn't tied to suspend or to power management.
> 
> The freezer does have potential to be expensive if used frequently and 
> if there are many threads.  But I don't think anyone has looked at 
> optimising it. [...]

i've measured it and it takes a few milliseconds typically - certainly 
not noticeable even for hypothetical highly scripted CPU-hotplug 
environments. (i doubt they really exist in practice)

in fact (new) kprobes uses the freezer, and it's far more performance 
sensitive than the handling of CPU hotplug events.

And there was almost no effort put into making the freezer fast, i'm 
sure if it ever becomes a problem we could improve it quite drastically. 
And that effort would improve suspend performance and kprobes 
performance as well - while the cpu-hotplug locking hell is just an 
unneeded distraction nobody really can deal with properly. So i say lets 
ditch CPU hotplug locking completely and lets go for the dead-simple 
freezer based design ...

	Ingo

  parent reply	other threads:[~2007-01-29 19:14 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20070124011519.GG1613@linux.vnet.ibm.com>
     [not found] ` <20070124090111.GC27221@in.ibm.com>
     [not found]   ` <20070124161559.GA1762@linux.vnet.ibm.com>
     [not found]     ` <20070124210645.GA19650@in.ibm.com>
2007-01-26 19:11       ` Dipankar Sarma
2007-01-26 19:28         ` Andrew Morton
2007-01-26 19:46           ` Dipankar Sarma
2007-01-26 20:17             ` Andrew Morton
2007-01-26 20:44               ` Dipankar Sarma
2007-01-26 21:29                 ` Andrew Morton
2007-01-28 22:47                   ` Paul E. McKenney
2007-01-28 23:30                     ` Andrew Morton
2007-01-29  2:40                       ` Paul E. McKenney
2007-01-29 19:12                       ` Ingo Molnar [this message]
2007-01-30  2:45                         ` Paul E. McKenney
2007-01-30  7:33                           ` Ingo Molnar
2007-01-30 16:02                             ` Paul E. McKenney
2007-01-30 16:44                               ` Rafael J. Wysocki
2007-01-30 18:27                                 ` Andrew Morton
2007-01-30 19:49                                   ` Paul E. McKenney
2007-01-31 23:10                                     ` Paul E. McKenney
2007-02-03  0:17                                       ` Pavel Machek
2007-02-04  4:39                                         ` Paul E. McKenney
2007-02-04 11:08                                           ` Rafael J. Wysocki
2007-02-04 12:53                                             ` Pavel Machek
2007-02-04 13:46                                               ` Rafael J. Wysocki
2007-02-04 13:50                                                 ` Pavel Machek
2007-02-04 13:59                                                   ` Rafael J. Wysocki
2007-02-03  0:01                                 ` Pavel Machek
2007-02-03 22:27                                   ` Rafael J. Wysocki
2007-02-04  0:31                                     ` Paul E. McKenney
2007-01-30 14:02                           ` Gautham R Shenoy
2007-01-30 16:47                             ` Paul E. McKenney

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=20070129191241.GB14353@elte.hu \
    --to=mingo@elte.hu \
    --cc=akpm@osdl.org \
    --cc=dipankar@in.ibm.com \
    --cc=ego@linux.vnet.ibm.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=paulmck@linux.vnet.ibm.com \
    /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

all inboxes | Powered by JetHome®