From patchwork Sat Sep 15 01:34:43 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sasha Levin X-Patchwork-Id: 146744 Delivered-To: patch@linaro.org Received: by 2002:a2e:1648:0:0:0:0:0 with SMTP id 8-v6csp1313616ljw; Fri, 14 Sep 2018 18:35:00 -0700 (PDT) X-Google-Smtp-Source: ANB0VdalXcdeVvXv017pIykcD/ZQ+/1rgE2K/FLQS3YRCXOt4DRtYqR6wrtdfm+L9xtb22cvN/zt X-Received: by 2002:a65:5545:: with SMTP id t5-v6mr13980735pgr.157.1536975300386; Fri, 14 Sep 2018 18:35:00 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1536975300; cv=none; d=google.com; s=arc-20160816; b=xvIN+2FhfjD19P/2gy/WO5cl7Uk5X03L5M0AtFSbOQCMHIyV6VpoH4oOUUHns5vZ48 kTAgHvJGmagl6X4Ae+6iAse1sFOkh6kbgThLioDGVHSxuvupCz+MZx6RDPO0IZcT2jum vV5FcutosqZ3Ry4jfuoN00Wxp/kLFhRbHj+PY4Qb/QwmMHdCC9aJ6WuXpLlEagS88qkP d5e/yXovXNjBR9RjpgazGbjLa8beJB65mGWeqAKpoTQdOp2ckXALxnyl06az1KIGiRIn 3iRVco6p3WirAv07d3waRPD6vflPhC28JvCfwQ34JCR+PoP4fpdKsaAVByWA2rv+ut/Z /Itg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:mime-version:content-transfer-encoding :spamdiagnosticmetadata:spamdiagnosticoutput:content-language :accept-language:in-reply-to:references:message-id:date:thread-index :thread-topic:subject:cc:to:from:dkim-signature; bh=DbAZiNiT0gN9K79+HMXUxZTs7MGFKCxxzAtWKlLJqT0=; b=vK0t0nAEDSaOLHzIlKA72BhWOuL0BK+nELv/y/uOb0Te+7ofNM7ewAPIdstR5OGZbC o799QqUFx6P0llPu5N2+tWSRV3hvOvZ7CpznWkZx0a9V4kCeeOK44V9XkGdW+1+MPgsM 3TKQlfIeXltCRX/e5xLvLXDM3YgMf6uUuDBmeEtLRPPySsVD1kVUsn6LxVg0cDfA1C0e YWX/A1k8FeQpjQREZar7VQ3XjP8SYYuMLSmRI/xA5PdNqcnUb96jArWFZo+o/AAIMLXp OZ1meLhyD+Bt7T1kECYWA5r58olTzqhoGHS5FP6ChAaWjX/KlX84glvsFg6LEwaZK5sa 36YA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@microsoft.com header.s=selector1 header.b=NeLvFYdL; spf=pass (google.com: best guess record for domain of stable-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=stable-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=microsoft.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id f185-v6si8412838pgc.625.2018.09.14.18.35.00; Fri, 14 Sep 2018 18:35:00 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of stable-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@microsoft.com header.s=selector1 header.b=NeLvFYdL; spf=pass (google.com: best guess record for domain of stable-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=stable-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=microsoft.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728972AbeIOGv5 (ORCPT + 13 others); Sat, 15 Sep 2018 02:51:57 -0400 Received: from mail-by2nam01on0130.outbound.protection.outlook.com ([104.47.34.130]:13056 "EHLO NAM01-BY2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1728970AbeIOGvv (ORCPT ); Sat, 15 Sep 2018 02:51:51 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=DbAZiNiT0gN9K79+HMXUxZTs7MGFKCxxzAtWKlLJqT0=; b=NeLvFYdLoIqh8/7aOys/BYy96EJ3MlGHso0KPfdz2fGqZpHtGImfnD/RK1e0npxXG3drW152F3idDrTnhbGdN8Z3YqSOqxbMDMnzIu2ioVBAcum98QjsDWQsOJgo9+IsjCGkGPPkXpk7LP4/cOfzVCrSLSEsQ6KYlAJ+fTz6f5s= Received: from CY4PR21MB0776.namprd21.prod.outlook.com (10.173.192.22) by CY4PR21MB0823.namprd21.prod.outlook.com (10.173.192.9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1164.9; Sat, 15 Sep 2018 01:34:43 +0000 Received: from CY4PR21MB0776.namprd21.prod.outlook.com ([fe80::151:b6fe:32c8:cccd]) by CY4PR21MB0776.namprd21.prod.outlook.com ([fe80::151:b6fe:32c8:cccd%9]) with mapi id 15.20.1164.008; Sat, 15 Sep 2018 01:34:43 +0000 From: Sasha Levin To: "stable@vger.kernel.org" , "linux-kernel@vger.kernel.org" CC: John Stultz , Thomas Gleixner , Ingo Molnar , Miroslav Lichvar , Richard Cochran , Prarit Bhargava , Stephen Boyd , Shuah Khan , "linux-kselftest@vger.kernel.org" , Sasha Levin Subject: [PATCH AUTOSEL 4.9 28/34] selftest: timers: Tweak raw_skew to SKIP when ADJ_OFFSET/other clock adjustments are in progress Thread-Topic: [PATCH AUTOSEL 4.9 28/34] selftest: timers: Tweak raw_skew to SKIP when ADJ_OFFSET/other clock adjustments are in progress Thread-Index: AQHUTJRH2JRWu2IX0k+2OewdTBXJeA== Date: Sat, 15 Sep 2018 01:34:43 +0000 Message-ID: <20180915013422.180023-28-alexander.levin@microsoft.com> References: <20180915013422.180023-1-alexander.levin@microsoft.com> In-Reply-To: <20180915013422.180023-1-alexander.levin@microsoft.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [52.168.54.252] x-ms-publictraffictype: Email x-microsoft-exchange-diagnostics: 1; CY4PR21MB0823; 6:vyR7EDLLvu8mHc6jjonXLyyJWdzYmxqW00Io2PrpumWH7aIaclKx686t9mjiFlJ6xnKFf3wNUNzY64ARPkBw2aIGRX7hMuVZSlBRDlxPP/oX+ndibjWWGc/+Tdp5+uX+vDs36JhIkev/FX3yUdqxjce/GULA2TOV1nD7Dk/3hqLqFU67LguK0BF9FFuTN67DzWflcM5gHSHbSxsHKlOYqiT8of8Nl9p8OKGFaJvo7Y9WtJiliXWDSWiz6q8VlsnXiDqkwlMmrDxs3zLM9n7u1Y4F3ZXh+E5hvVkfmEoxC4bbbj0xfYuhCVvoN8usHOdBMNKxpgGp9kE4nwJaAunqw9UO9ATsCJnYy+BvQ4as1QIDwjX8RiIqxHfbDCDuUEw+4m/5iEJTa0aO97KslmI1z8wl/nBninNaPVERurLs4J/iPTwSh92wcqqGc7H6uO+sG6oH3UYlMlHasuROw9BFyg==; 5:eC7Tf3c7tljxGHwhEJSeegwAUTZaJatvm+DY4q+cYJpmPi2xSWaDqG9M2ZparXNfo34BC4hGTyBgjeWilVFN8nTcCWqH1Xm5KNW9g64etnSMP5vZ73hMWmf+uvPEiYX7unPQ3vnEM2BKnWv3s+td9FS8Er5i+S5iewmpnH2k+a0=; 7:0wbulZb74yvB2ZyjFnF5hhetu/HuOXMrrCl4SZF311DDI4Y4+EzNvIPzF5s8e9dWYfCDdgG8qYa452IGDDQ9PMWOmY4jiVog/z7To8hBLtqPbhyDXkAJ7L1CznIqwHWA+ZQLplkOQltuJMauTuthJz59WD/hlHi3Y+6LXfxWb/QlwRP2DKa1H+WrfGFzZW+c5JrVh/sW2bB4/4H6dd+TnWMc7s6zzdszWf51Q1rNyN53lM8b9GnuvPNrDkqyMkHg x-ms-office365-filtering-correlation-id: f9599d52-45f9-44c8-a1ad-08d61aab69fa x-ms-office365-filtering-ht: Tenant x-microsoft-antispam: BCL:0; PCL:0; RULEID:(7020095)(4652040)(8989137)(4534165)(4627221)(201703031133081)(201702281549075)(8990107)(5600074)(711020)(4618075)(2017052603328)(7193020); SRVR:CY4PR21MB0823; x-ms-traffictypediagnostic: CY4PR21MB0823: x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:(28532068793085)(89211679590171)(9452136761055)(85827821059158); x-ms-exchange-senderadcheck: 1 x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(8211001083)(6040522)(2401047)(8121501046)(5005006)(3231353)(944501410)(52105095)(2018427008)(93006095)(93001095)(10201501046)(3002001)(6055026)(149027)(150027)(6041310)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123560045)(20161123562045)(20161123564045)(20161123558120)(201708071742011)(7699050)(76991041); SRVR:CY4PR21MB0823; BCL:0; PCL:0; RULEID:; SRVR:CY4PR21MB0823; x-forefront-prvs: 0796EBEDE1 x-forefront-antispam-report: SFV:NSPM; SFS:(10019020)(396003)(376002)(346002)(136003)(39860400002)(366004)(189003)(199004)(53936002)(14454004)(2900100001)(3846002)(6116002)(256004)(10290500003)(81166006)(81156014)(217873002)(76176011)(72206003)(478600001)(8936002)(25786009)(8676002)(86362001)(66066001)(6512007)(86612001)(7736002)(305945005)(7416002)(97736004)(11346002)(5660300001)(5250100002)(102836004)(54906003)(2616005)(446003)(6436002)(6506007)(476003)(186003)(110136005)(2501003)(2906002)(105586002)(22452003)(10090500001)(36756003)(4326008)(68736007)(39060400002)(26005)(107886003)(6486002)(486006)(106356001)(1076002)(6346003)(316002)(99286004); DIR:OUT; SFP:1102; SCL:1; SRVR:CY4PR21MB0823; H:CY4PR21MB0776.namprd21.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; MX:1; A:1; received-spf: None (protection.outlook.com: microsoft.com does not designate permitted sender hosts) authentication-results: spf=none (sender IP is ) smtp.mailfrom=Alexander.Levin@microsoft.com; x-microsoft-antispam-message-info: r+S2AHZ3GZR12CLgXlVsXZ0RPeTQ+sj1Y2f6DskPk7sMgT6hyDSFAxCqc2bTU0bcubggiMk0vWZCDS8RYmaNdJND4mAQRPx8P8rPERTXxsVY52dPar6x9MVhIE3x9McWUQcO4eCUtx9qLiN1DhA7nSSg/VAE+N84EFaD1B7ot2+SN6hFLKzwE55nTQ/ICQpI9f9coE64Gp3+1e6SrKfiDoNpWgIwurNtaJZQTQZ1Gc/liw2xafA3YeftBg2hg+nlyaB5wheyMKonaLbYdNmTAHrbi5KnINo2LFdPIbfU6DYDCmlxOVFEhGsB/1A8CY/54t/FPkk5HR+0miKTQp6UEY51DZb9p3ytzEnJYBskkSg= spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM MIME-Version: 1.0 X-OriginatorOrg: microsoft.com X-MS-Exchange-CrossTenant-Network-Message-Id: f9599d52-45f9-44c8-a1ad-08d61aab69fa X-MS-Exchange-CrossTenant-originalarrivaltime: 15 Sep 2018 01:34:43.7860 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 72f988bf-86f1-41af-91ab-2d7cd011db47 X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR21MB0823 Sender: stable-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: stable@vger.kernel.org From: John Stultz [ Upstream commit 1416270f4a1ae83ea84156ceba19a66a8f88be1f ] In the past we've warned when ADJ_OFFSET was in progress, usually caused by ntpd or some other time adjusting daemon running in non steady sate, which can cause the skew calculations to be incorrect. Thus, this patch checks to see if the clock was being adjusted when we fail so that we don't cause false negatives. Cc: Thomas Gleixner Cc: Ingo Molnar Cc: Miroslav Lichvar Cc: Richard Cochran Cc: Prarit Bhargava Cc: Stephen Boyd Cc: Shuah Khan Cc: linux-kselftest@vger.kernel.org Suggested-by: Miroslav Lichvar Signed-off-by: John Stultz --- v2: Widened the checks to look for other clock adjustments that could happen, as suggested by Miroslav v3: Fixed up commit message Signed-off-by: Sasha Levin --- tools/testing/selftests/timers/raw_skew.c | 5 +++++ 1 file changed, 5 insertions(+) -- 2.17.1 diff --git a/tools/testing/selftests/timers/raw_skew.c b/tools/testing/selftests/timers/raw_skew.c index 30906bfd9c1b..0ab937a17ebb 100644 --- a/tools/testing/selftests/timers/raw_skew.c +++ b/tools/testing/selftests/timers/raw_skew.c @@ -146,6 +146,11 @@ int main(int argv, char **argc) printf(" %lld.%i(act)", ppm/1000, abs((int)(ppm%1000))); if (llabs(eppm - ppm) > 1000) { + if (tx1.offset || tx2.offset || + tx1.freq != tx2.freq || tx1.tick != tx2.tick) { + printf(" [SKIP]\n"); + return ksft_exit_skip("The clock was adjusted externally. Shutdown NTPd or other time sync daemons\n"); + } printf(" [FAILED]\n"); return ksft_exit_fail(); }