From: Dave Jones <davej@redhat.com>
To: Linus Torvalds <torvalds@osdl.org>
Cc: Linux Kernel <linux-kernel@vger.kernel.org>,
Andrew Morton <akpm@osdl.org>, Gautham Shenoy <ego@in.ibm.com>
Subject: Re: Remove hotplug cpu crap from cpufreq.
Date: Wed, 1 Nov 2006 18:32:50 -0500 [thread overview]
Message-ID: <20061101233250.GA17706@redhat.com> (raw)
In-Reply-To: <Pine.LNX.4.64.0611011507480.25218@g5.osdl.org>
On Wed, Nov 01, 2006 at 03:09:52PM -0800, Linus Torvalds wrote:
> Hmm. People _have_ given a damn, and I think you were even cc'd.
You're right. In my defense, that stuff arrived the day I went
on vacation for two weeks, and I subsequently forgot all about it.
Looking back over that thread though, a few people seemed to pick a
number of holes in the patches, and there are some real gems in that
thread like.
> Really, the hotplug locking rules are fairly simple-
>
> 1. If you are in cpu hotplug callback path, don't take any lock.
Which is just great, as afair, the cpufreq locks were there _before_
someone liberally sprinkled lock_cpu_hotplug() everywhere.
> Right now, for 2.6.19, I'd prefer to not touch that mess unless there are
> known conditions that actually cause more problems than just stupid
> warnings..
>From what I can tell from looking at that thread back in August,
it went on for a while with a number of people picking holes in the
proposed patches, but there wasn't any reposted after that, and
certainly nothing that ended up in -mm.
_something_ needs to be done. If someone wants to fix it, great, but
until we see something mergable, we're left in this half-assed state
which is freaking people out.
Dave
--
http://www.codemonkey.org.uk
next prev parent reply other threads:[~2006-11-01 23:33 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-11-01 22:59 Remove hotplug cpu crap from cpufreq Dave Jones
2006-11-01 23:09 ` Rafael J. Wysocki
2006-11-01 23:21 ` Linus Torvalds
2006-11-02 7:33 ` Martin Lorenz
2006-11-01 23:09 ` Linus Torvalds
2006-11-01 23:32 ` Dave Jones [this message]
2006-11-02 18:46 ` Dipankar Sarma
2006-11-02 0:17 ` Andrew Morton
2006-11-02 4:27 ` Gautham R Shenoy
2006-11-02 19:04 ` Dipankar Sarma
2006-11-02 19:30 ` Linus Torvalds
2006-11-02 21:40 ` Dipankar Sarma
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=20061101233250.GA17706@redhat.com \
--to=davej@redhat.com \
--cc=akpm@osdl.org \
--cc=ego@in.ibm.com \
--cc=linux-kernel@vger.kernel.org \
--cc=torvalds@osdl.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