From: "Ville Syrjälä" <syrjala@sci.fi>
To: Eric Piel <Eric.Piel@tremplin-utc.net>
Cc: davej@redhat.com, Dominik Brodowski <linux@dominikbrodowski.net>,
CPUFreq Mailing List <cpufreq@lists.linux.org.uk>
Subject: Re: [PATCH 2/2 updated] Measure transition latency at driver initialization
Date: Fri, 2 Dec 2005 06:38:02 +0200 [thread overview]
Message-ID: <20051202043802.GA11942@sci.fi> (raw)
In-Reply-To: <438F659B.3070103@tremplin-utc.net>
On Thu, Dec 01, 2005 at 10:05:31PM +0100, Eric Piel wrote:
> 01.12.2005 20:31, Mattia Dongili wrote/a écrit:
> >ok, updated patch attached (with a shiny diffstat!!). Looks a lot
> >nicer now :)
> >
> >Oh, what do you think of the request to report out-of-range frequencies
> >and set CPUFREQ_ETERNAL instead of setting an arbitrary high value? If
> >ok, shall I use my email address instead of the list's?
> >
> I was about to say this is fairly reasonable... until I tried it on my
> laptop and notice that _I_ got this message :-( Transition measured:
> 4500 nSec. It's probably because the clock source on my laptop is the
> TSC, which on this old processor stops during transition. Therefore,
> after being in the shoe of the poor user still not having the ondemand
> governor working, I'd tend to prefer if the fallback transition latency
> could be a high and safe, but still usable, value. Anyway, we _know_
> that those processors are not so slugish to do transition that it should
> be considered "eternal" :-)
>
> So what about putting it to 500 uSec, knowing that on my 1GHz PIII it
> took something like 200uSec, it should be safe everywhere:
I'm going to jump in here with some measurements I got with
do_gettimeofday().
speedstep-ich on my 933 MHz Tualatin seems to agree with the 200 usec
figure.
speedstep-smi takes ~1670 usec on my 700 MHz Coppermine. That is when it
succeeds to change the frequency on the first try. That fails quite
often and with one retry the time goes up to ~72 msec (thanks to the
mdelay() in the loop). It will try to change the frequency six times
before giving up and at that point it may have wasted over one second.
Isn't that a "bit" too long to spend with irqs disabled?
--
Ville Syrjälä
syrjala@sci.fi
http://www.sci.fi/~syrjala/
next prev parent reply other threads:[~2005-12-02 4:38 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-11-29 23:58 [RFC][PATCH 0/0] measure speedstep-ich transition latency at CPU initialization Mattia Dongili
2005-11-29 23:58 ` [PATCH 1/2] Move PMBASE reading away and do it only once at initialization time Mattia Dongili
2005-11-29 23:58 ` [PATCH 2/2] Measure transition latency at driver initialization Mattia Dongili
2005-11-30 11:46 ` Eric Piel
2005-11-30 22:30 ` Mattia Dongili
2005-11-30 23:41 ` Eric Piel
2005-12-01 19:31 ` [PATCH 2/2 updated] " Mattia Dongili
2005-12-01 21:05 ` Eric Piel
2005-12-01 23:34 ` Mattia Dongili
2005-12-02 11:37 ` Eric Piel
2005-12-02 13:34 ` Mattia Dongili
2005-12-02 20:59 ` Mattia Dongili
2005-12-02 23:43 ` Eric Piel
2005-12-04 17:00 ` Dominik Brodowski
2005-12-02 4:38 ` Ville Syrjälä [this message]
2005-11-30 11:02 ` [PATCH 1/2] Move PMBASE reading away and do it only once at initialization time Eric Piel
2005-11-30 21:00 ` Mattia Dongili
2005-12-04 16:58 ` Dominik Brodowski
-- strict thread matches above, loose matches on Subject: below --
2005-12-03 0:50 [PATCH 2/2 updated] Measure transition latency at driver initialization Pallipadi, Venkatesh
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=20051202043802.GA11942@sci.fi \
--to=syrjala@sci.fi \
--cc=Eric.Piel@tremplin-utc.net \
--cc=cpufreq@lists.linux.org.uk \
--cc=davej@redhat.com \
--cc=linux@dominikbrodowski.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.