From: Linus Torvalds <torvalds@linux-foundation.org>
To: Kyle Moffett <mrmacman_g4@mac.com>
Cc: Bill Davidsen <davidsen@tmr.com>,
linux-kernel@vger.kernel.org,
Davide Libenzi <davidel@xmailserver.org>,
"Eric W. Biederman" <ebiederm@xmission.com>,
Oleg Nesterov <oleg@tv-sign.ru>, Robin Holt <holt@sgi.com>,
Chris Snook <csnook@redhat.com>,
Jack Steiner <steiner@americas.sgi.com>
Subject: Re: init's children list is long and slows reaping children.
Date: Mon, 9 Apr 2007 12:51:34 -0700 (PDT) [thread overview]
Message-ID: <Pine.LNX.4.64.0704091239030.6730@woody.linux-foundation.org> (raw)
In-Reply-To: <9B6136A3-52B4-4AE2-9805-21634E25530A@mac.com>
On Mon, 9 Apr 2007, Kyle Moffett wrote:
>
> Maybe "struct posix_process" is more descriptive? "struct process_posix"?
> "Ugly POSIX process semantics data" seems simple enough to stick in a struct
> name. "struct uglyposix_process"?
Guys, you didn't read my message.
It's *not* about "process" stuff. Anything that tries to call it a
"process" is *by*definition* worse than what it is now. Processes have all
the things that we've cleanly separated out for filesystem, VM, SysV
semaphore state, namespaces etc.
The "struct signal_struct" is the random *leftovers* from all the other
stuff. It's *not* about "processes". Never has been, and never will be.
It's mainly about signal cruft, because that's the nastiest part of POSIX
threads behaviour, and that can't be clearly separated as one clear
structure.
So
- it really *is* mostly about signal handling and signal sources.
- it has some random *cruft* in it that isn't about signals, but even
that is mostly a matter of "it was random cruft in the original task
structure too, and it DID NOT MERIT a flag of its own"
- if you wanted to clean things up, you'd actually make things like
the "rlimit" info structures of their own, and have pointers to them.
So that cruft largely got put into "signal_struct" just because they were
the last thing to be moved out, along with the signal stuff (which was the
big and painful part). NOT because "struct signal_struct" is somehow about
"process state".
So stop blathering about processes. It has *nothing* to do with processes.
It's primarily about signals, but it has "cruft" in it.
So an accurate name is
struct signal_struct_with_some_cruft_in_it_that_did_not_merit_a_struct_of_its_own
but that's actually fairly unwieldly to type, and so in the name of sanity
and clear source code, it's called
struct signal_struct
and that's it.
And people who have argued for renaming it don't even seem to understand
what it's *about*, so the arguments for renaming it have been very weak
indeed so far.
IT IS NOT ABOUT "PROCESSES".
To be a "posix process", you have to share *everything*. The signal-struct
isn't even a very important part of that sharing. In fact, it's quite
arguably the *least* important part to share, which is why it was
separated out as a separate thing absolutely *last*, and which is why we
could do without it for quite long, with just some annoyingly subtle
non-POSIX semantics.
Get it? It's the structure that is *least* important for a "process".
So stop blathering about "processes". Anybody who does that, just shows
that they haven't thought it through!
Linus
next prev parent reply other threads:[~2007-04-09 19:52 UTC|newest]
Thread overview: 112+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-04-05 19:51 Robin Holt
2007-04-05 20:57 ` Linus Torvalds
2007-04-06 0:51 ` Chris Snook
2007-04-06 1:03 ` Chris Snook
2007-04-06 1:29 ` Linus Torvalds
2007-04-06 2:15 ` Eric W. Biederman
2007-04-06 10:43 ` Robin Holt
2007-04-06 15:38 ` Eric W. Biederman
2007-04-06 16:31 ` Oleg Nesterov
2007-04-06 17:32 ` Ingo Molnar
2007-04-06 17:39 ` Roland Dreier
2007-04-06 18:04 ` Eric W. Biederman
2007-04-06 18:30 ` Eric W. Biederman
2007-04-06 19:18 ` [patch] sched: get rid of p->children use in show_task() Ingo Molnar
2007-04-06 19:22 ` Ingo Molnar
2007-04-10 13:48 ` init's children list is long and slows reaping children Ingo Molnar
2007-04-10 13:38 ` Oleg Nesterov
2007-04-10 15:00 ` Eric W. Biederman
2007-04-10 15:16 ` [PATCH] Only send pdeath_signal when getppid changes Eric W. Biederman
2007-04-10 16:37 ` Oleg Nesterov
2007-04-10 17:41 ` Eric W. Biederman
2007-04-10 17:48 ` Roland McGrath
2007-04-11 3:17 ` Albert Cahalan
2007-04-10 14:51 ` init's children list is long and slows reaping children Eric W. Biederman
2007-04-10 15:06 ` Ingo Molnar
2007-04-10 15:22 ` Eric W. Biederman
2007-04-10 15:53 ` Ingo Molnar
2007-04-10 16:17 ` Eric W. Biederman
2007-04-11 6:20 ` [patch] uninline remove/add_parent() APIs Ingo Molnar
2007-04-11 7:00 ` Eric W. Biederman
2007-04-11 22:06 ` Andrew Morton
2007-04-12 10:45 ` Eric W. Biederman
2007-04-12 22:50 ` Roland McGrath
2007-04-10 16:44 ` init's children list is long and slows reaping children Oleg Nesterov
2007-04-11 19:55 ` Bill Davidsen
2007-04-11 20:17 ` Eric W. Biederman
2007-04-11 21:24 ` Bill Davidsen
2007-04-11 20:19 ` Oleg Nesterov
2007-04-06 18:02 ` Eric W. Biederman
2007-04-06 18:21 ` Davide Libenzi
2007-04-06 18:56 ` Eric W. Biederman
2007-04-06 19:16 ` Davide Libenzi
2007-04-06 19:19 ` Ingo Molnar
2007-04-06 21:29 ` Davide Libenzi
2007-04-06 21:51 ` Linus Torvalds
2007-04-06 22:31 ` Davide Libenzi
2007-04-06 22:46 ` Linus Torvalds
2007-04-06 22:59 ` Davide Libenzi
2007-04-09 8:28 ` Ingo Molnar
2007-04-09 18:09 ` Bill Davidsen
2007-04-09 19:28 ` Kyle Moffett
2007-04-09 19:51 ` Linus Torvalds [this message]
2007-04-09 20:03 ` Davide Libenzi
2007-04-10 15:12 ` Bill Davidsen
2007-04-10 19:17 ` Davide Libenzi
2007-04-09 20:00 ` Eric W. Biederman
2007-04-06 16:41 ` Robin Holt
2007-04-09 17:37 ` Chris Snook
2007-04-06 18:05 ` Christoph Hellwig
2007-04-06 19:39 ` Eric W. Biederman
2007-04-06 22:38 ` Jeff Garzik
2007-04-06 22:51 ` Linus Torvalds
2007-04-06 23:37 ` Jeff Garzik
2007-04-11 7:28 ` Nick Piggin
2007-04-10 0:23 ` Andrew Morton
2007-04-10 0:48 ` Eric W. Biederman
2007-04-10 1:15 ` Andrew Morton
2007-04-10 6:53 ` Jeff Garzik
2007-04-10 9:42 ` Robin Holt
2007-04-10 1:59 ` Dave Jones
2007-04-10 2:30 ` Andrew Morton
2007-04-10 2:46 ` Linus Torvalds
2007-04-10 7:07 ` Jeff Garzik
2007-04-10 22:20 ` Ingo Oeser
2007-04-10 5:07 ` Alexey Dobriyan
2007-04-10 5:21 ` Dave Jones
2007-04-10 6:09 ` Torsten Kaiser
2007-04-10 7:08 ` Jeff Garzik
2007-04-10 7:05 ` Jeff Garzik
2007-04-10 7:37 ` Andrew Morton
2007-04-10 8:33 ` Jeff Garzik
2007-04-10 8:41 ` Andrew Morton
2007-04-10 8:48 ` Jeff Garzik
2007-04-10 22:35 ` Ingo Oeser
2007-04-10 16:35 ` Matt Mackall
2007-04-10 7:44 ` Russell King
2007-04-10 8:16 ` Jeff Garzik
2007-04-10 8:59 ` Ingo Molnar
2007-04-10 9:33 ` Jeff Garzik
2007-04-06 8:42 Oleg Nesterov
2007-04-06 9:10 ` Eric W. Biederman
2007-04-06 9:44 ` Oleg Nesterov
2007-04-06 15:45 ` Eric W. Biederman
2007-04-06 15:47 ` Oleg Nesterov
2007-04-06 17:16 ` Linus Torvalds
2007-04-06 17:27 ` Ingo Molnar
2007-04-06 17:31 ` Ingo Molnar
2007-04-06 17:34 ` Eric W. Biederman
2007-04-06 19:06 ` H. Peter Anvin
2007-04-06 19:15 ` Eric W. Biederman
2007-04-06 19:21 ` H. Peter Anvin
2007-04-06 21:04 ` Jeremy Fitzhardinge
2007-04-06 21:07 ` H. Peter Anvin
2007-04-06 19:36 ` Oleg Nesterov
2007-04-06 19:43 ` Ingo Molnar
2007-04-06 20:01 ` Oleg Nesterov
2007-04-06 20:21 ` Ingo Molnar
2007-04-06 19:47 ` Oleg Nesterov
2007-04-06 19:59 ` Eric W. Biederman
2007-04-07 20:31 ` Oleg Nesterov
2007-04-08 0:38 ` Eric W. Biederman
2007-04-08 15:46 ` Oleg Nesterov
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=Pine.LNX.4.64.0704091239030.6730@woody.linux-foundation.org \
--to=torvalds@linux-foundation.org \
--cc=csnook@redhat.com \
--cc=davidel@xmailserver.org \
--cc=davidsen@tmr.com \
--cc=ebiederm@xmission.com \
--cc=holt@sgi.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mrmacman_g4@mac.com \
--cc=oleg@tv-sign.ru \
--cc=steiner@americas.sgi.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®