From nobody Fri Dec 19 14:47:00 2025 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id EDF4AC4167B for ; Mon, 11 Dec 2023 12:21:44 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1343551AbjLKMVg (ORCPT ); Mon, 11 Dec 2023 07:21:36 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:58806 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1343626AbjLKMVN (ORCPT ); Mon, 11 Dec 2023 07:21:13 -0500 Received: from mgamail.intel.com (mgamail.intel.com [192.55.52.88]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 4F07110DA; Mon, 11 Dec 2023 04:21:05 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1702297265; x=1733833265; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=VBcdrvEK0XQ9ISAlNYAMavewDocUP/dWs8LzX3EURKw=; b=jC+jkXonTDVAzl3foRHR3om/PqTV/GajiYZlTLNQnqtmnV3vZNLEB7jL /L5daDR20F6q5I4d1Xq5bwRrtZVeEy+iogkgn8JX0uk4rnTvCZH7qrmau XX3z3SK09sutS/3quTEkOOiBUy6Fw0CUjw+ddgmhFDHMrVR4Ln3WZLZgc eqy4+bdzYSyDcy3ZNLQ0KPUnF5Y+Us4YGadwCC25PSZ2ORt2nc6ZqTDYb r9f0y7V4jmn/4DjFX95dDpU8ZK+29atybU8nr0GYJBrGmuHJD08/px02L jc4jwv+YXsUG/b/LK/VaIp83Z0aHl8xYJJzH3ln9FFrEuH6uEnIv54Cwj w==; X-IronPort-AV: E=McAfee;i="6600,9927,10920"; a="425765903" X-IronPort-AV: E=Sophos;i="6.04,267,1695711600"; d="scan'208";a="425765903" Received: from orviesa002.jf.intel.com ([10.64.159.142]) by fmsmga101.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 11 Dec 2023 04:21:04 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.04,267,1695711600"; d="scan'208";a="14427016" Received: from ijarvine-desk1.ger.corp.intel.com (HELO localhost) ([10.246.50.188]) by orviesa002-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 11 Dec 2023 04:21:01 -0800 From: =?UTF-8?q?Ilpo=20J=C3=A4rvinen?= To: linux-kselftest@vger.kernel.org, Reinette Chatre , Shuah Khan , Shaopeng Tan , =?UTF-8?q?Maciej=20Wiecz=C3=B3r-Retman?= , Fenghua Yu Cc: linux-kernel@vger.kernel.org, =?UTF-8?q?Ilpo=20J=C3=A4rvinen?= Subject: [PATCH v3 16/29] selftests/resctrl: Improve perf init Date: Mon, 11 Dec 2023 14:18:13 +0200 Message-Id: <20231211121826.14392-17-ilpo.jarvinen@linux.intel.com> X-Mailer: git-send-email 2.30.2 In-Reply-To: <20231211121826.14392-1-ilpo.jarvinen@linux.intel.com> References: <20231211121826.14392-1-ilpo.jarvinen@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org struct perf_event_attr initialization is spread into perf_event_initialize() and perf_event_attr_initialize() and setting ->config is hardcoded by the deepest level. perf_event_attr init belongs to perf_event_attr_initialize() so move it entirely there. Rename the other function perf_event_initialized_read_format(). Call each init function directly from the test as they will take different parameters (especially true after the perf related global variables are moved to local variables). Signed-off-by: Ilpo J=C3=A4rvinen Reviewed-by: Reinette Chatre --- tools/testing/selftests/resctrl/cache.c | 16 ++++++---------- 1 file changed, 6 insertions(+), 10 deletions(-) diff --git a/tools/testing/selftests/resctrl/cache.c b/tools/testing/selfte= sts/resctrl/cache.c index 7f45c4957f81..58ee6f6b0edf 100644 --- a/tools/testing/selftests/resctrl/cache.c +++ b/tools/testing/selftests/resctrl/cache.c @@ -15,8 +15,9 @@ static struct perf_event_read pe_read; static int pe_fd; char llc_occup_path[1024]; =20 -static void perf_event_attr_initialize(void) +static void perf_event_attr_initialize(__u64 config) { + memset(&pea_llc_miss, 0, sizeof(struct perf_event_attr)); pea_llc_miss.type =3D PERF_TYPE_HARDWARE; pea_llc_miss.size =3D sizeof(struct perf_event_attr); pea_llc_miss.read_format =3D PERF_FORMAT_GROUP; @@ -27,6 +28,7 @@ static void perf_event_attr_initialize(void) pea_llc_miss.inherit =3D 1; pea_llc_miss.exclude_guest =3D 1; pea_llc_miss.disabled =3D 1; + pea_llc_miss.config =3D config; } =20 /* Start counters to log values */ @@ -36,16 +38,9 @@ static void perf_event_reset_enable(void) ioctl(pe_fd, PERF_EVENT_IOC_ENABLE, 0); } =20 -static void perf_event_initialize(void) +static void perf_event_initialize_read_format(void) { - memset(&pea_llc_miss, 0, sizeof(struct perf_event_attr)); memset(&pe_read, 0, sizeof(struct perf_event_read)); - - /* Initialize perf_event_attr structures for HW_CACHE_MISSES */ - perf_event_attr_initialize(); - - pea_llc_miss.config =3D PERF_COUNT_HW_CACHE_MISSES; - pe_read.nr =3D 1; } =20 @@ -204,7 +199,8 @@ int cat_val(struct resctrl_val_param *param, size_t spa= n) if (ret) return ret; =20 - perf_event_initialize(); + perf_event_attr_initialize(PERF_COUNT_HW_CACHE_MISSES); + perf_event_initialize_read_format(); =20 /* Test runs until the callback setup() tells the test to stop. */ while (1) { --=20 2.30.2