From: Rusty Russell <rusty@rustcorp.com.au>
To: "Jike Song" <albcamus@gmail.com>
Cc: "Max Krasnyansky" <maxk@qualcomm.com>,
"Andrew Morton" <akpm@linux-foundation.org>,
LKML <linux-kernel@vger.kernel.org>,
"Linus Torvalds" <torvalds@linux-foundation.org>
Subject: Re: Module loading/unloading and "The Stop Machine"
Date: Tue, 4 Mar 2008 14:51:09 +1100 [thread overview]
Message-ID: <200803041451.10312.rusty@rustcorp.com.au> (raw)
In-Reply-To: <df9815e70802132102v7eb7d8e9k4d6b21ffa6325d0d@mail.gmail.com>
On Thursday 14 February 2008 16:02:06 Jike Song wrote:
> On 2/8/08, Max Krasnyansky <maxk@qualcomm.com> wrote:
> > Hi Rusty,
> >
> > I was hopping you could answer a couple of questions about module
> > loading/unloading and the stop machine.
>
> I'm curious to know why it is called `stop machine', which is a queer
> name without any relationship with its function.
Hi Jike,
I'm surprised.. stunned even, to hear you say this. stop_machine stops the
machine: every CPU is frozen with interrupts disabled. I am having trouble
thinking of a better name.
"schedule_a_thread_on_every_cpu_and_then_disable_interrupts" is a little
long, and describes the implementation not the effect.
"run_and_make_sure_no_cpus_have_preempt_disabled" also fails the "will Linus
flame me?" test, I think.
"freeze_machine" has connotations of the freezer, which is completely
different from this. "do_machine" is shorter, but sounds vaguely naughty.
> Regards,
> Jike
Perhaps you were angling for "jike_machine"? It's kind of catchy, but I don't
think that patch will be accepted even if you pushed it as a git tree.
Cheers,
Rusty.
prev parent reply other threads:[~2008-03-04 3:52 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-02-08 2:38 Module loading/unloading and "The Stop Machine" Max Krasnyansky
2008-02-08 23:11 ` Max Krasnyanskiy
2008-02-14 4:02 ` Tejun Heo
2008-02-22 1:24 ` Max Krasnyanskiy
2008-02-22 1:38 ` Tejun Heo
2008-02-22 1:47 ` Max Krasnyanskiy
2008-02-22 1:59 ` Tejun Heo
2008-02-22 2:16 ` Max Krasnyanskiy
2008-02-22 11:53 ` Andi Kleen
2008-02-22 22:41 ` Max Krasnyanskiy
2008-03-04 1:21 ` Rusty Russell
2008-02-14 5:02 ` Jike Song
2008-02-14 5:38 ` Tejun Heo
2008-03-04 3:51 ` Rusty Russell [this message]
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=200803041451.10312.rusty@rustcorp.com.au \
--to=rusty@rustcorp.com.au \
--cc=akpm@linux-foundation.org \
--cc=albcamus@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=maxk@qualcomm.com \
--cc=torvalds@linux-foundation.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.