public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
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

  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