From: Ingo Molnar <mingo@kernel.org>
To: Mike Galbraith <efault@gmx.de>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Peter Zijlstra <a.p.zijlstra@chello.nl>,
Thomas Gleixner <tglx@linutronix.de>,
Andrew Morton <akpm@linux-foundation.org>
Subject: Re: [RFC GIT PULL] scheduler fix for autogroups
Date: Sun, 2 Dec 2012 20:27:33 +0100 [thread overview]
Message-ID: <20121202192733.GA19824@gmail.com> (raw)
In-Reply-To: <1354436468.7048.35.camel@marge.simpson.net>
* Mike Galbraith <efault@gmx.de> wrote:
> On Sat, 2012-12-01 at 22:44 +0100, Ingo Molnar wrote:
>
> > Should we use some other file for that - or no file at all and
> > just emit a bootup printk for kernel hackers with a short
> > attention span?
>
> Or, whack the file and don't bother with a printk either. If
> it's in your config, and your command line doesn't contain
> noautogroup, it's on, so the info is already present (until
> buffer gets full). That makes for even fewer lines dedicated
> to dinky sideline feature.
>
> Or (as previously mentioned) just depreciate (or rip out) the
> whole thing since systemd is propagating everywhere anyway,
> and offers the same functionality.
>
> For 3.7, a revert of 800d4d30c8f2 would prevent the explosion
> when folks play with the now non-functional on/off switch
> (task groups are required to _always_ exist, that commit
> busted the autogroup assumption), so is perhaps a viable
> quickfix until autogroups fate is decided?
Linus, which one would be your preference? I'm fine with the
first and third options - #2 that rips it all out looks like
a sad removal of an otherwise useful feature.
( The fourth option would be to fix the dynamic knobs - there's
no patch for that yet. )
Thanks,
Ingo
next prev parent reply other threads:[~2012-12-02 19:27 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-12-01 11:16 Ingo Molnar
2012-12-01 21:16 ` Linus Torvalds
2012-12-01 21:44 ` Ingo Molnar
2012-12-01 21:47 ` Ingo Molnar
2012-12-01 22:03 ` Linus Torvalds
2012-12-02 7:34 ` Mike Galbraith
2012-12-02 8:21 ` Mike Galbraith
2012-12-02 19:27 ` Ingo Molnar [this message]
2012-12-02 19:36 ` Linus Torvalds
2012-12-03 5:25 ` [patch] " Mike Galbraith
2012-12-03 5:36 ` Ingo Molnar
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=20121202192733.GA19824@gmail.com \
--to=mingo@kernel.org \
--cc=a.p.zijlstra@chello.nl \
--cc=akpm@linux-foundation.org \
--cc=efault@gmx.de \
--cc=linux-kernel@vger.kernel.org \
--cc=tglx@linutronix.de \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Powered by JetHome