From: Pavel Machek <pavel@ucw.cz>
To: Peter Zijlstra <peterz@infradead.org>
Cc: Christian Ehrhardt <ehrhardt@linux.vnet.ibm.com>,
Ingo Molnar <mingo@elte.hu>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
Holger.Wolf@de.ibm.com, epasch@de.ibm.com,
Martin Schwidefsky <schwidefsky@de.ibm.com>
Subject: Re: Missing recalculation of scheduler tunables in case of cpu hot add/remove
Date: Thu, 3 Dec 2009 10:12:06 +0100 [thread overview]
Message-ID: <20091203091206.GA1478@ucw.cz> (raw)
In-Reply-To: <1259253950.31676.249.camel@laptop>
Hi!
> > >>> Aside from that, we probably should put an upper limit in place, as I
> > >>> guess large cpu count machines get silly large values
> > >>>
> > >> I agree to that, but in the code is already an upper limit of
> > >> 200.000.000 - well we might discuss if that is too low/high.
> > >>
> > >
> > > Yeah, I think we should cap it around the 8-16 CPUs.
> > >
> > >
> > ok for me, driven by that finding I think I have to measure different
> > kind of scalings anyway, but as usually that takes some time :-/
> > At least too time much for the discussion & solution of that bug I guess.
> >
> > The question for now is what we do on cpu hot add/remove?
> > Would hooking somewhere in kernel/cpu.c be the right approach - I'm not
> > quite sure about my own suggestion yet :-).
>
> Something like the below might work I suppose, just needs a cleanup and
> such.
I see a rather fundamental problem: what if user wants to override
those values, and wants them stay that way?
If you do this, suspend/resume will put the old values back AFAICT.
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
next prev parent reply other threads:[~2009-12-03 9:12 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-11-26 16:10 Missing recalculation of scheduler tunables in case of cpu hot add/remove Christian Ehrhardt
2009-11-26 16:19 ` Peter Zijlstra
2009-11-26 16:25 ` Christian Ehrhardt
2009-11-26 16:28 ` Peter Zijlstra
2009-11-26 16:31 ` Christian Ehrhardt
2009-11-26 16:45 ` Peter Zijlstra
2009-11-26 18:39 ` Christian Ehrhardt
2009-11-26 18:53 ` Peter Zijlstra
2009-12-03 9:12 ` Pavel Machek [this message]
2009-12-03 9:31 ` Christian Ehrhardt
2009-11-26 16:22 ` Peter Zijlstra
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=20091203091206.GA1478@ucw.cz \
--to=pavel@ucw.cz \
--cc=Holger.Wolf@de.ibm.com \
--cc=ehrhardt@linux.vnet.ibm.com \
--cc=epasch@de.ibm.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=peterz@infradead.org \
--cc=schwidefsky@de.ibm.com \
/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.