From patchwork Wed Oct 18 07:55:26 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Ilpo_J=C3=A4rvinen?= X-Patchwork-Id: 154735 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:612c:2908:b0:403:3b70:6f57 with SMTP id ib8csp4629902vqb; Wed, 18 Oct 2023 00:56:18 -0700 (PDT) X-Google-Smtp-Source: AGHT+IGZghovGDv01wuhpJFlqyLasQe+OzBzzMALAuuIWjGXOfQ2P+vX42T41CJoF87jf0uujpT6 X-Received: by 2002:aca:2b1a:0:b0:3ad:ae0d:f845 with SMTP id i26-20020aca2b1a000000b003adae0df845mr4318627oik.5.1697615778164; Wed, 18 Oct 2023 00:56:18 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1697615778; cv=none; d=google.com; s=arc-20160816; b=sEqtwYNFosW/jQFidxhhfaTrrdRhGeML/oNzvtYPKh/EvOONSJvEEB9aI/EfayxReK XqvwB0IKREGeqLvP0LxzNGfw0LxYeRv30PbMgi9Gctyi6auijbU+HvO+moholZfeG1g8 tDJuZ2ujEmxkn0ceC2Xpi46zP9iGulIQdrG+7badjg7QQVhSj1iQZvrMRv1/WhfJDPu+ Le3XkWV88VWZ22tKU5gRkDOUKToHk2SEZxKfhHRmZnKf6xaOp1r1cuJ/x7jpPNlDLXVo kxRVygxWO/dPH/F+SNJvHNJl9wLBezBuSrn8lKIYVZXaYntFW+uPfRK5DmX65nhZ0DkX C2PA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:to:from:dkim-signature; bh=/fLsTZEh+GH2fmNdpekWj+O/z8fEL0a0UxDGH+xC0Bw=; fh=M9ocEYFn8yrZ266S4+clsyb25gK7oCq66+Q1PeHBTAw=; b=jI2OChT+0YBLhRn/dB90lgg+z7ZfMomiBrfV51m2LHZmnGEg56G3Ag9IkhAariQncs CcsbCyYWlo5DwzNtLfwfH1d82U63Zbtqf8/9njrnUdkpeamtS+cwpJ6BSPCvhpoP+jZo 6mSzOIhp/WiSHB7w3IZsqf26/nwOhxKOmf8Ee4BTCBBrFO6VD9e3fX0w0y1Fdy4IXc6y Y8yTrUEkVWzy7n/h4ewagM8M+cpOVFuV4qEyewiGhYrnlt/brkI96rcgtb0PmeUZC5VX bLtYImi+Nsz0rK3In6bSCWKIXzVQl1KsVQZWylamZgaLOwIE9LoTQJqHPoutPqv+OzPj wjDg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b=CpffZcq8; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:8 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: from fry.vger.email (fry.vger.email. [2620:137:e000::3:8]) by mx.google.com with ESMTPS id d7-20020a656207000000b00563de199314si1583837pgv.896.2023.10.18.00.56.17 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 18 Oct 2023 00:56:18 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:8 as permitted sender) client-ip=2620:137:e000::3:8; Authentication-Results: mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b=CpffZcq8; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:8 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by fry.vger.email (Postfix) with ESMTP id 5137680B8F03; Wed, 18 Oct 2023 00:56:08 -0700 (PDT) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.10 at fry.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1344607AbjJRHzs (ORCPT + 24 others); Wed, 18 Oct 2023 03:55:48 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:39622 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229897AbjJRHzq (ORCPT ); Wed, 18 Oct 2023 03:55:46 -0400 Received: from mgamail.intel.com (mgamail.intel.com [134.134.136.65]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 39AAFC6; Wed, 18 Oct 2023 00:55:44 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1697615744; x=1729151744; h=from:to:subject:date:message-id:mime-version: content-transfer-encoding; bh=v9WPbSLAJSd1nBR3aP50+jggvvGNsUWZe+tO6/XexwM=; b=CpffZcq8sNN3QGmzwfYGxLH+lBkrsBU+9MqfW3YLnzPXXHlG3j5oduM0 Y2z1hU7bUTzglUg5Dc9LWYfFymeJGH6xeZ+H6Nm6jCn2j3Mvcr3spzYZw SVTWLiMvWfvxDoJJACE3fAcVRDqnPQZGC6Z/NNTzaww1++wIRjwZZ1Mir GKy9UlyJbx6RSmDRfRWRV5wSBc67hWGk1je/bfZ05Jm18hhms9A5OJh7Z SrbQxEPA3mPl+C7fHvgqViq/Rp89atDwUCimKarLjBqEk6bKHF25oUiau kzi0kzvAOK2R0x1IVxCxobBy19aK0WfG7hHN/6AKB7Kz2iDvjOLvIDtNm w==; X-IronPort-AV: E=McAfee;i="6600,9927,10866"; a="389837086" X-IronPort-AV: E=Sophos;i="6.03,234,1694761200"; d="scan'208";a="389837086" Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by orsmga103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 18 Oct 2023 00:55:43 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10866"; a="900235401" X-IronPort-AV: E=Sophos;i="6.03,234,1694761200"; d="scan'208";a="900235401" Received: from suguccin-mobl1.amr.corp.intel.com (HELO localhost) ([10.252.44.63]) by fmsmga001-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 18 Oct 2023 00:53:36 -0700 From: =?utf-8?q?Ilpo_J=C3=A4rvinen?= To: Fenghua Yu , Reinette Chatre , Shuah Khan , Maciej Wieczor-Retman , =?utf-8?q?Ilpo_J?= =?utf-8?q?=C3=A4rvinen?= , linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org Subject: [PATCH v2 1/1] selftests/resctrl: Fix MBM test failure when MBA unavailable Date: Wed, 18 Oct 2023 10:55:26 +0300 Message-Id: <20231018075526.2251-1-ilpo.jarvinen@linux.intel.com> X-Mailer: git-send-email 2.30.2 MIME-Version: 1.0 X-Spam-Status: No, score=-0.8 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on fry.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (fry.vger.email [0.0.0.0]); Wed, 18 Oct 2023 00:56:08 -0700 (PDT) X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1780079161855122720 X-GMAIL-MSGID: 1780079161855122720 Commit 20d96b25cc4c ("selftests/resctrl: Fix schemata write error check") exposed a problem in feature detection logic in MBM selftest. If schemata does not support MB:x=x entries, the schemata write to initialize 100% memory bandwidth allocation in mbm_setup() will now fail with -EINVAL due to the error handling corrected by the commit 20d96b25cc4c ("selftests/resctrl: Fix schemata write error check"). That commit just uncovers the failed write, it is not wrong itself. If MB:x=x is not supported by schemata, it is safe to assume 100% memory bandwidth is always set. Therefore, the previously ignored error does not make the MBM test itself wrong. Restore the previous behavior of MBM test by checking MB support before attempting to write it into schemata which results in behavior equivalent to ignoring the write error. Fixes: 20d96b25cc4c ("selftests/resctrl: Fix schemata write error check") Signed-off-by: Ilpo Järvinen Reviewed-by: Reinette Chatre --- tools/testing/selftests/resctrl/mbm_test.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/testing/selftests/resctrl/mbm_test.c b/tools/testing/selftests/resctrl/mbm_test.c index d3c0d30c676a..741533f2b075 100644 --- a/tools/testing/selftests/resctrl/mbm_test.c +++ b/tools/testing/selftests/resctrl/mbm_test.c @@ -95,7 +95,7 @@ static int mbm_setup(struct resctrl_val_param *p) return END_OF_TESTS; /* Set up shemata with 100% allocation on the first run. */ - if (p->num_of_runs == 0) + if (p->num_of_runs == 0 && validate_resctrl_feature_request("MB", NULL)) ret = write_schemata(p->ctrlgrp, "100", p->cpu_no, p->resctrl_val);