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 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.