From: Feng Tang <feng.tang@intel.com>
To: "Paul E. McKenney" <paulmck@kernel.org>
Cc: Thomas Gleixner <tglx@linutronix.de>,
<linux-kernel@vger.kernel.org>, <sboyd@kernel.org>,
<corbet@lwn.net>, <Mark.Rutland@arm.com>, <maz@kernel.org>,
<kernel-team@meta.com>, <neeraju@codeaurora.org>,
<ak@linux.intel.com>, <zhengjun.xing@intel.com>,
Waiman Long <longman@redhat.com>,
John Stultz <jstultz@google.com>
Subject: Re: [PATCH clocksource 5/6] clocksource: Suspend the watchdog temporarily when high read latency detected
Date: Thu, 12 Jan 2023 08:59:12 +0800 [thread overview]
Message-ID: <Y79bYIAfZ8sBjvUs@feng-clx> (raw)
In-Reply-To: <20230111213210.GA4028633@paulmck-ThinkPad-P17-Gen-1>
On Wed, Jan 11, 2023 at 01:32:10PM -0800, Paul E. McKenney wrote:
> On Wed, Jan 11, 2023 at 10:19:50PM +0100, Thomas Gleixner wrote:
> > On Wed, Jan 11 2023 at 09:50, Paul E. McKenney wrote:
> > > On Wed, Jan 11, 2023 at 12:26:58PM +0100, Thomas Gleixner wrote:
> > > Yes, if a system was 100% busy forever, this patch would suppress these
> > > checks. But 100% busy forever is not the common case, due to thermal
> > > throttling and to security updates if nothing else.
> > >
> > > With all that said, is there a better way to get the desired effects of
> > > this patch?
> >
> > Sane hardware?
>
> I must let Feng talk to his systems, but most of the systems I saw were
> production systems. A few were engineering samples, from which some
> insanity might be expected behavior.
I've tested with several generations of Xeon servers, and they all
can reproduce the issue with stress-ng stress load. Those platforms
are not bought from market :), but they have latest stepping and
firmware, which are close to production systesm.
The issue originally came from customer, and there were engineers
who reproduced it on production systems(even from different vendors)
Thanks,
Feng
> Clearly, something about the hardware or firmware was insane in order
> to get this result, but that is what diagnostics are for, even on
> engineering samples.
>
> Thanx, Paul
next prev parent reply other threads:[~2023-01-12 1:02 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-01-05 1:04 [PATCH clocksource 0/6] Clocksource watchdog updates for v6.3 Paul E. McKenney
2023-01-05 1:06 ` [PATCH clocksource 1/6] clocksource: Print clocksource name when clocksource is tested unstable Paul E. McKenney
2023-01-05 1:06 ` [PATCH clocksource 2/6] clocksource: Loosen clocksource watchdog constraints Paul E. McKenney
2023-01-05 1:06 ` [PATCH clocksource 3/6] clocksource: Improve read-back-delay message Paul E. McKenney
2023-01-05 1:06 ` [PATCH clocksource 4/6] clocksource: Improve "skew is too large" messages Paul E. McKenney
2023-01-05 1:07 ` [PATCH clocksource 5/6] clocksource: Suspend the watchdog temporarily when high read latency detected Paul E. McKenney
2023-01-11 11:26 ` Thomas Gleixner
2023-01-11 12:31 ` Feng Tang
2023-01-11 17:50 ` Paul E. McKenney
2023-01-11 21:19 ` Thomas Gleixner
2023-01-11 21:32 ` Paul E. McKenney
2023-01-12 0:59 ` Feng Tang [this message]
2023-01-05 1:07 ` [PATCH clocksource 6/6] clocksource: Verify HPET and PMTMR when TSC unverified Paul E. McKenney
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=Y79bYIAfZ8sBjvUs@feng-clx \
--to=feng.tang@intel.com \
--cc=Mark.Rutland@arm.com \
--cc=ak@linux.intel.com \
--cc=corbet@lwn.net \
--cc=jstultz@google.com \
--cc=kernel-team@meta.com \
--cc=linux-kernel@vger.kernel.org \
--cc=longman@redhat.com \
--cc=maz@kernel.org \
--cc=neeraju@codeaurora.org \
--cc=paulmck@kernel.org \
--cc=sboyd@kernel.org \
--cc=tglx@linutronix.de \
--cc=zhengjun.xing@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