All of lore.kernel.org
 help / color / mirror / Atom feed
From: William Lee Irwin III <wli@holomorphy.com>
To: Simon Derr <Simon.Derr@bull.net>
Cc: Stephen Hemminger <shemminger@osdl.org>,
	Sylvain Jeaugey <sylvain.jeaugey@bull.net>,
	linux-kernel@vger.kernel.org
Subject: Re: (1/4) [PATCH] cpuset -- 2.6.0-test8
Date: Wed, 22 Oct 2003 07:57:11 -0700	[thread overview]
Message-ID: <20031022145711.GC1108@holomorphy.com> (raw)
In-Reply-To: <Pine.A41.4.53.0310221620420.139942@isabelle.frec.bull.fr>

On Tue, 21 Oct 2003, William Lee Irwin III wrote:
>> Best not to insist NR_CPUS % BITS_PER_LONG == 0.

On Wed, Oct 22, 2003 at 04:34:21PM +0200, Simon Derr wrote:
> Actually we don't, but you're right, NR_CPUS should definately be used
> here.

The insistence mentioned was implicit, of course.


On Tue, 21 Oct 2003, William Lee Irwin III wrote:
>> Unfair rwlocks can take boxen out when abused by quadratic algorithms.

On Wed, Oct 22, 2003 at 04:34:21PM +0200, Simon Derr wrote:
> I don't see exactly which lock you are talking about here ?
> Anyway, the current state of the cpusets is OK for a 'gentle' use. I'm
> sure some improvements are needed to protect it from 'evil' users ;-)

tasklist_lock, infamous for setting off the NMI oopser (i.e. watchdogs
that panic machines when interrupts are ignored for long periods of
time) when users run many instances of top(1) or other nonsense. Not
a performance concern per se, but it may very well be hopeless anyway.

There are several threads about the rwlock algorithms being unfair and
taking out machines and/or persistently starving writers. Probably the
most recent one was started by Kevin van Maren. There Linus suggested
eventually adding a fair rwlock primitive to address this.


-- wli

  reply	other threads:[~2003-10-22 14:57 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-10-13 13:07 [RFC] cpuset proposal Simon Derr
2003-10-21 23:20 ` (1/4) [PATCH] cpuset -- 2.6.0-test8 Stephen Hemminger
2003-10-22  0:08   ` William Lee Irwin III
2003-10-22 14:34     ` Simon Derr
2003-10-22 14:57       ` William Lee Irwin III [this message]
2003-10-21 23:20 ` (2/4) [PATCH] cpuset - Kconfig Stephen Hemminger
2003-10-21 23:20 ` (3/4) [PATCH] cpuset - build without CPUSET configured Stephen Hemminger
2003-10-21 23:20 ` (4/4) [PATCH] cpuset -- seqfile change Stephen Hemminger

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=20031022145711.GC1108@holomorphy.com \
    --to=wli@holomorphy.com \
    --cc=Simon.Derr@bull.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=shemminger@osdl.org \
    --cc=sylvain.jeaugey@bull.net \
    /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.