Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   "Sun, Yunying" <yunying.sun@intel.com>
To:     "paulmck@kernel.org" <paulmck@kernel.org>
CC:     "tglx@linutronix.de" <tglx@linutronix.de>,
        "peterz@infradead.org" <peterz@infradead.org>,
        "longman@redhat.com" <longman@redhat.com>,
        "x86@kernel.org" <x86@kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "Tang, Feng" <feng.tang@intel.com>
Subject: RE: [PATCH] clocksource: Print clocksource name when clocksource is
 tested unstable
Thread-Topic: [PATCH] clocksource: Print clocksource name when clocksource is
 tested unstable
Thread-Index: AQHY+ZJ1xXsis4rp9kmS/pgu4XQ8G65CAZCAgABMr6A=
Date:   Thu, 17 Nov 2022 01:12:28 +0000
Message-ID: <DM6PR11MB40924705F72403D0E7B534C6F2069@DM6PR11MB4092.namprd11.prod.outlook.com>
References: <20221116082221.25243-1-yunying.sun@intel.com>
 <20221116203107.GX4001@paulmck-ThinkPad-P17-Gen-1>
In-Reply-To: <20221116203107.GX4001@paulmck-ThinkPad-P17-Gen-1>
Accept-Language: en-US
Content-Language: en-US
dlp-version: 11.6.500.17
dlp-product: dlpe-windows
dlp-reaction: no-action
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?INcYnLqYY5sBB9lS0m7Be4KWlrKT3sDPI3N7grDn0g/XKuOBuX/yRNrBqe7x?=
 =?us-ascii?Q?xdskkgXArKcY/4DGDr6svgCOiOuHKCcsrMHeT59hkJFQqrSDWdYzWzBfzb4y?=
 =?us-ascii?Q?wI3HGmzmSJ1l33xI4CqE3H7van9jQBNPIsE5PKo0T/hWyMOWImQKH9x8kS7l?=
 =?us-ascii?Q?Zukc7O2MfWr1wpnAZFGN/9my6QCZ4lEdh4Ba6D+tzLDYYsuyp/1Cjf+Wc8aq?=
 =?us-ascii?Q?6YG9q62WArjCT2G5A7qAFI98C2nxgJFYW2SjSZEYvgSY56l4xpNyL0I1u6Io?=
 =?us-ascii?Q?U7dGY+xIh6hOPkSkhJchxQIA0/Zpj1kxNZ3GetH3dxq7C7WIIT68DuBCCErg?=
 =?us-ascii?Q?yi+m0BJuy8+FQgXH6AHASkhbqigKbVnF2V5284WVXiZBdYnPinM1cKvASYyU?=
 =?us-ascii?Q?Yc9x7YFBluaiyIuqby+1oHuDdVPvNOpsrUJXhGju3Aw7UzCDNFj1mzSHlg1Y?=
 =?us-ascii?Q?nnzBkmNACTqbsH781i7zNkXDSvUL6kkwP59KMTol6Dx0Z225buzdUm+yxvHH?=
 =?us-ascii?Q?1tFWYFWtllCH5uUuQuihycy+tV2V22vX/pxJhBQjka9kU2Kb+1H6AvrpYJ62?=
 =?us-ascii?Q?4LM5XN6nbsFc/sqqwRb+5xd6YHDAb36UhHX8dxXu0VD0RVNyCqSbKe/lQNgA?=
 =?us-ascii?Q?j0ZCp2osPBISA66f9hN8aGtNuDyUrhuBeAu07RRGMSQ665hnSyUo0NqGxZwE?=
 =?us-ascii?Q?9NysjUeiDGoGTBuyWdijTAHs+LDfyBf5xrF/D1dE6xVEJagZuGCq3DtpiI9B?=
 =?us-ascii?Q?vOrZkposWiOf5PwlLl65eXNffV7EmRGN/BTA7md4bkS6jsAki3wmXACP8b4P?=
 =?us-ascii?Q?4NjEF1C0TedqhCXUbzlOlaINwTn+rPazE02TQhjvWcLb7N0SeOZQb//Adbbg?=
 =?us-ascii?Q?iAVLbr8rk23bE5m/7afttUlhkPTGq+88QENiHWeO7F6bW8QJyp+yf+FLWP8/?=
 =?us-ascii?Q?Q5B0E7N6370Zr14oubJGjMJDXkUyHWT7UCD4kv+KUj05TXhGHuI1BpfwK9C2?=
 =?us-ascii?Q?hy0R+nXojm7eX9ZxF1wJHCocTNnB4JtxpMazSurlDoMOuhhxM1FsXbEovDeM?=
 =?us-ascii?Q?bZM+UdWFj7gPwWZc+LgyWXEyQZP+Vlu5+v4zge8USB9tA5PiwEg0cRaehNfn?=
 =?us-ascii?Q?A8iGyd1b/k4W8pgvscupDGyDAXVejwsofjCiCID7g/CBGTsoAl21ZyN9Yxja?=
 =?us-ascii?Q?L+gPCH2L5BYXmTUp/IZ8au2KeTvO0da6rYwiyIXNCrZsd8P5aEjflQwg17MY?=
 =?us-ascii?Q?yku6oI6rAMhMiMyx676usRXKdfuO8HXDVTkNZlo1KIZii8/DouoisKGbpeMW?=
 =?us-ascii?Q?FdIKHp3g1IeOYILDDkrmFehslM7lvkLChK2bZZbonsAK7CGPrNy9erikm0m/?=
 =?us-ascii?Q?ZgV1XmuAvRG7nxJ/bFwCmpkJSeHlMIuYTBbEf7zmfX3bNq71laYXv9ZL9ist?=
 =?us-ascii?Q?JJPjsYvPeTb0kuMGZkyowMC8EGS3NfuIGM7vbfP/SDNG5m8i1weu19n4Zw4f?=
 =?us-ascii?Q?yb2HwpRHz6pfsMPv8WgsdM92iKKb82sECk0nn+x/2COAm+Z9Q1oSlMsqOYNB?=
 =?us-ascii?Q?xxgPBzx73RIXNCvq85wtg9eYgbO+pIvUzj2Lmfyv?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DM6PR11MB4092.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 0d97bb5d-fb98-4c3f-3fb7-08dac838cb8b
X-MS-Exchange-CrossTenant-originalarrivaltime: 17 Nov 2022 01:12:28.4658
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: 6lxyJ/imJot16qDKssDRVhEPuOaTYhr8gM3uxP2re6MvnGLE9eCDBRNXSLv3bL2WkkbYXEGEwH/XYLDSHawTrA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN9PR11MB5289
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_NONE,
        SPF_NONE autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Paul,

Thank you for the quick review. The commit message looks more accurate and =
clear after your wordsmithing.

-Yunying

-----Original Message-----
From: Paul E. McKenney <paulmck@kernel.org>=20
Sent: Thursday, 17 November, 2022 04:31
To: Sun, Yunying <yunying.sun@intel.com>
Cc: tglx@linutronix.de; peterz@infradead.org; longman@redhat.com; x86@kerne=
l.org; linux-kernel@vger.kernel.org; Tang, Feng <feng.tang@intel.com>
Subject: Re: [PATCH] clocksource: Print clocksource name when clocksource i=
s tested unstable

On Wed, Nov 16, 2022 at 04:22:21PM +0800, Yunying Sun wrote:
> Some "TSC fall back to HPET" cases are seen on systems that have more=20
> than 2 numa nodes. When this happens, in kernel log it has:
>=20
> clocksource: timekeeping watchdog on CPU168: hpet read-back delay of=20
> 4296200ns, attempt 4, marking unstable
>=20
> The "hpet" here is misleading since it prints only the name of=20
> watchdog, where actually it's measuring the delay of 3 reads: wd-clocksou=
rce-wd.
>=20
> Signed-off-by: Yunying Sun <yunying.sun@intel.com>

Good eyes!

I queued this with wordsmithing as shown below.  Please let me know if I me=
ssed something up.

							Thanx, Paul

------------------------------------------------------------------------

commit 72ff63e2d3f19963a4ef401e5f8c1bd16646b587
Author: Yunying Sun <yunying.sun@intel.com>
Date:   Wed Nov 16 16:22:21 2022 +0800

    clocksource: Print clocksource name when clocksource is tested unstable
   =20
    Some "TSC fall back to HPET" messages appear on systems having more tha=
n
    2 NUMA nodes:
   =20
    clocksource: timekeeping watchdog on CPU168: hpet read-back delay of 42=
96200ns, attempt 4, marking unstable
   =20
    The "hpet" here is misleading the clocksource watchdog is really
    doing repeated reads of "hpet" in order to check for unrelated delays.
    Therefore, print the name of the clocksource under test, prefixed by
    "wd-" and suffixed by "-wd", for example, "wd-tsc-wd".
   =20
    Signed-off-by: Yunying Sun <yunying.sun@intel.com>
    Signed-off-by: Paul E. McKenney <paulmck@kernel.org>

diff --git a/kernel/time/clocksource.c b/kernel/time/clocksource.c index 40=
15ec6503a52..6f681d56a843f 100644
--- a/kernel/time/clocksource.c
+++ b/kernel/time/clocksource.c
@@ -257,8 +257,8 @@ static enum wd_read_status cs_watchdog_read(struct cloc=
ksource *cs, u64 *csnow,
 			goto skip_test;
 	}
=20
-	pr_warn("timekeeping watchdog on CPU%d: %s read-back delay of %lldns, att=
empt %d, marking unstable\n",
-		smp_processor_id(), watchdog->name, wd_delay, nretries);
+	pr_warn("timekeeping watchdog on CPU%d: wd-%s-wd read-back delay of %lldn=
s, attempt %d, marking unstable\n",
+		smp_processor_id(), cs->name, wd_delay, nretries);
 	return WD_READ_UNSTABLE;
=20
 skip_test: