linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Feng Tang <feng.tang@intel.com>
To: "Paul E. McKenney" <paulmck@kernel.org>
Cc: Xing Zhengjun <zhengjun.xing@linux.intel.com>,
	Thomas Gleixner <tglx@linutronix.de>,
	John Stultz <john.stultz@linaro.org>,
	Stephen Boyd <sboyd@kernel.org>, Jonathan Corbet <corbet@lwn.net>,
	Mark Rutland <Mark.Rutland@arm.com>,
	Marc Zyngier <maz@kernel.org>, Andi Kleen <ak@linux.intel.com>,
	Chris Mason <clm@fb.com>, LKML <linux-kernel@vger.kernel.org>,
	lkp@lists.01.org, lkp@intel.com
Subject: Re: [LKP] Re: [clocksource] 6c52b5f3cf: stress-ng.opcode.ops_per_sec -14.4% regression
Date: Sat, 24 Apr 2021 20:29:20 +0800	[thread overview]
Message-ID: <20210424122920.GB85095@shbuild999.sh.intel.com> (raw)
In-Reply-To: <20210423140254.GM975577@paulmck-ThinkPad-P17-Gen-1>

Hi Paul,

On Fri, Apr 23, 2021 at 07:02:54AM -0700, Paul E. McKenney wrote:
[...]
> > > > > Following is the tsc freq info from kernel log
> > > > > 
> > > > > [    0.000000] DMI: Intel Corporation S2600WFT/S2600WFT, BIOS SE5C620.86B.02.01.0008.031920191559 03/19/2019
> > > > > [    0.000000] tsc: Detected 2100.000 MHz processor
> > > > > ...
> > > > > [   13.859982] tsc: Refined TSC clocksource calibration: 2095.077 MHz
> > > > 
> > > > So what are our options?
> > > > 
> > > > 1.	Clear CLOCK_SOURCE_MUST_VERIFY from tsc-early.
> > > > 
> > 
> > I think option 1 is fine, as tsc will still get checked once 'tsc'
> > clocksource is registered, but Thomas and Peter should know more
> > background and corner cases of tsc.
> 
> I will look at adding such a patch to my series, preceding the change
> to 1/1000 deviation.
> 
> > Also we have been working on another patchset to skip watchdog check
> > for x86 platforms with stable tsc:
> > 
> > https://lore.kernel.org/lkml/1618291897-71581-1-git-send-email-feng.tang@intel.com/
> > https://lore.kernel.org/lkml/1618291897-71581-2-git-send-email-feng.tang@intel.com/
> 
> It will be interesting to see what fraction of those with large numbers
> of systems choose to revert your 2/2, and for what period of time.
> You really needed my clocksource patch series to have been in place some
> years back so that people wouldn't have been seeing the false-postive
> clock-skew complaints.  Those complaints did not help people build up
> their trust in the TSC.  :-/

I read you patchset, and I think the recheck to avoid false alarm makes
sense to me, as well as the debug method you adds, and they have no
conflict with my patches which tends to newer x86 platforms only.

And yes, I only have met and debugged tsc wrongly marked unstable cases
on several clients platforms, and in one case I disabled the HPET for
Baytrail 10 years ago. Our test farm has different kinds of servers,
only up to 4 sockets and 192 CPUs, where no tsc unstable issue has been
seen.

And I'm eager to know if there is any real case of an unreliable tsc
on the 'large numbers' of x86 system which complies with our cpu feature
check. And if there is, my 2/2 definitely should be dropped.   

> This last sentence is not a theoretical statement.  In the past, I have
> suggested using the existing "tsc=reliable" kernel boot parameter,
> which disables watchdogs on TSC, similar to your patch 2/2 above.
> The discussion was short and that boot parameter was not set.  And the
> discussion motivated to my current clocksource series.  ;-)
> 
> I therefore suspect that someone will want a "tsc=unreliable" boot
> parameter (or similar) to go with your patch 2/2.
 
Possibly :)

But I wonder if tsc is disabled on that 'large system', what will be
used instead? HPET is known to be much slower for clocksource, as shown
in this regression report :) not mentioning the 'acpi_pm' timer. 

Again, I want to know the real tsc unstable case. I have spent lots
of time searching these info from git logs and mail archives before
writing the patches.

Thanks,
Feng

> 							Thanx, Paul

  reply	other threads:[~2021-04-24 12:29 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-20  6:49 [clocksource] 6c52b5f3cf: stress-ng.opcode.ops_per_sec -14.4% regression kernel test robot
2021-04-20 13:43 ` Paul E. McKenney
2021-04-20 14:05   ` Paul E. McKenney
2021-04-21  6:07     ` [LKP] " Xing, Zhengjun
2021-04-21 13:42       ` Paul E. McKenney
2021-04-22  6:58         ` Xing Zhengjun
2021-04-22  7:41           ` Feng Tang
2021-04-22 14:24             ` Paul E. McKenney
2021-04-22 16:57               ` Paul E. McKenney
2021-04-23  6:11                 ` Feng Tang
2021-04-23 14:02                   ` Paul E. McKenney
2021-04-24 12:29                     ` Feng Tang [this message]
2021-04-24 17:53                       ` Paul E. McKenney
2021-04-25  2:14                         ` Feng Tang
2021-04-25  3:14                           ` Feng Tang
2021-04-25 19:15                             ` Paul E. McKenney
2021-04-25 19:14                           ` Paul E. McKenney
2021-04-26 12:39                       ` Thomas Gleixner
2021-04-26 14:05                         ` Feng Tang
2021-04-26 14:33                           ` Thomas Gleixner
2021-04-26 15:12                             ` Feng Tang
2021-04-23  2:15               ` Xing Zhengjun
2021-04-23  4:12                 ` Paul E. McKenney
2021-04-23 19:14               ` Thomas Gleixner
2021-04-23 21:14                 ` Paul E. McKenney
2021-04-23 23:39                   ` Paul E. McKenney
2021-04-23 19:09             ` Thomas Gleixner

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=20210424122920.GB85095@shbuild999.sh.intel.com \
    --to=feng.tang@intel.com \
    --cc=Mark.Rutland@arm.com \
    --cc=ak@linux.intel.com \
    --cc=clm@fb.com \
    --cc=corbet@lwn.net \
    --cc=john.stultz@linaro.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lkp@intel.com \
    --cc=lkp@lists.01.org \
    --cc=maz@kernel.org \
    --cc=paulmck@kernel.org \
    --cc=sboyd@kernel.org \
    --cc=tglx@linutronix.de \
    --cc=zhengjun.xing@linux.intel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).