From nobody Sun Dec 14 13:45:47 2025 Received: from NAM11-DM6-obe.outbound.protection.outlook.com (mail-dm6nam11on2061.outbound.protection.outlook.com [40.107.223.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 29C2A20A5CF for ; Wed, 11 Dec 2024 18:56:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.107.223.61 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733943394; cv=fail; b=ASHff2SU6ngntHnOtL7Yb/JLFEUvlg6mTqFCp/d5P8ogZUGy8nAtEwzEdCSOV2sWcnZgBeS30QCzwPLz+h7tz/gc3HXzSjJaqe3JGN0TTpGmqzo70VnMy/rErDO1hjjFqF/eEEay0WiUoZOIUnA7TrVxkxAEpZjX/dVejb8SSwM= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733943394; c=relaxed/simple; bh=Bre41trVRBohC4zvVFPB5M587fLjir/blwn/CJkgP+s=; h=From:To:CC:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=MzCpmfCoLmKcm7ehmm5NJae/JR9hxrYWL7gqEjHVuWE/SeOei9C8wmvOX6n8F2VWd7ZbUln5jJAWiJyGnQYj4QwsOgRC/d/S47YtmwHOsoySdIoAYxRWFxNs7F7e0iuYqCZbD2xnPK4x6upuowfYaE4oUR+M9PGk68y4x/wXtp0= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com; spf=fail smtp.mailfrom=amd.com; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b=g7dlhhmf; arc=fail smtp.client-ip=40.107.223.61 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="g7dlhhmf" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=UjkXLvTPUpVRuNCJ320+rFQ8W31Tj+cZ7yPJKSrqzx22oNjq78qm6DtOZ+kIE8pRGA7+Tgv6zGxPU3WDLPjwZxndSPhsNKpIiwLB3GpEDyL/oMXFGYtEkm6rtpCd+CopXg2RWzqKAC1y+OScxpKTfLGKZoCffz4eZq9ePY2zeib3/HVwaQEsGMclh7KWTUQvwH688MOcdLXAY6AJNZhFE+YWjYz6tsJvwPUCuJvxMMS9DzUaijTJHbthnRuCunNk+4fCzc1k3k9FL5q6UEMzn9npmowHR71UYiTtFNeCkFJvqNDaKWmTKXuAs9aZlLk4pT4JoF+PZh2jVbTIknibPg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=3Gad3kKJiLb5f5dDBQyplklCc4JCyjxOFc/r1zDbNQQ=; b=gZobCLs14dU16ujzVuM/TmZyD/A9rqnMirR82Fx57alymGe7cYPVwuNPMUSkz2wRcc3Ezii6D6kYMYuQzxkQR603FKq4mFX8NeIUeoYtGsTjxx4cZv6EVO4JPQ0CAP2xQ1afiWwQL9S4RHMDYkCmbLdxpZW7XE9zmIgqTjUQsow0ZajLhDhCPkXZF5aoPje7gN08gj2zngubB6358YLdMNYYsr24uVGXQammncOBOP0mWemvBC6YmNwhwBTKAjTAFTCXlJQx4u8QdbClTn8q8RDefxUjEk2JP7iJmQmCEZP4qX3z3t4rDVWTPFzMcEgM3KnN8qq9EHAEcsGVxAfZuw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=linutronix.de smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=3Gad3kKJiLb5f5dDBQyplklCc4JCyjxOFc/r1zDbNQQ=; b=g7dlhhmf89dWfwGpmnB2YQBo/YNWirGNE1PZ01DOof2NwnI2UpRgx7SorLglzrtMhxxixR1Q/2VNvwxranPjWe5tuRdkQet2vfSi6vpWKeOFknAb3hK+uXnxqNWysrsgrMk5fJiCO9z8SuXH6mNU/BBs0uIkuPUWfj5gCHCTKxA= Received: from PH7P222CA0010.NAMP222.PROD.OUTLOOK.COM (2603:10b6:510:33a::9) by SA3PR12MB7831.namprd12.prod.outlook.com (2603:10b6:806:311::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8230.18; Wed, 11 Dec 2024 18:56:26 +0000 Received: from SA2PEPF000015C9.namprd03.prod.outlook.com (2603:10b6:510:33a:cafe::c5) by PH7P222CA0010.outlook.office365.com (2603:10b6:510:33a::9) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.8230.15 via Frontend Transport; Wed, 11 Dec 2024 18:56:26 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=SATLEXMB04.amd.com; pr=C Received: from SATLEXMB04.amd.com (165.204.84.17) by SA2PEPF000015C9.mail.protection.outlook.com (10.167.241.199) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.8251.15 via Frontend Transport; Wed, 11 Dec 2024 18:56:26 +0000 Received: from BLRKPRNAYAK.amd.com (10.180.168.240) by SATLEXMB04.amd.com (10.181.40.145) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.39; Wed, 11 Dec 2024 12:56:19 -0600 From: K Prateek Nayak To: Thomas Gleixner , Ingo Molnar , Borislav Petkov , Dave Hansen , Peter Zijlstra , Juri Lelli , Vincent Guittot , , CC: "H. Peter Anvin" , Dietmar Eggemann , Steven Rostedt , Ben Segall , Mel Gorman , Valentin Schneider , "Rafael J. Wysocki" , Ricardo Neri , Tim Chen , Mario Limonciello , Meng Li , Huang Rui , "Gautham R. Shenoy" , K Prateek Nayak Subject: [PATCH 1/8] x86/itmt: Convert "sysctl_sched_itmt_enabled" to boolean Date: Wed, 11 Dec 2024 18:55:44 +0000 Message-ID: <20241211185552.4553-2-kprateek.nayak@amd.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20241211185552.4553-1-kprateek.nayak@amd.com> References: <20241211185552.4553-1-kprateek.nayak@amd.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: SATLEXMB03.amd.com (10.181.40.144) To SATLEXMB04.amd.com (10.181.40.145) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SA2PEPF000015C9:EE_|SA3PR12MB7831:EE_ X-MS-Office365-Filtering-Correlation-Id: f4a7fcbc-4314-40cc-c490-08dd1a1583bc X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|36860700013|82310400026|1800799024|7416014|376014; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?gJhpWzNCBsJj6xt1E//YC6w9rMWmh95CXsfDnuOZgC6JHMJVOUBGAmsUCXA5?= =?us-ascii?Q?VfTYjmFsD3wQpmfaxfUehs+YrixeRljd2daWZSwWwGpuNVlRsAXm34VoR/lu?= =?us-ascii?Q?PAvV0ZpTTPGoXeMLcvVM1g4F5ZUDfPjRozFKlLUKRdx9EZoE8z1Obxqbx+aS?= =?us-ascii?Q?0spSWCZVJj+AEdfdTkzjY7/AhfO9fU5pVLPkxkqUefzdkoK0BCdIWvVsZSv/?= =?us-ascii?Q?eyScEEKoB73J6HP5x2qf9G0svRozbCxm6lFQAkcqAKijZhjIBq9UKcXByinR?= =?us-ascii?Q?WIcZ/nkCGVG4hayWbxoBJudeSew1E0CS5W+/pGrnr+hYuFDwA2uTduSWaqS5?= =?us-ascii?Q?7ErRAkJis8tmroWwtFp3G/tvLN778rexdwo9YKb5c1VKmZ7ALjoWyb9nE2yc?= =?us-ascii?Q?aGHArSxu7WF6FZNxi4pI1mN2bPfFfcaKVZskKmdZ8CvFC7Y1rRLXc/BdsdFc?= =?us-ascii?Q?HL3NXJ3v7ZCfw49mp2jjLVvoMQc6yS+dwMdWPWa2g84scwjE6wvAiTzijGuu?= =?us-ascii?Q?eQQMEv/kSs7Lanw/BFEivpn+LBOEmLBpZ4uAFpVZNOLw5h2OPg1ST5xqpCv+?= =?us-ascii?Q?v8Fp7kksOG7AZPBavFCnoWydCzlOu55y+UdAnC7yVdIKJhYYQXCFHrXMgkSe?= =?us-ascii?Q?dw+q8MLQes5rTwWy6gCzGLiw/+p08coyJMh327sRTBKcG4zZCZiGRkv1M4KM?= =?us-ascii?Q?G85ubJ+w3l6g3SqkjOVUjxYpUwdesHvQdfGodXaDq0xNSOq9ztSY0EaOE84r?= =?us-ascii?Q?8D+lpKcw14C0rA7ndqQD5rZwrX4v6PIkT1xezaKgSbOKSsPhH/1c/Vt0RMra?= =?us-ascii?Q?LSXqzXXXDL3ybKp9vTDf/iClU0OnW5jFpc/4kmO+0vM49R6wnp21Pxv9fbrE?= =?us-ascii?Q?oQXRJ7piRThlfO5FrcWZl3I2U80u8WwQk332jRkQroUVOLPCyVvEAhbE16AI?= =?us-ascii?Q?EYveyQrEc3+oVMlrEVbKq6TCQvO7eok7mGtrDi2xawDZkfELnLY00vQIxDr6?= =?us-ascii?Q?+0v8UZMM73dBZIXN1RDESzJCQKECLLfdlZUk6g7HIwPbJXacbflMgipnzKtm?= =?us-ascii?Q?9oyiOCqX1fexMnMKrgcCYsFfGsLzOtgptEbEwj5EDB7NHl8dYDmXhL5y8ES6?= =?us-ascii?Q?0DS7Ckl9nB65qK8b+0Lw+88Kf2GkoO8oX5Tzgb2/D3kU61lAWRABzrXZEMV0?= =?us-ascii?Q?zUAaWDlRV3o2VJIgejxreOTCGSpWhSmMmWy3JXzFfVdjHEOhCNlugjtKzH4d?= =?us-ascii?Q?D2N41SdCMPGRs9HboBcaqqg0VYchpTleB4+l7mJxOEcl5YcB0N4D2vL9FglV?= =?us-ascii?Q?uqkDlCUzvEKvtPFMD4nPHYrutCX4r8RS3u6GxlH3tkDkAyVDWgB/r3nKoeiC?= =?us-ascii?Q?t6FcMzcUKqlSCd50/GssPq8FCD0doV7lmAk3EyAf2Ge4encW4oLQpaKlVtyN?= =?us-ascii?Q?fK0cHOZzVwE4a9Y28ruK3zoIbczOjwzS?= X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:SATLEXMB04.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230040)(36860700013)(82310400026)(1800799024)(7416014)(376014);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Dec 2024 18:56:26.2362 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: f4a7fcbc-4314-40cc-c490-08dd1a1583bc X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[SATLEXMB04.amd.com] X-MS-Exchange-CrossTenant-AuthSource: SA2PEPF000015C9.namprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA3PR12MB7831 Content-Type: text/plain; charset="utf-8" In preparation to move "sysctl_sched_itmt_enabled" to debugfs, convert the unsigned int to bool since debugfs readily exposes boolean fops primitives (debugfs_read_file_bool, debugfs_write_file_bool) which can streamline the conversion. Since the current ctl_table initializes extra1 and extra2 to SYSCTL_ZERO and SYSCTL_ONE respectively, the value of "sysctl_sched_itmt_enabled" can only be 0 or 1 and this datatype conversion should not cause any functional changes. Signed-off-by: K Prateek Nayak Reviewed-by: Tim Chen --- arch/x86/include/asm/topology.h | 4 ++-- arch/x86/kernel/itmt.c | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/arch/x86/include/asm/topology.h b/arch/x86/include/asm/topolog= y.h index fd41103ad342..63bab25a4896 100644 --- a/arch/x86/include/asm/topology.h +++ b/arch/x86/include/asm/topology.h @@ -250,7 +250,7 @@ extern bool x86_topology_update; #include =20 DECLARE_PER_CPU_READ_MOSTLY(int, sched_core_priority); -extern unsigned int __read_mostly sysctl_sched_itmt_enabled; +extern bool __read_mostly sysctl_sched_itmt_enabled; =20 /* Interface to set priority of a cpu */ void sched_set_itmt_core_prio(int prio, int core_cpu); @@ -263,7 +263,7 @@ void sched_clear_itmt_support(void); =20 #else /* CONFIG_SCHED_MC_PRIO */ =20 -#define sysctl_sched_itmt_enabled 0 +#define sysctl_sched_itmt_enabled false static inline void sched_set_itmt_core_prio(int prio, int core_cpu) { } diff --git a/arch/x86/kernel/itmt.c b/arch/x86/kernel/itmt.c index 51b805c727fc..28f449123d68 100644 --- a/arch/x86/kernel/itmt.c +++ b/arch/x86/kernel/itmt.c @@ -36,7 +36,7 @@ static bool __read_mostly sched_itmt_capable; * * It can be set via /proc/sys/kernel/sched_itmt_enabled */ -unsigned int __read_mostly sysctl_sched_itmt_enabled; +bool __read_mostly sysctl_sched_itmt_enabled; =20 static int sched_itmt_update_handler(const struct ctl_table *table, int wr= ite, void *buffer, size_t *lenp, loff_t *ppos) --=20 2.34.1 From nobody Sun Dec 14 13:45:47 2025 Received: from NAM12-BN8-obe.outbound.protection.outlook.com (mail-bn8nam12on2071.outbound.protection.outlook.com [40.107.237.71]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id A1B7B20A5CF for ; Wed, 11 Dec 2024 18:56:57 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.107.237.71 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733943419; cv=fail; b=DMVUBApTm+uBki3C8I09saSFgOgkzk4gosUDi5xbwiep8HqQQNXfxvUW7i9mRb+/nvWMRpk18d0heqY6MUntzDYeuEFyPqBHJL6B7Rs4BjxtjY7VT2OIKVOZEOZsKt9Cm21ScROU2fSfe1layW7RERQEoIn29Q3ZDEmKEPmZG78= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733943419; c=relaxed/simple; bh=oCZxi9XQ9uIRRMKF52NZsTtbklkyWtIIRdUBT8vMGUQ=; h=From:To:CC:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=Ku699TS8v6+tiwNd5ImwNWrj7KHOlXxKnZLwdlz3jnaVthdmbBTeoVNzFNY5NztEQrqzg3RnHmEzvj+n2LUq/aaUgL2ckiqkXSH1f39xT1d14o7A5RgpY/MSkaYap3aMDAYaiCBQOu3z2R99jl8VSUnnjIbtWXJAARuwM4q6ESE= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com; spf=fail smtp.mailfrom=amd.com; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b=gVUE6/eq; arc=fail smtp.client-ip=40.107.237.71 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="gVUE6/eq" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=LgX6UPTUDoBaJ/Io4kv37l5EreBCR4oTPvlJ+yWKjtjJHZPkjIUFjSHK/u4gi54upn3SS330Zs2cHACKCAgc0fwPgcnsTgRdrAmsaWhe3v9IrxgCnLwbx+iRMXAqvCx/aAi5KrUy/XpdD45kzxaL7cM1KMR9VKGBdS7WNRZJc8iW8x4Xdm11t2dQcyQawMVKjW1UIt9giYKv963Q94ricmwkXbjMfTu8SBO1FuWtqALWAy7CkCcuxsm4T5YrUD10/sRWi0kRyujkqTM2RBqocW1kAD0Ou79mGyzc3k4Srl/h6FQiltSjcWF9o58d5Rx5t9uoOzLEo/3/8x+BcFsTHg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=GgEV8Rli23FQjzDVZgnAq8bXtnae+/SnvmunJimgeYE=; b=TtO1gDzkhzo1isCKEd7xyOFI2m2giwXcMWiW/sP92rX2Pei+ghQb1NbRdixRu6Ff0HT+laGq5GUDeEEohCFDIe30NnKyfUFCA0s48bOvU0YaWdkqKoS7Huu+pjRe9yRspFPeElyhRqo4LvyIyuJpBgJX9D8cLbjHoMmUnPfuNQArsGaMTQ65Y79yE3pl8/Z/FTLWk434v8wMdElQlPHb9Q5HEz6B5uSkJZxZFgIwDvr9bCEaLJvj8KByXY0LpJ4cl9V4FHWDhfJBC9/hIuXdctD+XtLLF6IynKzIYK9hg4JxQy7zvpYdFgD1l3GDEXN1U1S8Mrjo9h29FY97YsM5ww== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=linutronix.de smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=GgEV8Rli23FQjzDVZgnAq8bXtnae+/SnvmunJimgeYE=; b=gVUE6/eq2prsBCSn1BIlyh83qYb8435Sqo7dd6dfDJ+dn+7SgswbrxePCt07JvJE5Axto+MvFlS8XouHZ4lvRv2RQXzHR3ufKXhwTcWrKrxzhv3Xfprc2XJxBB2TYjpLkNkd+2mYEvpYJvnyGilbA5btN4pJMPWnrIdoabTYr1k= Received: from PH0PR07CA0047.namprd07.prod.outlook.com (2603:10b6:510:e::22) by MW5PR12MB5684.namprd12.prod.outlook.com (2603:10b6:303:1a1::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8251.14; Wed, 11 Dec 2024 18:56:46 +0000 Received: from SA2PEPF000015C6.namprd03.prod.outlook.com (2603:10b6:510:e:cafe::27) by PH0PR07CA0047.outlook.office365.com (2603:10b6:510:e::22) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.8230.18 via Frontend Transport; Wed, 11 Dec 2024 18:56:46 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=SATLEXMB04.amd.com; pr=C Received: from SATLEXMB04.amd.com (165.204.84.17) by SA2PEPF000015C6.mail.protection.outlook.com (10.167.241.196) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.8251.15 via Frontend Transport; Wed, 11 Dec 2024 18:56:45 +0000 Received: from BLRKPRNAYAK.amd.com (10.180.168.240) by SATLEXMB04.amd.com (10.181.40.145) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.39; Wed, 11 Dec 2024 12:56:38 -0600 From: K Prateek Nayak To: Thomas Gleixner , Ingo Molnar , Borislav Petkov , Dave Hansen , Peter Zijlstra , Juri Lelli , Vincent Guittot , , CC: "H. Peter Anvin" , Dietmar Eggemann , Steven Rostedt , Ben Segall , Mel Gorman , Valentin Schneider , "Rafael J. Wysocki" , Ricardo Neri , Tim Chen , Mario Limonciello , Meng Li , Huang Rui , "Gautham R. Shenoy" , K Prateek Nayak Subject: [PATCH 2/8] x86/itmt: Use guard() for itmt_update_mutex Date: Wed, 11 Dec 2024 18:55:45 +0000 Message-ID: <20241211185552.4553-3-kprateek.nayak@amd.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20241211185552.4553-1-kprateek.nayak@amd.com> References: <20241211185552.4553-1-kprateek.nayak@amd.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: SATLEXMB03.amd.com (10.181.40.144) To SATLEXMB04.amd.com (10.181.40.145) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SA2PEPF000015C6:EE_|MW5PR12MB5684:EE_ X-MS-Office365-Filtering-Correlation-Id: aa7614e7-dbc1-4015-ef40-08dd1a158f4e X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|1800799024|376014|36860700013|7416014|82310400026; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?hBS7Cs+ryq2DzvVp4i/wia4alqTNmArvvvwM4s7b5RGn0Wyznf+OWOjDP+xK?= =?us-ascii?Q?44oJsec0W0jaCHLWYGez/nICVqC2farxmgiIfBGXy+bP0xFQy/O/sGLoUMp8?= =?us-ascii?Q?qNkJDHijLKxsVXU4bYdeIWsKXvJSdGc3K2QUPKbDbJYtZIIvtNl/AeAg2/ut?= =?us-ascii?Q?Xab1rJaywhDK0btwAP+gaBME08ZBQvtpTJjWrFFmTSRHVfS9satv4kBEHapi?= =?us-ascii?Q?2TXVKT/HFwJc+/1wuNtql30X3Fx/0+41V0D1wNBZUR3sUnE9b155NOxdufaw?= =?us-ascii?Q?aB0slGJr7VfFk7bdoE/7r6yMjqHVtYDlwznk1+GOdteMqB/gDNCQmdFA1arK?= =?us-ascii?Q?GXCph/9/P1fCGgzFpXaw3hvn3YsuHT6OvGGg2/Oie5qcl/aJ9PNf3qUuqxgL?= =?us-ascii?Q?0sm+jyDNQErWwoRIDshOwXk+v9wzdz720iqkduypyM7OMgWanjk3bqXCHN5T?= =?us-ascii?Q?aix+F/CZohPnp4N+RO2U20yMrhsHQA24dvdve5LSYSZkzRXQjzieq6Mi40VZ?= =?us-ascii?Q?5H1wvSrR5WpgCKmVXx+grF8A15bgFa6thiUKqRDpa9zY4/PzsvnNKiZZ605O?= =?us-ascii?Q?7Rh5dOJBNn2t6/+XSaJYfuRE2C/t+CnGh/ikGZeDpocDUk9jatLizdCTXafp?= =?us-ascii?Q?EWxDP6Y/TcbSHhq+VnvuCsAkOXQf5xTD8npV291s2Y78DRb7qvSEgV3XYcOf?= =?us-ascii?Q?DCQ5L6kLCYckrh2mtZZs+jUm1Jw1v06QRyA3V4ppFGEL3TzyqV36FxoM9uLk?= =?us-ascii?Q?KutpJ8g+1OgXxCWCdWOoeZA3qwl66ZpA9lTEi3ivGUdqFM+d8bAB1Kw7oREr?= =?us-ascii?Q?Pdecn8kAh0Hxlh6YL4jlUe12Gm9AVlgyDnvkMYNqdVhW7i/carj1WmhSTxIM?= =?us-ascii?Q?hU1Mvo5Mgms+9VkI8cx8jwjpnpbS9oeuJH/J59ruKi8FDz+lvwqTTZejD96d?= =?us-ascii?Q?yIb08mM0/JC1OXCa56UlLH+AiRiI7KmYmPZsJxRZNJuQkdqO3unEawpB958I?= =?us-ascii?Q?8n/sg3tDpchyjCYgRTx8yOMujClGOTQ2QEm+V2/y3voVy2Jy1fXNVdB9+amL?= =?us-ascii?Q?xen8xJveSMFxBOpK2OfkCQ5xTIIskgceAUhw0Zn2ilRRtK9pZHIly8U4ktdk?= =?us-ascii?Q?MaOVviVw3rPrUfREu/TPEckyE4YeXYddemuHIEvXc5lwW/YAji8OjT0AbNrY?= =?us-ascii?Q?pU5zvm2oPyYuvdYMeG3B4BdSMElk49RUiWQbW6jvXgDdPYPH49mKtXfb2vV8?= =?us-ascii?Q?tUTtj+/Vr1bRDg7ALRc0aFATxwgWdUk3AkGAPbTvtorXMs2uC3UAdVZWsCbr?= =?us-ascii?Q?OXI48jXoNEAWQMAcsKuZud6MaILkmpmbRqEL1V21EhKqdf0LtWTl6oDaU8A6?= =?us-ascii?Q?vmQrJ9tqIERUm5I+LQYwxSLTnDVgnMZmDI3731BZm+DoW9p+b35uPUG3L4bb?= =?us-ascii?Q?Pccm0XTmlridL1DnjZ5k7Nhb9+KesORU?= X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:SATLEXMB04.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230040)(1800799024)(376014)(36860700013)(7416014)(82310400026);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Dec 2024 18:56:45.6491 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: aa7614e7-dbc1-4015-ef40-08dd1a158f4e X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[SATLEXMB04.amd.com] X-MS-Exchange-CrossTenant-AuthSource: SA2PEPF000015C6.namprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW5PR12MB5684 Content-Type: text/plain; charset="utf-8" Use guard() for itmt_update_mutex which avoids the extra mutex_unlock() in the bailout and return paths. Signed-off-by: K Prateek Nayak Reviewed-by: Tim Chen --- arch/x86/kernel/itmt.c | 29 ++++++++--------------------- 1 file changed, 8 insertions(+), 21 deletions(-) diff --git a/arch/x86/kernel/itmt.c b/arch/x86/kernel/itmt.c index 28f449123d68..ee43d1bd41d0 100644 --- a/arch/x86/kernel/itmt.c +++ b/arch/x86/kernel/itmt.c @@ -44,12 +44,10 @@ static int sched_itmt_update_handler(const struct ctl_t= able *table, int write, unsigned int old_sysctl; int ret; =20 - mutex_lock(&itmt_update_mutex); + guard(mutex)(&itmt_update_mutex); =20 - if (!sched_itmt_capable) { - mutex_unlock(&itmt_update_mutex); + if (!sched_itmt_capable) return -EINVAL; - } =20 old_sysctl =3D sysctl_sched_itmt_enabled; ret =3D proc_dointvec_minmax(table, write, buffer, lenp, ppos); @@ -59,8 +57,6 @@ static int sched_itmt_update_handler(const struct ctl_tab= le *table, int write, rebuild_sched_domains(); } =20 - mutex_unlock(&itmt_update_mutex); - return ret; } =20 @@ -97,18 +93,14 @@ static struct ctl_table_header *itmt_sysctl_header; */ int sched_set_itmt_support(void) { - mutex_lock(&itmt_update_mutex); + guard(mutex)(&itmt_update_mutex); =20 - if (sched_itmt_capable) { - mutex_unlock(&itmt_update_mutex); + if (sched_itmt_capable) return 0; - } =20 itmt_sysctl_header =3D register_sysctl("kernel", itmt_kern_table); - if (!itmt_sysctl_header) { - mutex_unlock(&itmt_update_mutex); + if (!itmt_sysctl_header) return -ENOMEM; - } =20 sched_itmt_capable =3D true; =20 @@ -117,8 +109,6 @@ int sched_set_itmt_support(void) x86_topology_update =3D true; rebuild_sched_domains(); =20 - mutex_unlock(&itmt_update_mutex); - return 0; } =20 @@ -134,12 +124,11 @@ int sched_set_itmt_support(void) */ void sched_clear_itmt_support(void) { - mutex_lock(&itmt_update_mutex); + guard(mutex)(&itmt_update_mutex); =20 - if (!sched_itmt_capable) { - mutex_unlock(&itmt_update_mutex); + if (!sched_itmt_capable) return; - } + sched_itmt_capable =3D false; =20 if (itmt_sysctl_header) { @@ -153,8 +142,6 @@ void sched_clear_itmt_support(void) x86_topology_update =3D true; rebuild_sched_domains(); } - - mutex_unlock(&itmt_update_mutex); } =20 int arch_asym_cpu_priority(int cpu) --=20 2.34.1 From nobody Sun Dec 14 13:45:47 2025 Received: from NAM12-MW2-obe.outbound.protection.outlook.com (mail-mw2nam12on2045.outbound.protection.outlook.com [40.107.244.45]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8FCD623690B for ; Wed, 11 Dec 2024 18:57:09 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.107.244.45 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733943431; cv=fail; b=nDT76gWfkv7TzQL6VRanRhGxMFRWDEX+vXrqpumsDuT/dMHCamH/ThVShX9vDoN97hMTiyhhm8j5QDj3MPL+wfag1xBAGzI9HAYK3FJkxIXuQq1KP9fxwExU5gYRgtUZRfSIsqxYaKO9FTfInrsQz/28K7S+W4Jtv31dQoOnpLY= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733943431; c=relaxed/simple; bh=oifstS/3xdu/TGwcVv/W0X5NywSY9pGBpn2NbnLwLTk=; h=From:To:CC:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=XTGZvat66idrXrSy+Rd0L2F8/PK8SvFYtXef8mWyjbE8Wg6WzGLvP6lMnlcdXlwmbXcrDWny9H0/3MEsYjSKqEf67upBvlztGHbz+nS7gXhkUqgAkaymmUFw+ZqlJNwA+cUjaR4ngnzx5UeVAR+tYy0vX+vb/9IO/rY6XZcYW+s= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com; spf=fail smtp.mailfrom=amd.com; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b=r3XSz09c; arc=fail smtp.client-ip=40.107.244.45 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="r3XSz09c" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=WxgznX3VjbJtCh+fgrY43zStCrgC4jUoUVX5fD3eAmtZ+jxB7vBlSHTqYYQHtfpLprHXXqCqlrZMHBtlqVwgtW4xtIW2KEALsAUgPD6vGPJNepK8aWtm7Kz+fTTj8beuBasgsmG13PunazEnigkE2IBvbs2xgtFLaF7LEhsw+4KQ8B+EdQm29DlYCIYzUkSNY/+X5YAlt+3u6WAQHc7Kd3s8vf2Tjek0qB02r0O6puAmVrUA3jUEYy5yqkoDOyk9cVe9PgaMhOxjai5IJvWw4bAWhjUFPjwQTfCHL0mKo4h+RaLHTOItwSSX9dvCrmlXfvJsTdgOa7u3iiqQf7nnfQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=V5H2GKciuTg/W4k/aysfZfkHNp8qGLg686HBGrF7OhQ=; b=L5uWfGAEgc/YQX7+HGe2j+UM56SWblZ5Zmc1iV67Jn2CjiFUZc381CisXEYuKgMFgMKvt7ynSYY3N+svGimkJfuNjjvQQDO88Sf9kJKnITCjRRP6yhihtT5e6I0QTi/ffAmZZUvE5kw4CiPf+zp99zk8j1kJzEdG7Ru01gRLeImB1gXTqOvp+pM4CPmSrP6aIJw/J/lvY5bbDEmqEtHjvvxw68SFPw134G5qpGAUOzfo0bOouqVCJXrpMpIF8hKXvGwi7r2ozHVj1dGBCPsaFQ4Lt/sWYadSEMXjlfJ+c8wTp0HWPg6AAa8pwKyOREo3n8YLv45SfI8wIPyBxBIbbQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=linutronix.de smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=V5H2GKciuTg/W4k/aysfZfkHNp8qGLg686HBGrF7OhQ=; b=r3XSz09cWkDHCVM64xSKhfHIpVpPU/B5O1owxa/LcwqgfjFKlaYJfFUdxKX9I8xq/wSfVkCL//JUjqHmY0J3sFrHNKF70UY5dnrYD8+0FNTL+0biwywEGyOBx/UL1IC7PCdsGVAqz9Re5s42Gcr5YUAqPhJilF+pdjtuQq3UEgg= Received: from PH7P221CA0058.NAMP221.PROD.OUTLOOK.COM (2603:10b6:510:33c::28) by BL1PR12MB5779.namprd12.prod.outlook.com (2603:10b6:208:392::9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8230.11; Wed, 11 Dec 2024 18:57:05 +0000 Received: from SA2PEPF000015C7.namprd03.prod.outlook.com (2603:10b6:510:33c:cafe::24) by PH7P221CA0058.outlook.office365.com (2603:10b6:510:33c::28) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.8251.15 via Frontend Transport; Wed, 11 Dec 2024 18:57:05 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=SATLEXMB04.amd.com; pr=C Received: from SATLEXMB04.amd.com (165.204.84.17) by SA2PEPF000015C7.mail.protection.outlook.com (10.167.241.197) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.8251.15 via Frontend Transport; Wed, 11 Dec 2024 18:57:05 +0000 Received: from BLRKPRNAYAK.amd.com (10.180.168.240) by SATLEXMB04.amd.com (10.181.40.145) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.39; Wed, 11 Dec 2024 12:56:58 -0600 From: K Prateek Nayak To: Thomas Gleixner , Ingo Molnar , Borislav Petkov , Dave Hansen , Peter Zijlstra , Juri Lelli , Vincent Guittot , , CC: "H. Peter Anvin" , Dietmar Eggemann , Steven Rostedt , Ben Segall , Mel Gorman , Valentin Schneider , "Rafael J. Wysocki" , Ricardo Neri , Tim Chen , Mario Limonciello , Meng Li , Huang Rui , "Gautham R. Shenoy" , K Prateek Nayak Subject: [PATCH 3/8] x86/itmt: Move the "sched_itmt_enabled" sysctl to debugfs Date: Wed, 11 Dec 2024 18:55:46 +0000 Message-ID: <20241211185552.4553-4-kprateek.nayak@amd.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20241211185552.4553-1-kprateek.nayak@amd.com> References: <20241211185552.4553-1-kprateek.nayak@amd.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: SATLEXMB03.amd.com (10.181.40.144) To SATLEXMB04.amd.com (10.181.40.145) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SA2PEPF000015C7:EE_|BL1PR12MB5779:EE_ X-MS-Office365-Filtering-Correlation-Id: 72510cf6-4547-4a16-a006-08dd1a159ae9 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|1800799024|36860700013|376014|7416014|82310400026; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?lxe+rsFs4sKvrQMM9sAdE5rkzV9+dAWjadKVOpLH5bw76pcPZAGC0RkQS6FP?= =?us-ascii?Q?omp07Og0HPQbmqXmFV4WWjQKmfZqhbaGGjukPgDnZUZ0XDlFdjxqoERxsb23?= =?us-ascii?Q?vP24kZSx+DG+q7a8j7Ls6Q711IdvtGcE4iQ5pBOAus1JRP8G7GMxmuM2CWBe?= =?us-ascii?Q?w69U6+uxags3WvY/Hy/hfMF9dt7aoDwBve91z0z5RybUUp2HoiYuNtI5IGmp?= =?us-ascii?Q?88BuqQrwtt1T7bS+tAfhpdH5l1IR5cPV+i/25z3IS6DhZOIM8db8ERNU62tx?= =?us-ascii?Q?qxECF9rjr1P5Jc7K8vY337yvqeBiLVJnlvYX0znHLwZDnz8D22GXdxLb9r/+?= =?us-ascii?Q?jOBptkdEXjk2qDAKDS2oJQGPIUrY24tpnYNVhr7TztJ1dIjTI6cP/8TnbFhe?= =?us-ascii?Q?Bvzx3Qhvd67mOtwUebftfeWl39OHCutffwchRRLAt6TaaZppACj1x6Yxw7uK?= =?us-ascii?Q?s6F59Ovz9ee7k9B8h18xZoV/mCWM6HsmQlo1gfAMK4Uw1PqDmi4oQT/tAYIk?= =?us-ascii?Q?Q3/L/stRS+FVXyKpbxoiwtfeqvBsBDViJmkskXcTWdfPutHnAmuHha+AVbWn?= =?us-ascii?Q?1NFhxW629ClCGC5ltec1RqjF7+mhl9Nq2gdju3jgX2OsnEG9A835wRhhLpXI?= =?us-ascii?Q?pIoXVnLcBrd0ZzVAQ+yUh1wbVuFNtwpajJvfz9BLqldarSLNmNPA8BpUFv5k?= =?us-ascii?Q?1L1939ktB6nYMfmDm/nU+/P2ZPzej189eYgd7mFFzojeo/pD6hECwpQe5uf4?= =?us-ascii?Q?bpqTKAtXiigZyInJy5IVGmjCjmXlBc97nExT93b22EvmfSWOmyHJkCL3jJdf?= =?us-ascii?Q?AhYjXT8tXfjL2qcel2CfW4DcSs4UAP0sQBCm/7JAeQ096nslJr6vrusBgCk6?= =?us-ascii?Q?L2eEvi8/pnKdIlUcVcOi6YJWpB26Q7czDFZi21SSBu7rVRE3+BzezuB+1DWt?= =?us-ascii?Q?EiWIp64D0UNjHBRUVQlQYafhVK5KHwGSNFBBheTQqkCBHe6fsecFHQNxrfB+?= =?us-ascii?Q?QZgchv9h07ni7a5Q8yleUtW/cioXH0UB8ZSY5+cP0jrxPZv9AA2cFxFjTGjK?= =?us-ascii?Q?raP2NMloa9/KoQKyG1Efhdx9qf2TFuw3ZgToeFLayxeGIosGQrCV59naWMm2?= =?us-ascii?Q?16Plf0yDoqhPV+QKXL13TBqk1YmcYph5Z5SELl7ErtUSl45dB2u8j/C9ehNZ?= =?us-ascii?Q?ua5EZKejddooAOUiY1McFEydzlZHTobNsUtt9sKJuc4rDs42uA+zy9j6kvvC?= =?us-ascii?Q?Y6La/Z389G3vCvNgOr3gJfY8a1IaWgDGdgToziNDyztNBUknC6nix8UGbF8f?= =?us-ascii?Q?xfaa5E8HMM+k4SaJWDjXkRUewBBh0/25IexDjquoP69ZcycXzGKjKUni873Q?= =?us-ascii?Q?m33Ob9r66xIJ+izE3D9jXYO4AJHbpM+M5GFsYLoB+iIHcURzmf2l2/l8B+TM?= =?us-ascii?Q?NFBzPrDntqHeR3BYG3KEhZ0vt5zUv2id?= X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:SATLEXMB04.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230040)(1800799024)(36860700013)(376014)(7416014)(82310400026);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Dec 2024 18:57:05.1334 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 72510cf6-4547-4a16-a006-08dd1a159ae9 X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[SATLEXMB04.amd.com] X-MS-Exchange-CrossTenant-AuthSource: SA2PEPF000015C7.namprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL1PR12MB5779 Content-Type: text/plain; charset="utf-8" "sched_itmt_enabled" was only introduced as a debug toggle for any funky ITMT behavior. Move the sysctl controlled from "/proc/sys/kernel/sched_itmt_enabled" to debugfs at "/sys/kernel/debug/x86/sched_itmt_enabled" with a notable change that a cat on the file will return "Y" or "N" instead of "1" or "0" to indicate that feature is enabled or disabled respectively. Since ITMT is x86 specific (and PowerPC uses SD_ASYM_PACKING too), the toggle was moved to "/sys/kernel/debug/x86/" as opposed to "/sys/kernel/debug/sched/" Suggested-by: Peter Zijlstra Signed-off-by: K Prateek Nayak Reviewed-by: Tim Chen --- arch/x86/kernel/itmt.c | 56 ++++++++++++++++++++---------------------- 1 file changed, 27 insertions(+), 29 deletions(-) diff --git a/arch/x86/kernel/itmt.c b/arch/x86/kernel/itmt.c index ee43d1bd41d0..9cea1fc36c18 100644 --- a/arch/x86/kernel/itmt.c +++ b/arch/x86/kernel/itmt.c @@ -19,6 +19,7 @@ #include #include #include +#include #include #include #include @@ -34,45 +35,38 @@ static bool __read_mostly sched_itmt_capable; * of higher turbo frequency for cpus supporting Intel Turbo Boost Max * Technology 3.0. * - * It can be set via /proc/sys/kernel/sched_itmt_enabled + * It can be set via /sys/kernel/debug/x86/sched_itmt_enabled */ bool __read_mostly sysctl_sched_itmt_enabled; =20 -static int sched_itmt_update_handler(const struct ctl_table *table, int wr= ite, - void *buffer, size_t *lenp, loff_t *ppos) +static ssize_t sched_itmt_enabled_write(struct file *filp, + const char __user *ubuf, + size_t cnt, loff_t *ppos) { - unsigned int old_sysctl; - int ret; + ssize_t result; + bool orig; =20 guard(mutex)(&itmt_update_mutex); =20 - if (!sched_itmt_capable) - return -EINVAL; - - old_sysctl =3D sysctl_sched_itmt_enabled; - ret =3D proc_dointvec_minmax(table, write, buffer, lenp, ppos); + orig =3D sysctl_sched_itmt_enabled; + result =3D debugfs_write_file_bool(filp, ubuf, cnt, ppos); =20 - if (!ret && write && old_sysctl !=3D sysctl_sched_itmt_enabled) { + if (sysctl_sched_itmt_enabled !=3D orig) { x86_topology_update =3D true; rebuild_sched_domains(); } =20 - return ret; + return result; } =20 -static struct ctl_table itmt_kern_table[] =3D { - { - .procname =3D "sched_itmt_enabled", - .data =3D &sysctl_sched_itmt_enabled, - .maxlen =3D sizeof(unsigned int), - .mode =3D 0644, - .proc_handler =3D sched_itmt_update_handler, - .extra1 =3D SYSCTL_ZERO, - .extra2 =3D SYSCTL_ONE, - }, +static const struct file_operations dfs_sched_itmt_fops =3D { + .read =3D debugfs_read_file_bool, + .write =3D sched_itmt_enabled_write, + .open =3D simple_open, + .llseek =3D default_llseek, }; =20 -static struct ctl_table_header *itmt_sysctl_header; +static struct dentry *dfs_sched_itmt; =20 /** * sched_set_itmt_support() - Indicate platform supports ITMT @@ -98,9 +92,15 @@ int sched_set_itmt_support(void) if (sched_itmt_capable) return 0; =20 - itmt_sysctl_header =3D register_sysctl("kernel", itmt_kern_table); - if (!itmt_sysctl_header) + dfs_sched_itmt =3D debugfs_create_file_unsafe("sched_itmt_enabled", + 0644, + arch_debugfs_dir, + &sysctl_sched_itmt_enabled, + &dfs_sched_itmt_fops); + if (IS_ERR_OR_NULL(dfs_sched_itmt)) { + dfs_sched_itmt =3D NULL; return -ENOMEM; + } =20 sched_itmt_capable =3D true; =20 @@ -131,10 +131,8 @@ void sched_clear_itmt_support(void) =20 sched_itmt_capable =3D false; =20 - if (itmt_sysctl_header) { - unregister_sysctl_table(itmt_sysctl_header); - itmt_sysctl_header =3D NULL; - } + debugfs_remove(dfs_sched_itmt); + dfs_sched_itmt =3D NULL; =20 if (sysctl_sched_itmt_enabled) { /* disable sched_itmt if we are no longer ITMT capable */ --=20 2.34.1 From nobody Sun Dec 14 13:45:47 2025 Received: from NAM11-DM6-obe.outbound.protection.outlook.com (mail-dm6nam11on2062.outbound.protection.outlook.com [40.107.223.62]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 86B67242EF1 for ; Wed, 11 Dec 2024 18:57:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.107.223.62 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733943451; cv=fail; b=jlhJ8jn8MB/Wt9L9au13IpoYKnqV8uCG4OBmBaU6PFUJMYbexJXGD/9W0bpCmbB43slSBMljhxCTWYVbj0oBBXVVlDzV7yv0DRUHUBNb0seit1gSSzSmIrA1731lf9n43LaVs3i+F/o/tUM0MzEemOt7BQQ0qYKfRVRX9aVglhw= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733943451; c=relaxed/simple; bh=B51GHzOohB1lNQx5rAlHi5Y1NSmlRh8kDZVcZPnXxYY=; h=From:To:CC:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=hC3w8VGnkLYlMF7F/v6Hr5VS+eFYzP5ydYkhoJkiA5ZG4Xha4tiwe8hNVihxTLZ+HDX82kHhyETR3aXI3BqHvKHuPOGaqLAuFsqNhmIsHUlDI8ueAcvKFyVpFm3s3KptUawle5297qM/GCV6y331WskFpIu5qbNe3EZC8xM0fdU= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com; spf=fail smtp.mailfrom=amd.com; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b=DmKapwTL; arc=fail smtp.client-ip=40.107.223.62 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="DmKapwTL" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=DWKR5YEH28ke6xgjQlk/85VDgS0m2OtkebmBXAvdNjbmVzmNTO0j9o7QIBkB3ePw3i4jbnpUWhTA2XgqhHFuPJaGqxkfs7ar2RL6zWMVsqt8OwwRyCzeU2vLKkCQ+nmiuaF0bghRMU2tPYNaiEvoEc677XGaqMA8eNhEIeiHFgXdFjbwIoA7izjezqQrlD3nuvxx1LCjg1YI0thFU16aBay1zejxjluXBLwTf8XA5gRE8ls4ABY5rE3OXWa+a5zdqTUerm/LhHquUGdoz8hIn6TJgDb0psVKTB4QElHD02dHWzQMYSfk4jBLLIQgPpPpXuBnjxlOLqsfEvRLhBdybQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=o0jPE2mwkxamNBkoP+Y4ib5FhZsW/NDw4r9svhjcvoQ=; b=ORumpoeG6TxIkUI6ooXhIIiALUvJBb/hfo5ixIqu3pHEY8yPZgL1pG/ESQgPzVdbhuXMLuDJIjSfkct4FgOToFUa3tfWqoVe0Y5boBYXRrWCfGRBlcduDNc4fyt21vQq+qHIWq6LsmB2f9tJrXy6/jSbjVmVUjCMhUGfVOOk1HrshPzLCLgQIz0BGhQ5YiCmcpqC4dqQjbTJWB9oUwNhc/JLOSJ9scpwzaHLcmmqeOa/fG++yoj1htLbxxaYz0wj4UeUeKlTCn02BKUQ1C7ST/9E/sO/3T96O1RCgfelvKRK32s9W/6vxkQSIa0mi0JKEhtnGH+oGFDDc3aVzCdhyg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=linutronix.de smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=o0jPE2mwkxamNBkoP+Y4ib5FhZsW/NDw4r9svhjcvoQ=; b=DmKapwTLQ4xp4Fn7zgsOYld7CKEUU4F4dQIjKR07qOmCFNtFkjKCVy0M6eHbwoj7efI+sY6KYe8hDcX5CHE0fzEPPn7whx31WxgdXFlpG7brI3OP97wrd6KGznNsK6yheegCH8taIGuC7tcKLxVrzLjvjCBT4OzQD+GBoSq7f1I= Received: from PH0PR07CA0033.namprd07.prod.outlook.com (2603:10b6:510:e::8) by LV8PR12MB9406.namprd12.prod.outlook.com (2603:10b6:408:20b::20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8251.15; Wed, 11 Dec 2024 18:57:25 +0000 Received: from SA2PEPF000015C6.namprd03.prod.outlook.com (2603:10b6:510:e:cafe::a4) by PH0PR07CA0033.outlook.office365.com (2603:10b6:510:e::8) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.8230.15 via Frontend Transport; Wed, 11 Dec 2024 18:57:25 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=SATLEXMB04.amd.com; pr=C Received: from SATLEXMB04.amd.com (165.204.84.17) by SA2PEPF000015C6.mail.protection.outlook.com (10.167.241.196) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.8251.15 via Frontend Transport; Wed, 11 Dec 2024 18:57:24 +0000 Received: from BLRKPRNAYAK.amd.com (10.180.168.240) by SATLEXMB04.amd.com (10.181.40.145) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.39; Wed, 11 Dec 2024 12:57:17 -0600 From: K Prateek Nayak To: Thomas Gleixner , Ingo Molnar , Borislav Petkov , Dave Hansen , Peter Zijlstra , Juri Lelli , Vincent Guittot , , CC: "H. Peter Anvin" , Dietmar Eggemann , Steven Rostedt , Ben Segall , Mel Gorman , Valentin Schneider , "Rafael J. Wysocki" , Ricardo Neri , Tim Chen , Mario Limonciello , Meng Li , Huang Rui , "Gautham R. Shenoy" , K Prateek Nayak Subject: [PATCH 4/8] x86/topology: Remove x86_smt_flags and use cpu_smt_flags directly Date: Wed, 11 Dec 2024 18:55:47 +0000 Message-ID: <20241211185552.4553-5-kprateek.nayak@amd.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20241211185552.4553-1-kprateek.nayak@amd.com> References: <20241211185552.4553-1-kprateek.nayak@amd.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: SATLEXMB03.amd.com (10.181.40.144) To SATLEXMB04.amd.com (10.181.40.145) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SA2PEPF000015C6:EE_|LV8PR12MB9406:EE_ X-MS-Office365-Filtering-Correlation-Id: 38196cbc-fcda-46dd-7162-08dd1a15a6b6 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|82310400026|7416014|376014|1800799024|36860700013; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?v9HQSxzVLgRDqD6Lg/bDzwCajkxVHblcdQFKUA2WLbeq2a2C8fCpJ4zHVInd?= =?us-ascii?Q?tduJKTLm6BbWQVl7mihTUYEOPhNc2zbFcG181DpgPNg4ZNfxAsrt7jEYiFDP?= =?us-ascii?Q?boxr/yFrTpKM1BIi+7CpRFwMhsCPcSp+noO0wkEIh8tJxs5ue//3JGOB/IPV?= =?us-ascii?Q?iqPDyePU8FMlKDTUGOJAVYuYtO9pgqOpdH2000egGVFr7UmBm4nIm8WDn6Sh?= =?us-ascii?Q?hIUDu50fdCQ2F7AUxGF0FrVddYEJ4X03fhrlhDyt56GwO40lz1gM5d6vilyY?= =?us-ascii?Q?D+KePnc99YFhGEgM9KsR8KzBAo1r53UUynFmUk5xGEiosIpHBnbPETeaXE/U?= =?us-ascii?Q?Q9Lu2sOr1jN8IV0cB45hNyyuQiHEPuxfftuqdCJILagf59kJM0C5NPJgDqF8?= =?us-ascii?Q?nVLq8gTu6Yyjs8L9giuGN9KlLe0buAacXsJpDG7Mzkw9ydWs0wR8uqKtFoRP?= =?us-ascii?Q?qsgaGwVdWBS6bu8Kt7JAAegJ2htpbW94HS1nlHAEMhxKIJYNT7Fqg4Hlf+PE?= =?us-ascii?Q?+XwbwZjyN2eUSdzrsCzqxk6ANL/5QdeMK7eooMBAM8MaTqkkL3U/F6LUmEuB?= =?us-ascii?Q?vb7/J303gLZI2nIBNYllX9X4DUI5tOagLIEKjoLZoIZcq7dwP+/6b2gFGSrl?= =?us-ascii?Q?sUg3FXhaVEkNyMLr4IkWqqOT+HEU3XjNOaRcO1t+RUGHbJwtwkWUyxgDObgr?= =?us-ascii?Q?tkcTr8Ze3WBfWmY8oI4tQqUGS8mqRMXPHEvMR8yRgRJTgxSNfWwKrYq62R7f?= =?us-ascii?Q?8ka6BUDlXqYG/mgdwTGrOOGiLQd2P2rbaITbCpDSpSGZ8p/gGnMJIhDpvOPi?= =?us-ascii?Q?Rea8byNQPQpYRsASHm8xoF/G4P4xf5m4N7xzR8Qve7DHx4quha/NPqmujjz4?= =?us-ascii?Q?fR6Fg97i40y+fatOZfPi1PDJ2/v0TBYqSAglMc3EOunPQ2El3FD64SdsnPvi?= =?us-ascii?Q?UuEP5w7Pwq6Fm9Gx73t0J/kCSWyipKHvozV+GoUTICq5smrG65uKbA3U+raX?= =?us-ascii?Q?7ey4j/B5sw5d0w7VvxUNQOOvbr+w7eWcanndXzmeYkbwfvZjWVp0PDaZB1rB?= =?us-ascii?Q?wXlzcw8oCuG6uKzwAHas+IjYFZLNpvTTAxkKOcXPLIc0PE5gzi720ebdA78u?= =?us-ascii?Q?wWivditj8tg/ETFhw/XA/H8wrD7Osb/it/vj5/tUYcrEpBKBJPD4M76SJ/ft?= =?us-ascii?Q?oeD3J1rvSBAgeXPHfe1tRVUxduHtzj3cloXBvSQef5w1gVKZUjefhrc8m68Z?= =?us-ascii?Q?UxtGU64hRmh6wQtUScLUFMHFIL8aEHcs4DWOuLXc2vb3iS4cE/OkLFsmMVuJ?= =?us-ascii?Q?s5GFaG+pNCi9G5O6Jylp4eYtOTMptlY9Ep5uUFgNI3XpCoVL7rKF7svg5Ln2?= =?us-ascii?Q?oKKKgwnjwewBrBcQNfu2ZIO6DUaNH8SxtkdPAk45thvYvSI3TtTcpTtnAXWH?= =?us-ascii?Q?bmT8cpP0fhFWbZjwSQaNkESVlMjiheII?= X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:SATLEXMB04.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230040)(82310400026)(7416014)(376014)(1800799024)(36860700013);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Dec 2024 18:57:24.9150 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 38196cbc-fcda-46dd-7162-08dd1a15a6b6 X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[SATLEXMB04.amd.com] X-MS-Exchange-CrossTenant-AuthSource: SA2PEPF000015C6.namprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: LV8PR12MB9406 Content-Type: text/plain; charset="utf-8" x86_*_flags() wrappers were introduced with commit d3d37d850d1d ("x86/sched: Add SD_ASYM_PACKING flags to x86 ITMT CPU") to add x86_sched_itmt_flags() in addition to the default domain flags for SMT and MC domain. commit 995998ebdebd ("x86/sched: Remove SD_ASYM_PACKING from the SMT domain flags") removed the ITMT flags for SMT domain but not the x86_smt_flags() wrappers which directly returns cpu_smt_flags(). Remove x86_smt_flags() and directly use cpu_smt_flags() to derive the flags for SMT domain. No functional changes intended. Signed-off-by: K Prateek Nayak Reviewed-by: Tim Chen --- arch/x86/kernel/smpboot.c | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/arch/x86/kernel/smpboot.c b/arch/x86/kernel/smpboot.c index b5a8f0891135..6e300897b7ee 100644 --- a/arch/x86/kernel/smpboot.c +++ b/arch/x86/kernel/smpboot.c @@ -482,12 +482,6 @@ static int x86_core_flags(void) return cpu_core_flags() | x86_sched_itmt_flags(); } #endif -#ifdef CONFIG_SCHED_SMT -static int x86_smt_flags(void) -{ - return cpu_smt_flags(); -} -#endif #ifdef CONFIG_SCHED_CLUSTER static int x86_cluster_flags(void) { @@ -519,7 +513,7 @@ static void __init build_sched_topology(void) =20 #ifdef CONFIG_SCHED_SMT x86_topology[i++] =3D (struct sched_domain_topology_level){ - cpu_smt_mask, x86_smt_flags, SD_INIT_NAME(SMT) + cpu_smt_mask, cpu_smt_flags, SD_INIT_NAME(SMT) }; #endif #ifdef CONFIG_SCHED_CLUSTER --=20 2.34.1 From nobody Sun Dec 14 13:45:47 2025 Received: from NAM11-BN8-obe.outbound.protection.outlook.com (mail-bn8nam11on2062.outbound.protection.outlook.com [40.107.236.62]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9AD7E236922 for ; Wed, 11 Dec 2024 18:57:50 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.107.236.62 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733943472; cv=fail; b=sryyDLNz0Xk52a4TDL/PqGC5XO5kcdVStUdV1rwQOKzKRIz9GwjL8RCAd4Nacr5Gpt5m1e5QXxHQn8LMogE4We88Cii67ERJXavk9hWFOAh1zH80VV7JKIGkekkmrSPj24JlQOiZ5xc5wlgTxUHXLl0W7DhEneR3krtGxAZAFbo= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733943472; c=relaxed/simple; bh=jfbFhRvWdBJCN7Ujoprw7/0NCMifpSme5/gHONEu90g=; h=From:To:CC:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=WQbrhFCj51tYuZ0QnJOG802fF+vGhHRA5doim3VgUC6+9Ai1VZGsYa8D4iMbHgykOpy3g989YO1iNN0Fgt/7A1r6XNbH8Y6xUOpVvIGOy5nOgSD9ea48DT6hgMfVEZ07tH4P28P2PBapzmTIiDTNV2O7rQKauEghvps+KDaOKi4= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com; spf=fail smtp.mailfrom=amd.com; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b=fvarJAJa; arc=fail smtp.client-ip=40.107.236.62 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="fvarJAJa" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=tkwE0I7ubU93CRnL1tsPPhiKhYxyJV6Cz9w19V1WYG5s5Z7ZX3DT45VXS/HPPN0GUDnDlD3UHxWITYSpxelUmjzloHtbeNSHSEZBOe3WvF5ZSkwR3zRs4l7/w/7r9oH7nSQHM45Fq0twIKcK6mdKWDs1nLz9HwKtvvLOMbyCZhE/sH9Yp7u2+7iuCU5ufhzlnfhdq/V4ORmI9VnMyLEgoJxGsScNoxOV8xwiWjo5o2F8LHrUp5e5v5Eq8+oeMAztLRIN/2fVYmfYoekMnJ862eiXlzzgFECPeWN4+MfeokqiNN3R/YBnLeJgBhe/ftssrbzeCPZXjFZCzbqkUVA7nQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=3+84XmnE6pEcSd/r4oC/BHypl2a4yj76z0zfZN2bEkU=; b=bgEs0ZcVvyKaBTsU/+gv2A1Xi9BEAU3cVRn2OZ32DJXiT7QcbFI8u7fp/VwSqQ6vYCOvddjKLxw24vuYOwNuL+HGs+XUt7bQ9j2HFRGP0jB03j9BqZT4INiWR9HIVatBZO69rxQUEa3st678dwqZMmhGE8ve14p1E5zmaWlOJNMLpJavYOwvb2BXeZCt9tz7XcuYrhzu2+iByF+VdxOMLjk1dlac1c/IZ5LSk4hvEmkh1gwepWnryns9zvr3ysIG3FDjRFF5wW/WpcgIQlSLqJFgyWQTTK72NE99qpD7bza9ZwQsCMPccjMZvy/BWAB70lDy8Cp6x97X3/LoW7JTnQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=linutronix.de smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=3+84XmnE6pEcSd/r4oC/BHypl2a4yj76z0zfZN2bEkU=; b=fvarJAJac+BC1gMoAe9tMWLK9nvfbr252jYWICqwiUqtpmttXWMht5gsnuwSQ02BLtLEhYcQkwRTYDIddMDhiNE/lprjfVuqwe+hAA2YQwgr6Exokro3ahyU7OooOBCcl4KCItwsRaf9ZFdE3pAip5RGfn+ZCGE8a/5F0mZZgIc= Received: from PH7P220CA0109.NAMP220.PROD.OUTLOOK.COM (2603:10b6:510:32d::11) by DM4PR12MB5939.namprd12.prod.outlook.com (2603:10b6:8:6a::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8230.22; Wed, 11 Dec 2024 18:57:45 +0000 Received: from CY4PEPF0000EE35.namprd05.prod.outlook.com (2603:10b6:510:32d:cafe::92) by PH7P220CA0109.outlook.office365.com (2603:10b6:510:32d::11) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.8251.15 via Frontend Transport; Wed, 11 Dec 2024 18:57:45 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=SATLEXMB04.amd.com; pr=C Received: from SATLEXMB04.amd.com (165.204.84.17) by CY4PEPF0000EE35.mail.protection.outlook.com (10.167.242.41) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.8251.15 via Frontend Transport; Wed, 11 Dec 2024 18:57:44 +0000 Received: from BLRKPRNAYAK.amd.com (10.180.168.240) by SATLEXMB04.amd.com (10.181.40.145) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.39; Wed, 11 Dec 2024 12:57:37 -0600 From: K Prateek Nayak To: Thomas Gleixner , Ingo Molnar , Borislav Petkov , Dave Hansen , Peter Zijlstra , Juri Lelli , Vincent Guittot , , CC: "H. Peter Anvin" , Dietmar Eggemann , Steven Rostedt , Ben Segall , Mel Gorman , Valentin Schneider , "Rafael J. Wysocki" , Ricardo Neri , Tim Chen , Mario Limonciello , Meng Li , Huang Rui , "Gautham R. Shenoy" , K Prateek Nayak Subject: [PATCH 5/8] x86/topology: Use x86_sched_itmt_flags for PKG domain unconditionally Date: Wed, 11 Dec 2024 18:55:48 +0000 Message-ID: <20241211185552.4553-6-kprateek.nayak@amd.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20241211185552.4553-1-kprateek.nayak@amd.com> References: <20241211185552.4553-1-kprateek.nayak@amd.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: SATLEXMB03.amd.com (10.181.40.144) To SATLEXMB04.amd.com (10.181.40.145) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CY4PEPF0000EE35:EE_|DM4PR12MB5939:EE_ X-MS-Office365-Filtering-Correlation-Id: 076cc923-30a9-4975-445c-08dd1a15b2a6 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|82310400026|36860700013|1800799024|7416014|376014; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?9Cn0KD5xVITBkHvy/l8JdlccBD8WnYu1/JXlHrgGlBIcnqSzrNf+eJBTZ6DS?= =?us-ascii?Q?dVyU/Tiu8qWyktkox5WyxK1ffxN9z+Z/Vam5YbSSjzPdV1BYJpPE9vRrxL8H?= =?us-ascii?Q?24tH6nLicliKlpM0rkFuJNTcoI6kBfMeQkF+uzSgHp6IroBlQbaiIbgJ44NV?= =?us-ascii?Q?QX4yJzFNg071WeHbblk0r5OlLXFg6gnNak/gYirUk1+eBcLAh6I9epgqtBHf?= =?us-ascii?Q?iN1FjI++lxUO1RIfmVXlrz1j+hTdDCCOnBQ1i6EG/GLjmE0NUagwOqDE5dSu?= =?us-ascii?Q?BXZrTERbtuQ6WVT860XG+QGTd6Dmugu5bXu3FwRQ78W/nQOw6reJbrkFVn/F?= =?us-ascii?Q?MGXvn9Ky7ekVVSGxR3380rPTqVkDrM27Fs0OLsiwKcVC90vL+Klk3g9YuxiT?= =?us-ascii?Q?PfMLNuPUC1H5NIOKAes2BfSTXYr+5fvOTKSZWr3eR6Q1ybCUzhThSOEYlf68?= =?us-ascii?Q?jT7vbEJC/bmGONO8QJMjhW76lEWeWjK8jSTW8zzIPgfo8B7z5DNG1DnmKhBH?= =?us-ascii?Q?ibRv4dBs70xmSwrL7h2uQLQZJWc+IcyMfunH7HxyEx/Z2A7C+8/cDFMla7Lp?= =?us-ascii?Q?SG5SASn5ZgXw4mCV4QHsCNP0OLfTu/3H3shcKUl5c1hAieTGAbb/zX7+UI++?= =?us-ascii?Q?gJ3N7o8Z76L77klaTd5Z9DjTEYqB7GgZi+CYVJ/ta6Cgf+22bcOY5QlkGa7l?= =?us-ascii?Q?yoUZNvVAeTBP2IzzMe/z3+g6psYaHc+qt/24pYCvktUwsHDSPS+QUlMLHZ59?= =?us-ascii?Q?bH5noqkpo8u3QIPpZ2wBkThWsVEX3/8LMlB+luUU5ZmLQaSGWy8syZeimdBh?= =?us-ascii?Q?onoE08lLe4UBSZo2gzIFemz7Xae4wksb9DPz7LwK+ne7o3xvvLZSob5pmodz?= =?us-ascii?Q?v3IyOgGfQmqSLrc6iwrwkHZCQ4YoeSuWIZcbzw4pXWGtLGhVQQ3FroUozHhf?= =?us-ascii?Q?mUG5Bg6n14nmYPKd1prO4bzoy/9uDfzppOWPeRi7amRfztA1pedGDavD7NqN?= =?us-ascii?Q?JTP22CeKhbiZE10KIRoYCDxXs1+zl66HA+oAKEZzZWhBw48QWTb2CujhtktB?= =?us-ascii?Q?Zu4XHNJYDa79usfAOfbtF7jtf3/INx+e1guyK/fEjvmVdahon56N4HNX4Kgs?= =?us-ascii?Q?D5pPD7PIMQstKEdYtVqjRey6nbtKU2Se2qRO6B6QLLDmQlZZskZjHB+JAAJy?= =?us-ascii?Q?Afx798YXFFkAvGYGv5mKtSNX9ywTT06unrsCSv6KoKL/kRK7GX2kPxdIt4YE?= =?us-ascii?Q?EuIQR15295TIwVx7vVwC4mODKt2aB9/2APUJOpyxirhHeqnHhof5hLDyYAAL?= =?us-ascii?Q?J7nVsemgBJSuvjiJhP54vA7u0ZEgjZeH5YnEBCVXteV8vGpop6hTlOPXXzZ4?= =?us-ascii?Q?V+I2GuB5CMr74PCsDeLRd2cZlBJxcCg119Lkvzl6ecAE0PB8hamNm3taC563?= =?us-ascii?Q?+/j1C9t1TQRprtzH6cO9yiFKIZQnJx1g?= X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:SATLEXMB04.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230040)(82310400026)(36860700013)(1800799024)(7416014)(376014);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Dec 2024 18:57:44.8636 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 076cc923-30a9-4975-445c-08dd1a15b2a6 X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[SATLEXMB04.amd.com] X-MS-Exchange-CrossTenant-AuthSource: CY4PEPF0000EE35.namprd05.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR12MB5939 Content-Type: text/plain; charset="utf-8" x86_sched_itmt_flags() returns SD_ASYM_PACKING if ITMT support is enabled by the system. Without ITMT support being enabled, it returns 0 similar to current x86_die_flags() on non-Hybrid systems (!X86_HYBRID_CPU and !X86_FEATURE_AMD_HETEROGENEOUS_CORES) On Intel systems that enable ITMT support, either the MC domain coincides with the PKG domain, or in case of multiple MC groups within a PKG domain, either Sub-NUMA Cluster (SNC) is enabled or the processor features Hybrid core layout (X86_HYBRID_CPU) which leads to three distinct possibilities: o If PKG and MC domains coincide, PKG domain is degenerated by sd_parent_degenerate() when building sched domain topology. o If SNC is enabled, PKG domain is never added since "x86_has_numa_in_package" is set and the topology will instead contain NODE and NUMA domains. o On X86_HYBRID_CPU which contains multiple MC groups within the PKG, the PKG domain requires x86_sched_itmt_flags(). Thus, on Intel systems that contains multiple MC groups within the PKG and enables ITMT support, the PKG domain requires x86_sched_itmt_flags(). In all other cases PKG domain is either never added or is degenerated. Thus, returning x86_sched_itmt_flags() unconditionally at PKG domain on Intel systems should not lead to any functional changes. On AMD systems with multiple LLCs (MC groups) within a PKG domain, enabling ITMT support requires setting SD_ASYM_PACKING to the PKG domain since the core rankings are assigned PKG-wide. Core rankings on AMD processors is currently set by the amd-pstate driver when Preferred Core feature is supported. A subset of systems that support Preferred Core feature can be detected using X86_FEATURE_AMD_HETEROGENEOUS_CORES however, this does not cover all the systems that support Preferred Core ranking. Detecting Preferred Core support on AMD systems requires inspecting CPPC Highest Perf on all present CPUs and checking if it differs on at least one CPU. Previous suggestion to use a synthetic feature to detect Preferred Core support [1] was found to be non-trivial to implement since BSP alone cannot detect if Preferred Core is supported and by the time AP comes up, alternatives are patched and setting a X86_FEATURE_* then is not possible. Since x86 processors enabling ITMT support that consists multiple non-NUMA MC groups within a PKG requires SD_ASYM_PACKING flag set at the PKG domain, return x86_sched_itmt_flags unconditionally for the PKG domain. Since x86_die_flags() would have just returned x86_sched_itmt_flags() after the change, remove the unnecessary wrapper and pass x86_sched_itmt_flags() directly as the flags function. Link: https://lore.kernel.org/lkml/20241203221746.GKZ0-Dii5rnZppkM_e@fat_cr= ate.local/ [1] Fixes: f3a052391822 ("cpufreq: amd-pstate: Enable amd-pstate preferred core= support") Signed-off-by: K Prateek Nayak Reviewed-by: Tim Chen --- arch/x86/kernel/smpboot.c | 11 +---------- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/arch/x86/kernel/smpboot.c b/arch/x86/kernel/smpboot.c index 6e300897b7ee..ef63b1c0b491 100644 --- a/arch/x86/kernel/smpboot.c +++ b/arch/x86/kernel/smpboot.c @@ -489,15 +489,6 @@ static int x86_cluster_flags(void) } #endif =20 -static int x86_die_flags(void) -{ - if (cpu_feature_enabled(X86_FEATURE_HYBRID_CPU) || - cpu_feature_enabled(X86_FEATURE_AMD_HETEROGENEOUS_CORES)) - return x86_sched_itmt_flags(); - - return 0; -} - /* * Set if a package/die has multiple NUMA nodes inside. * AMD Magny-Cours, Intel Cluster-on-Die, and Intel @@ -533,7 +524,7 @@ static void __init build_sched_topology(void) */ if (!x86_has_numa_in_package) { x86_topology[i++] =3D (struct sched_domain_topology_level){ - cpu_cpu_mask, x86_die_flags, SD_INIT_NAME(PKG) + cpu_cpu_mask, x86_sched_itmt_flags, SD_INIT_NAME(PKG) }; } =20 --=20 2.34.1 From nobody Sun Dec 14 13:45:47 2025 Received: from NAM12-BN8-obe.outbound.protection.outlook.com (mail-bn8nam12on2068.outbound.protection.outlook.com [40.107.237.68]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D49A623693F for ; Wed, 11 Dec 2024 18:58:09 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.107.237.68 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733943491; cv=fail; b=ibWe6TWH8C2KbTGpnhGV6+IaIj+y8kByYcRtM4MlU2bdeR3G6dk0DezwLtpqdYmYlSyYKVXHXdmNyOoLAKs1MrNX4Rs/2fymp9/1upTPjWHzHruCGAN0R5kZPmmr8awdCnynuV8wAuYz67qW5B+LvxApVAgQd9uvN+h5w0RJmGE= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733943491; c=relaxed/simple; bh=orVU3DpIUNHREqyvCb7Lx9japWgoQxgoSZMfx8sJvHQ=; h=From:To:CC:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=LXtyYdHfp+F1WChIMp+T3SG2KHfpCGg//OL9LYmreLnErrQtFUaZ5adcETwM46AR4TvjYA9ZXb+kXaX5KQ3b+gykPR8dLBwXy2Pd6gVUboTf8J/qO0vd0hzHE2kdWR8D1Y+1SxFumN+54b2dW5hudc6F21caUKmCALWmmMgeEWk= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com; spf=fail smtp.mailfrom=amd.com; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b=bC3U23W0; arc=fail smtp.client-ip=40.107.237.68 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="bC3U23W0" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=m9j6KSi+srA+xXuZIKHmjlmhklSNWwKvIpUKNQGrya9DuLowyuf1FWYkyegCZ0nyFqD59kxq9Hr8P8aXBpWnN0Y9WQDacN/Uce0IGKjQoqkOddGxh9MXKgJ/i9fp2SoziRNnUNpuvO8Bk6JOor249dd1C3YMUlw0qkYLHy97drOgP7eLMmmyJteYujuxyAWFruRqs3PaRhrf0xwV5YbjovZaDAU51L0lq5KIyhey3zPeEM/UCrDOUvfg0X0hse7Bpg78LhyM8bYPCZuWhz4CBVqwu0HoRTsP6K2rFzD8GPWJz0RTAmG5Gr6o+RXB+qw6DyqZQDYCEZQ09WWfTIw6AQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=czR4LFJ0nw50Wlh0iMcFwavhn1+iCqr2DFUhIwIVjBM=; b=eTr17aSl1KYeBclj4YJiHr9O5oefLW60XXavz8+3ozodxUnNjCWySdF7HoMzahB8QN9PX2+jhlF733A4V8oNY8vUa/mrfXtfl8l6UmbtmSSn+u1+Fuxlh0Wb1x7gi4z1OMwtGq5X82tsMFTJf8wIfSMD/sUaqWfIYmeEsOcJX9Av76yYuaf7M60wRmzUd6xRwCyLRcEFXrSU38jNMXBEcCrFsuUP+U2JVgt7g2li1ai2Z92FEcpTXSjHiQqHNPEA6lHbdysH8nqU25pMVDiefhp+JSN58Tq7/fKDKutYnU+ypbnh+Y6Mj9tuEk0s1FLs7gT7tP/6voQV81m1B/PSZg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=linutronix.de smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=czR4LFJ0nw50Wlh0iMcFwavhn1+iCqr2DFUhIwIVjBM=; b=bC3U23W02cje9QEVuqHdSqvDoBSKYLpu01ZLT33rbxnhOYQItOWrqzpaGNhcAL//f7kZt6HCQzZKPNk8LOTviI1IU19P4Zd6CJW/BUGH3DxfZKgJiH4b1SZ0eQ5hPRgbbNsMuyS7gtrQkGWQwGUcw7QAvF1ZC7mV6ZeIkEvGMGk= Received: from PH7P221CA0075.NAMP221.PROD.OUTLOOK.COM (2603:10b6:510:328::26) by DS7PR12MB9473.namprd12.prod.outlook.com (2603:10b6:8:252::5) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8251.16; Wed, 11 Dec 2024 18:58:05 +0000 Received: from CY4PEPF0000EE37.namprd05.prod.outlook.com (2603:10b6:510:328:cafe::55) by PH7P221CA0075.outlook.office365.com (2603:10b6:510:328::26) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.8251.16 via Frontend Transport; Wed, 11 Dec 2024 18:58:04 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=SATLEXMB04.amd.com; pr=C Received: from SATLEXMB04.amd.com (165.204.84.17) by CY4PEPF0000EE37.mail.protection.outlook.com (10.167.242.43) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.8251.15 via Frontend Transport; Wed, 11 Dec 2024 18:58:04 +0000 Received: from BLRKPRNAYAK.amd.com (10.180.168.240) by SATLEXMB04.amd.com (10.181.40.145) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.39; Wed, 11 Dec 2024 12:57:57 -0600 From: K Prateek Nayak To: Thomas Gleixner , Ingo Molnar , Borislav Petkov , Dave Hansen , Peter Zijlstra , Juri Lelli , Vincent Guittot , , CC: "H. Peter Anvin" , Dietmar Eggemann , Steven Rostedt , Ben Segall , Mel Gorman , Valentin Schneider , "Rafael J. Wysocki" , Ricardo Neri , Tim Chen , Mario Limonciello , Meng Li , Huang Rui , "Gautham R. Shenoy" , K Prateek Nayak Subject: [PATCH 6/8] sched/fair: Do not compute NUMA Balancing stats unnecessarily during lb Date: Wed, 11 Dec 2024 18:55:49 +0000 Message-ID: <20241211185552.4553-7-kprateek.nayak@amd.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20241211185552.4553-1-kprateek.nayak@amd.com> References: <20241211185552.4553-1-kprateek.nayak@amd.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: SATLEXMB03.amd.com (10.181.40.144) To SATLEXMB04.amd.com (10.181.40.145) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CY4PEPF0000EE37:EE_|DS7PR12MB9473:EE_ X-MS-Office365-Filtering-Correlation-Id: 8e166dad-20ee-4bc9-058b-08dd1a15be2b X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|82310400026|376014|1800799024|7416014|36860700013; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?NWCHPaQ9aIo0SrRYs+MCxmMB1o2dXeHQk0+4MlF9kCVZhvSr/BqhDp8madsb?= =?us-ascii?Q?jMVt71oi+3yICdOYdWP2WiebHMyn7rkBkZPvMOP+sTfZl6JFExAANsBj5O5a?= =?us-ascii?Q?o9nwiI8I8pq3gI1wLtdzuIWrKyPG55GLGs1OUOxsDl+ZaZdeUC3pHxFE5ajU?= =?us-ascii?Q?5pchSRXkVb2TlnM0Uy1RsUcWGrwucrG80N/Ngdko8sWJgPVN9oxNpqidebga?= =?us-ascii?Q?KE8kEoE8BOckIzzyUYzZeq0cIfiL+aE7lscectur9qA6ElzDfHbXjVKO5Ohj?= =?us-ascii?Q?7aRgQoVHMpeZGxq/9tk5AW2uTfxqljbA1uVWdans3EEfdrhwAXatm4pri/Hw?= =?us-ascii?Q?AXBAyTdpv4T01F9xbCbdS5M7K1atJAAET7cJPVePhgNVXkMKvEhNAmO15jsS?= =?us-ascii?Q?Vs3Q/P3kI5yScsQDK8n0kw9VMegOGEC8mmHxrkLv6zuN4gOhzaTNzElbJ9WX?= =?us-ascii?Q?2u9Fgj+IdSMTvQTDBFW/+SlkkyPI6DiOxb34z/gHb1KoSFkYPKw1RTakclTy?= =?us-ascii?Q?nd4LnxxSukwEefscCxnvT1BK5DRlipSyIwbN7DULGAWEYcEMMVAxreEarH5z?= =?us-ascii?Q?nSJH1H1JD5cVaqipsp6eXL/p+uQeFBQf5fii4GzvPvM9Du/o8sTPCcHngoF/?= =?us-ascii?Q?0JlZ5+NQGhz8rNvPSJVt32xsF+614DAYYn4XeQ+iobEl22Ca+YnpdQ+h6exZ?= =?us-ascii?Q?N6o5L9L0jL+Q9dHO9Nqz7wXVe6qbLfor94PoTavp/BeAw1dNhK6DdbmT+8KI?= =?us-ascii?Q?MHksdWQrCPEmg2mFHawESnhjwcWW9WjRvdGKZMRWKVehCKBnE+WW/G02kYeU?= =?us-ascii?Q?ZZNZmu0wdf3RN0ZstrvRhz4ax1mMAK6mNtYo6CJeebgB7UDPYkNhfSNUIqq5?= =?us-ascii?Q?NdPCe9JrSJHinoHz0KX21TQBy0IAVrmhIe5yj7QOZ8PxI53WhVCmvy8hlqDu?= =?us-ascii?Q?liOUqmKCOpd039X01KQQWxUyvPQTokadUtj3Rt/4q/CrYAw1YB8BWd+UkBm1?= =?us-ascii?Q?Ftm6Utv44i7FljcaV5ZV56XSpbRt14/b2KYLE1iiEkQt9ykHGOA/SFJPg+Yh?= =?us-ascii?Q?6W2h7h2PUKPohDBVQyF6Yz3xFAhWUqmFTNYo42W4thkZ+ys2aAq6mm/L9S9V?= =?us-ascii?Q?hbLM84xmoescAU9z1JFqQ5xrm4HChwgUBSws3/iLKFZWy+cSUdwP0MwVSb3m?= =?us-ascii?Q?Ju4cSEeaqhRK6VFWF7wrlujBq6527Dd+OanxUknaufT1Yxb4pk/kvthWndKD?= =?us-ascii?Q?wsBNpxlnuZipQjzbH8d1dirgZu80BJ5hhhr/p3iJpaevZW6yDbB5ZbB+5G2x?= =?us-ascii?Q?QLLpLTUmfGdMaG56iJ0UXaX/RJlsTe26r8TD4RDOS2gUqL+7bSGxSVibJBu9?= =?us-ascii?Q?FdDhj7saIjQoV97lVH+FzIdkntl+hZNyPqMfbk8BMzdDW02OrJ3sqBROvizy?= =?us-ascii?Q?lL4Rdz9+Eetmq4yvItMdQzfa+wST4K1L?= X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:SATLEXMB04.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230040)(82310400026)(376014)(1800799024)(7416014)(36860700013);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Dec 2024 18:58:04.1885 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 8e166dad-20ee-4bc9-058b-08dd1a15be2b X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[SATLEXMB04.amd.com] X-MS-Exchange-CrossTenant-AuthSource: CY4PEPF0000EE37.namprd05.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS7PR12MB9473 Content-Type: text/plain; charset="utf-8" Aggregate nr_numa_running and nr_preferred_running when load balancing at NUMA domains only. While at it, also move the aggregation below the idle_cpu() check since an idle CPU cannot have any preferred tasks. Signed-off-by: K Prateek Nayak Reviewed-by: Shrikanth Hegde Reviewed-by: Vincent Guittot --- kernel/sched/fair.c | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c index 2c4ebfc82917..ec2a79c8d0e7 100644 --- a/kernel/sched/fair.c +++ b/kernel/sched/fair.c @@ -10340,7 +10340,7 @@ static inline void update_sg_lb_stats(struct lb_env= *env, bool *sg_overloaded, bool *sg_overutilized) { - int i, nr_running, local_group; + int i, nr_running, local_group, sd_flags =3D env->sd->flags; =20 memset(sgs, 0, sizeof(*sgs)); =20 @@ -10364,10 +10364,6 @@ static inline void update_sg_lb_stats(struct lb_en= v *env, if (cpu_overutilized(i)) *sg_overutilized =3D 1; =20 -#ifdef CONFIG_NUMA_BALANCING - sgs->nr_numa_running +=3D rq->nr_numa_running; - sgs->nr_preferred_running +=3D rq->nr_preferred_running; -#endif /* * No need to call idle_cpu() if nr_running is not 0 */ @@ -10377,10 +10373,17 @@ static inline void update_sg_lb_stats(struct lb_e= nv *env, continue; } =20 +#ifdef CONFIG_NUMA_BALANCING + /* Only fbq_classify_group() uses this to classify NUMA groups */ + if (sd_flags & SD_NUMA) { + sgs->nr_numa_running +=3D rq->nr_numa_running; + sgs->nr_preferred_running +=3D rq->nr_preferred_running; + } +#endif if (local_group) continue; =20 - if (env->sd->flags & SD_ASYM_CPUCAPACITY) { + if (sd_flags & SD_ASYM_CPUCAPACITY) { /* Check for a misfit task on the cpu */ if (sgs->group_misfit_task_load < rq->misfit_task_load) { sgs->group_misfit_task_load =3D rq->misfit_task_load; --=20 2.34.1 From nobody Sun Dec 14 13:45:47 2025 Received: from NAM11-DM6-obe.outbound.protection.outlook.com (mail-dm6nam11on2086.outbound.protection.outlook.com [40.107.223.86]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 11EB523692C for ; Wed, 11 Dec 2024 18:58:30 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.107.223.86 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733943512; cv=fail; b=gqsgA/Jbo9N1oYsCsU5NrSrHSvD4M0wGqFYKUq1Jpj7nWqLMIxPngQfC0Tp2nGc3qbqRVtsmTsR1zn8HTms38IfK338UBXzgUBfUeyOEiOhiYDUVbI6/dYtdpQ7PnpXANtM/OthENuNMObqTlYcLi/kl77bnSxvM44cBVHDobsw= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733943512; c=relaxed/simple; bh=Uw3bB+CuLgj4X7zY/PDQKWKN6sIEDhLWBytko2m18T4=; h=From:To:CC:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=s6WZCi8+2Eho2LkFhIvzKIMv7UBZLFxk0TVRUXZUie5NCPUCovqX542FdFDbETVLNpWNjZ5MLasuYwq1Jr4MHdFggbPPlt0oWIpXKaxYgTRtie2lSFMxDiZuwmiSW06k7WvC1cD+fwhwe6042CW1iPZSSD4xpJDbHNMNlrwWPoI= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com; spf=fail smtp.mailfrom=amd.com; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b=wD3E3avN; arc=fail smtp.client-ip=40.107.223.86 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="wD3E3avN" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=d8vZdQ07nk0QeK8spTeS66ShHg63g+z/aBj1Ow9fkZ+02GNnlktKg+k+qJcPR/ua3+sR4nm3FNrYMxe6q6EpR+XA67G6y0cQGR2JzWuk+FHPKrSSxHLKcDvL4UGuKYD8FsPueLUN4/rzjosQ3ZnubmGkUErcOeoa+F6dyvBjgBQ7KTclmP9zSSy+D1uJuT0EUwPKv+tgshTbjSmSseO4PEwpdacJJfyPHirlEgX+waNdi/4UwtKo05uq8MCFhi1w9G3SJdbjPQHUzSko0Rlk/hof6NPZls8M3ReHlnPE/lHotwwOlI27SXIdk3QgudjLm5G8wT0QPVe0pCkgUkT0qA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=YSLnuv+OaEpnFLdjjGptwzgvpg9YCKa0ZWNocp6fJ/A=; b=UfVYLWuIBxNPLRCjP91R6XK5tMFDf+c+SCjH8/T957pCd687NOZk7MuaJ83inJwEDbleQAjaj+ATuufBBJLbp7fuOEcQB1gczcuZauxIueflAGEnEgUOVY7ZGXIhIrBLxmQEY3LlUHNiwo5EYXWIX0xOp4hSM4Nj5qcB5gueegQXKsI/OsDXT3FYPF+9ZPph+H8OS44eIl4NtyXPMsObfYjKK5w1PAcMx2c44Mxvz91kXfV1SiM4xzec9ouaw6k/2ygU82tMZFkZVoEhuTynJnKnmXAr+6AJuxkeiJfuI9uThm/B9hggPCgLqt+XKDuZbPgHmeDdnN5yuFCeqvRteQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=linutronix.de smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=YSLnuv+OaEpnFLdjjGptwzgvpg9YCKa0ZWNocp6fJ/A=; b=wD3E3avNE7SHKwZqqXbKdnVTiQr5oMA/rHRRwYgmk7oir8VRBYd/73jS4Hc7UJxYynChEuTBdNjFetVCf/eyIiEX+0tlvQe/T98oVqR9Hgo35Wke++YKAKMQbAjkmGDm9x4O96pNG/KEzMARdiSR1C5agQkGP0+Sjr2goZy3280= Received: from DM6PR17CA0023.namprd17.prod.outlook.com (2603:10b6:5:1b3::36) by DS7PR12MB6046.namprd12.prod.outlook.com (2603:10b6:8:85::20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8251.16; Wed, 11 Dec 2024 18:58:26 +0000 Received: from CY4PEPF0000EE33.namprd05.prod.outlook.com (2603:10b6:5:1b3:cafe::5) by DM6PR17CA0023.outlook.office365.com (2603:10b6:5:1b3::36) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.8251.15 via Frontend Transport; Wed, 11 Dec 2024 18:58:24 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=SATLEXMB04.amd.com; pr=C Received: from SATLEXMB04.amd.com (165.204.84.17) by CY4PEPF0000EE33.mail.protection.outlook.com (10.167.242.39) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.8251.15 via Frontend Transport; Wed, 11 Dec 2024 18:58:24 +0000 Received: from BLRKPRNAYAK.amd.com (10.180.168.240) by SATLEXMB04.amd.com (10.181.40.145) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.39; Wed, 11 Dec 2024 12:58:16 -0600 From: K Prateek Nayak To: Thomas Gleixner , Ingo Molnar , Borislav Petkov , Dave Hansen , Peter Zijlstra , Juri Lelli , Vincent Guittot , , CC: "H. Peter Anvin" , Dietmar Eggemann , Steven Rostedt , Ben Segall , Mel Gorman , Valentin Schneider , "Rafael J. Wysocki" , Ricardo Neri , Tim Chen , Mario Limonciello , Meng Li , Huang Rui , "Gautham R. Shenoy" , K Prateek Nayak Subject: [PATCH 7/8] sched/fair: Do not compute overloaded status unnecessarily during lb Date: Wed, 11 Dec 2024 18:55:50 +0000 Message-ID: <20241211185552.4553-8-kprateek.nayak@amd.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20241211185552.4553-1-kprateek.nayak@amd.com> References: <20241211185552.4553-1-kprateek.nayak@amd.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: SATLEXMB03.amd.com (10.181.40.144) To SATLEXMB04.amd.com (10.181.40.145) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CY4PEPF0000EE33:EE_|DS7PR12MB6046:EE_ X-MS-Office365-Filtering-Correlation-Id: f95855c8-1285-4217-8065-08dd1a15ca21 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|1800799024|36860700013|376014|7416014|82310400026; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?Vwnv1NialWr21Zk1qA3XIWaT43WGbt4OKBLjAIE9Hinkzl0Fu9vay3SPiFpt?= =?us-ascii?Q?PTuj0VTTAPEMmVp/c6WIm3AEJJr7e1mSJV05gIOylxXa/OwNxaTnSrVWq4Nk?= =?us-ascii?Q?zeLFAmIOcUv8RCyesehBMBfL41+W+4o5LHUU+pBlPpvxZTuxsfyaGv0Cf7Sc?= =?us-ascii?Q?+ig9RPEDzkk7ASfW8ysB8THhovp6421bu4/KHHIT4lpu+vjOady211tj9KM2?= =?us-ascii?Q?6Z8Q6oKFmjHYDzVKqd8Jj+Y7dC+A3YjWb63nQopjtUV7bz0Ar/0snhU6Vxb2?= =?us-ascii?Q?El+F8kEplNq3jyr/9LgDrnm37jEOzABm7lSWH/1vwbYhytjPYXvRik5MF1mV?= =?us-ascii?Q?rOiCSYPp9/NpaqGTIbv0acfZqR+6Cd45Cv78lYuicTgt76f190AmdWnKq9iE?= =?us-ascii?Q?vGKe4FXoYzL+XfuDpdeUD2L9DybUidj9y1ti1TdQ7iJPpvIMiiqdBcxgomgs?= =?us-ascii?Q?w5w7GWFnqoelSLbuR9GX3hYw8Oi2WXdjvnyhuygZrnsWqI4fcqpPJr/jrtTC?= =?us-ascii?Q?fkvUnNOKGJ15EdkYu/PnkpOA/7r4QYnKyyns37w2tybWVVMF/k6Ocyd0XYYX?= =?us-ascii?Q?M30nmO/7LQMzopDXpanFJWXzwOI06z4P38LqmfS1kmBI9Wz/lYdEd4vTAnNR?= =?us-ascii?Q?xPIRIYOSv0qVN9ViN/F/FcJPZEunYg+Wg+LYIhmiiyFQwJgb6MGru+7qRyxT?= =?us-ascii?Q?S0hLzUcdMLppEs8j1KS95IjVHMgq6Kg47o/1/yYCw68XVoY1kQGHAiJHyweU?= =?us-ascii?Q?RqevqFgRgR2vh/cyzwdzasTn33BIEGQv7+iOVvtWl90F1FmIVXPoNiyQAUzK?= =?us-ascii?Q?wcS7d2+Q6OWg4AeWjjZ3JTITLVLJDlDEW2FNA+SRmTbONqIva8WMhNSmvfqs?= =?us-ascii?Q?3j24jjHBt5SKk0AWHI2aEARisI558jj6NSjUgcoclmpYSldM7VUxzKas94Gr?= =?us-ascii?Q?eTtrGZ8xzpmdRm28lM7uJnClMKmTbm8wyrYBdaweceWExdD0uRZYMRT2FRMj?= =?us-ascii?Q?18gS9LIa5CWp9p2HjNAeXx4+oXDtC2nvFihMaCgkZHDaXmmvj0nnDXVeO53M?= =?us-ascii?Q?HzAbj5rcmYgSafk2ilTulUdn2CkP+ecFljV9c+dZ8UCK+4xVTcSlF7blKdld?= =?us-ascii?Q?qUqIF/QKiMlkk9iJ86w4me32h8d57BoOmE2IhZGikWWiVMDCOYrW9U4K6bDE?= =?us-ascii?Q?JPQibHXsLUO81D2T/nGURY5tuxWZqoyEdYg84zle+sY7vjqng1msqpit326N?= =?us-ascii?Q?Wh/1DQ27GDr4+aJbPzm+pFKqchieGHsEOT+TQ7qNtxCSuQP1caJebYb2NQvb?= =?us-ascii?Q?pAj+NuEY3H9oNFFh7g5kg7ILyVrL0glxG+B3z+pGebfDzlFolK0Cv0TwrNQH?= =?us-ascii?Q?0R+3ePRB4xpxOko1Dk/aRuolqnabn28goRvOg305eJodgy2aDRwk2GcnqN0b?= =?us-ascii?Q?lvx9xBQ7vNwFIT5IM8JhUakgv0n46X4y?= X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:SATLEXMB04.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230040)(1800799024)(36860700013)(376014)(7416014)(82310400026);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Dec 2024 18:58:24.2588 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: f95855c8-1285-4217-8065-08dd1a15ca21 X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[SATLEXMB04.amd.com] X-MS-Exchange-CrossTenant-AuthSource: CY4PEPF0000EE33.namprd05.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS7PR12MB6046 Content-Type: text/plain; charset="utf-8" Only set sg_overloaded when computing sg_lb_stats() at the highest sched domain since rd->overloaded status is updated only when load balancing at the highest domain. While at it, move setting of sg_overloaded below idle_cpu() check since an idle CPU can never be overloaded. Signed-off-by: K Prateek Nayak Reviewed-by: Shrikanth Hegde Reviewed-by: Vincent Guittot --- kernel/sched/fair.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c index ec2a79c8d0e7..3f36805ecdca 100644 --- a/kernel/sched/fair.c +++ b/kernel/sched/fair.c @@ -10358,9 +10358,6 @@ static inline void update_sg_lb_stats(struct lb_env= *env, nr_running =3D rq->nr_running; sgs->sum_nr_running +=3D nr_running; =20 - if (nr_running > 1) - *sg_overloaded =3D 1; - if (cpu_overutilized(i)) *sg_overutilized =3D 1; =20 @@ -10373,6 +10370,10 @@ static inline void update_sg_lb_stats(struct lb_en= v *env, continue; } =20 + /* Overload indicator is only updated at root domain */ + if (!env->sd->parent && nr_running > 1) + *sg_overloaded =3D 1; + #ifdef CONFIG_NUMA_BALANCING /* Only fbq_classify_group() uses this to classify NUMA groups */ if (sd_flags & SD_NUMA) { --=20 2.34.1 From nobody Sun Dec 14 13:45:47 2025 Received: from NAM10-MW2-obe.outbound.protection.outlook.com (mail-mw2nam10on2042.outbound.protection.outlook.com [40.107.94.42]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 1BAEA236953 for ; Wed, 11 Dec 2024 18:58:49 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.107.94.42 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733943531; cv=fail; b=p0r6qj8PerB3acq/D/7qE3Gf89tdsJa6IJCsWi5hFrpy6qH5uqXT6LrLO+2xHwOcFsVdJskUXhgEWptKnaMB2xxJkwFyoSb/wsW88aCMMoGrb8PGBhRXkvYBjQGd+aZ6F3vIZCpS9uh5EOhXEhPIeMwMXCpdbqM1CXFMdRl7ntQ= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733943531; c=relaxed/simple; bh=odguSQaQf53uKboP+a0h6cMou9KDoHk9Y0jAM8k3OEI=; h=From:To:CC:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=Da4NAawiZgwP4py4flNKyE3vn//6rv6jUph/lwdQc3kDixUhGTp5KjiF7YJa+3cK5XQ9PeePZW6mJCZ1VSz5wKSayg4ttJT04CeAakXyMs0Bso1Bc22kZ3Obwd1+4xzxO4J5YRPWWudaccUZZaVLlJsVOUXbNtDeTN/zwo6zTpI= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com; spf=fail smtp.mailfrom=amd.com; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b=IJX1KwAl; arc=fail smtp.client-ip=40.107.94.42 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="IJX1KwAl" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=nGIAmi+u+xNcdCxMFNNg507dsJDu7HyTdFYxScDA2i7paJHOPNfSIGRGr+NGNTgS8xDTm1XzIPiPd0Vbqz0QlOCxR43dPmrp/G/0SHAlWJfGV8SzztfIin1PdFyy6yNPbRnYGZMANhMrJ3tUsrmmboDjYDxfl1rtnsV9s7TyDcljnskrYtQPIkyBenEJOEvPLqnFds5ym2DJCqTKcDqTVha76+tAnuA4tOBKkUFtMKGcH6S/Ar6Zg91JZ7ELTOQD00r8vrJRYV25MsR9Y6dlKRsuZyXb6s7r80WpRK1zBxim6Vo8BpOun7BZ+ycXooWSFzk9FzRxA8zmshutcd2lqA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=WsEJBbHEFXd8YT5f5YBngvC9jmpr7LmqyTd0BJHkDYw=; b=qCoAnNpDVf2PMoay1DZdfr9XFuW/fI9QBfA/RHpiQOUBQm8iw56DAq+8qsLS/Q6fx9N34jqtfa4sxq+1UdKgirVDmznZZbCsb6EXuo0eSWF55xMLe1w0X5lZc+oigSxJ/UGjh1iqonPZA35QC0rjH5zTvzdeYANqFt/bCIU6/KLUmViTi+MvHK2RnODOev6fQjwjnxynMxAOCCnWnkIwAAuL7yar9sEX+NUxF3zyxFwMwdLExWp58GVwoeYmJEF6+LRh3AQicxn63Gl0jajMzb1vKhxYj0f2G0VlqTe8/ZZuL7tTEd2cP1xeaN8JT/V2tXe5m4aDnis0te83f2NgjQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=linutronix.de smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=WsEJBbHEFXd8YT5f5YBngvC9jmpr7LmqyTd0BJHkDYw=; b=IJX1KwAlerxJh78AYuSfdgn/DwTKsQJ9Puej9fxaS+zNeskgPinqiGWFXs5/ruZP/1dOoPRFajvnwlGOVnNMAUmAou8Jhg1TiVTNr5FHd0Ozm/EM8IHgdtFBXww5lk2hZjVNdo6mddR2WKDRbRl+bxF8IRQJFk4La2KcFRUB9zU= Received: from DM6PR02CA0153.namprd02.prod.outlook.com (2603:10b6:5:332::20) by SN7PR12MB7108.namprd12.prod.outlook.com (2603:10b6:806:2a3::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8251.15; Wed, 11 Dec 2024 18:58:43 +0000 Received: from CY4PEPF0000EE37.namprd05.prod.outlook.com (2603:10b6:5:332:cafe::78) by DM6PR02CA0153.outlook.office365.com (2603:10b6:5:332::20) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.8251.15 via Frontend Transport; Wed, 11 Dec 2024 18:58:43 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=SATLEXMB04.amd.com; pr=C Received: from SATLEXMB04.amd.com (165.204.84.17) by CY4PEPF0000EE37.mail.protection.outlook.com (10.167.242.43) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.8251.15 via Frontend Transport; Wed, 11 Dec 2024 18:58:43 +0000 Received: from BLRKPRNAYAK.amd.com (10.180.168.240) by SATLEXMB04.amd.com (10.181.40.145) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.39; Wed, 11 Dec 2024 12:58:36 -0600 From: K Prateek Nayak To: Thomas Gleixner , Ingo Molnar , Borislav Petkov , Dave Hansen , Peter Zijlstra , Juri Lelli , Vincent Guittot , , CC: "H. Peter Anvin" , Dietmar Eggemann , Steven Rostedt , Ben Segall , Mel Gorman , Valentin Schneider , "Rafael J. Wysocki" , Ricardo Neri , Tim Chen , Mario Limonciello , Meng Li , Huang Rui , "Gautham R. Shenoy" , K Prateek Nayak Subject: [RFC PATCH 8/8] sched/fair: Uncache asym_prefer_cpu and find it during update_sd_lb_stats() Date: Wed, 11 Dec 2024 18:55:51 +0000 Message-ID: <20241211185552.4553-9-kprateek.nayak@amd.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20241211185552.4553-1-kprateek.nayak@amd.com> References: <20241211185552.4553-1-kprateek.nayak@amd.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: SATLEXMB03.amd.com (10.181.40.144) To SATLEXMB04.amd.com (10.181.40.145) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CY4PEPF0000EE37:EE_|SN7PR12MB7108:EE_ X-MS-Office365-Filtering-Correlation-Id: 8878dd80-0422-433f-b060-08dd1a15d58d X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|1800799024|376014|36860700013|7416014|82310400026; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?uqKff//UNUdo4zLeMjU1ZACFQl3jOLPEa3JgN9/X1w8P45smwWa9T9sOMUEx?= =?us-ascii?Q?DvCrghFWV97R13WNHeVcJkAYMVKP//QM2yTBX7gAg9SF1VPrgmURlxRTkHQW?= =?us-ascii?Q?u65CemaM3RCjA8TCgPuO0jPAjlL1MweX+l+KkLf74ZHiz8UorH01+1E9CtGR?= =?us-ascii?Q?rd2IeO1xOZ5Nwq0oBjNQVRNqmAB4xOJ8Hk/lbS5mwrqCGVSkDEHIn1GOgCIG?= =?us-ascii?Q?6CuumEOB3kSamj0qXauXmzmIO/en4L5m0ktUlZXj0IJrXC/xFfh3fxl85U4Q?= =?us-ascii?Q?ZjD7X3MB2I8bUkjd6KtuaonMbsrYbvtQp7hqtkyMZ5v+qUlW+fRkNbDwWYMd?= =?us-ascii?Q?CKBJ9VQqwv/4KlbgXNC/JB//GPpAzgjcmW9/rQRAmqq7XLsNxiA7WPoRX70T?= =?us-ascii?Q?eBr5TKBOZSaX2sUJIbHO1wdPNlrnsLnL75Q1EHSv00pzgrmxHPOcSij3H5mQ?= =?us-ascii?Q?gz6jP+uUD91oOCjEGwHrfsTXd3fOh0KP7lNREEq33NUqAiKv5QhnbXmgiotP?= =?us-ascii?Q?PzgIEKLW9yUPoJfmIDCB5z/H7IIUNE3siPrA3Oj7FxG8DfbQCtFztVqtzhP2?= =?us-ascii?Q?EVymDGjBSmGT1Hc+rbro5ZxAkocC6R8Wcle8rh4UDhzZngjDhAjjGbNqZRoT?= =?us-ascii?Q?+jdczIDWnP/8Dfsps/KndLtbiJOVZjYDXtJvUcy9t0C0nEInE0mecfkYcPee?= =?us-ascii?Q?dbgIQlDZYYo6nPY5nYvmqJ9IGPtL1XTa4YjuuqfH1DLJYZ2o8kbGpDwWaPjc?= =?us-ascii?Q?ncue87E4Z8YKfu7XHYMSJIST1NiTdde93bO9tkV+5w/NhYOoomuLMVhqt8fH?= =?us-ascii?Q?RgCnzaTj9jpf6axrBNoQkIGEmTTfrZCm4jgHqoCQUQBSMnLqKaR7du/5jNtn?= =?us-ascii?Q?COlxHkVqa/yLpWNrpU+tq9IA8mAXko0TQ463mNKY64jK/mgmWlwzoX9r6E/n?= =?us-ascii?Q?yde3cggAZi2plnRugdHkBZMP4VApxBFvm2QWztT/2S4D6h0CNVagS1tyJqoF?= =?us-ascii?Q?JqzdIPl3gAcTMvy4j8cTbyWOJmtRs0IXjeVEgAHANJjIfGlfuX+8y6Blurq3?= =?us-ascii?Q?+N3RreiibvtiW1XAE9HINxbvyyazZ+EVtE+E+dAX2e5aDm/bLTonxG321em5?= =?us-ascii?Q?nwPIWpKD+7+KQSicF+e7qjce+6ReYAqT1sEdTYIHpV+UyMvjDgZk8PjMNDn9?= =?us-ascii?Q?1rwFU/zy6mHN5Hw4T7W2jzZnhtRQ2kdX6G53IE/tIGdt3i5QK429H7j+t/dM?= =?us-ascii?Q?+knekRl1tfJb6R1FGhzjhngR6ApkgdKdSZsOVJiwazbqTnEd5hbzH3XpPKy8?= =?us-ascii?Q?ee5hplaKOHwfxMg3yznf+CDKSl2TbmoYiLJUBqU54JpDP7WHp+r86nK9Yb1C?= =?us-ascii?Q?05+E4gJNbWVu+idFAPWWrHTMyHbrr5L39Tmh6L5K1syn3yFxrfa6gNaPLso8?= =?us-ascii?Q?4mHlIfy0oj7MiRxIQcENr80IrZAX7r2v?= X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:SATLEXMB04.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230040)(1800799024)(376014)(36860700013)(7416014)(82310400026);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Dec 2024 18:58:43.4227 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 8878dd80-0422-433f-b060-08dd1a15d58d X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[SATLEXMB04.amd.com] X-MS-Exchange-CrossTenant-AuthSource: CY4PEPF0000EE37.namprd05.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN7PR12MB7108 Content-Type: text/plain; charset="utf-8" On AMD processors supporting dynamic preferred core ranking, the asym_prefer_cpu cached in sched_group can change dynamically. Since asym_prefer_cpu is cached when the sched domain hierarchy is built, updating the cached value across the system would require rebuilding the sched domain which is prohibitively expensive. All the asym_prefer_cpu comparisons in the load balancing path are only carried out post the sched group stats have been updated after iterating all the CPUs in the group. Uncache the asym_prefer_cpu and compute it while sched group statistics are being updated as a part of sg_lb_stats. Fixes: f3a052391822 ("cpufreq: amd-pstate: Enable amd-pstate preferred core= support") Signed-off-by: K Prateek Nayak --- kernel/sched/fair.c | 21 +++++++++++++++++++-- kernel/sched/sched.h | 1 - kernel/sched/topology.c | 15 +-------------- 3 files changed, 20 insertions(+), 17 deletions(-) diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c index 3f36805ecdca..166b8e831064 100644 --- a/kernel/sched/fair.c +++ b/kernel/sched/fair.c @@ -9911,6 +9911,8 @@ struct sg_lb_stats { unsigned int sum_nr_running; /* Nr of all tasks running in the group */ unsigned int sum_h_nr_running; /* Nr of CFS tasks running in the group */ unsigned int idle_cpus; /* Nr of idle CPUs in the= group */ + unsigned int asym_prefer_cpu; /* CPU with highest asym priority */ + int highest_asym_prio; /* Asym priority of asym_prefer_cpu */ unsigned int group_weight; enum group_type group_type; unsigned int group_asym_packing; /* Tasks should be moved to preferred CP= U */ @@ -10243,7 +10245,7 @@ sched_group_asym(struct lb_env *env, struct sg_lb_s= tats *sgs, struct sched_group (sgs->group_weight - sgs->idle_cpus !=3D 1)) return false; =20 - return sched_asym(env->sd, env->dst_cpu, group->asym_prefer_cpu); + return sched_asym(env->sd, env->dst_cpu, sgs->asym_prefer_cpu); } =20 /* One group has more than one SMT CPU while the other group does not */ @@ -10324,6 +10326,17 @@ sched_reduced_capacity(struct rq *rq, struct sched= _domain *sd) return check_cpu_capacity(rq, sd); } =20 +static inline void +update_sg_pick_asym_prefer(struct sg_lb_stats *sgs, int cpu) +{ + int asym_prio =3D arch_asym_cpu_priority(cpu); + + if (asym_prio > sgs->highest_asym_prio) { + sgs->asym_prefer_cpu =3D cpu; + sgs->highest_asym_prio =3D asym_prio; + } +} + /** * update_sg_lb_stats - Update sched_group's statistics for load balancing. * @env: The load balancing environment. @@ -10345,6 +10358,7 @@ static inline void update_sg_lb_stats(struct lb_env= *env, memset(sgs, 0, sizeof(*sgs)); =20 local_group =3D group =3D=3D sds->local; + sgs->highest_asym_prio =3D INT_MIN; =20 for_each_cpu_and(i, sched_group_span(group), env->cpus) { struct rq *rq =3D cpu_rq(i); @@ -10358,6 +10372,9 @@ static inline void update_sg_lb_stats(struct lb_env= *env, nr_running =3D rq->nr_running; sgs->sum_nr_running +=3D nr_running; =20 + if (sd_flags & SD_ASYM_PACKING) + update_sg_pick_asym_prefer(sgs, i); + if (cpu_overutilized(i)) *sg_overutilized =3D 1; =20 @@ -10479,7 +10496,7 @@ static bool update_sd_pick_busiest(struct lb_env *e= nv, =20 case group_asym_packing: /* Prefer to move from lowest priority CPU's work */ - return sched_asym_prefer(sds->busiest->asym_prefer_cpu, sg->asym_prefer_= cpu); + return sched_asym_prefer(busiest->asym_prefer_cpu, sgs->asym_prefer_cpu); =20 case group_misfit_task: /* diff --git a/kernel/sched/sched.h b/kernel/sched/sched.h index aef716c41edb..a3f0d326bd11 100644 --- a/kernel/sched/sched.h +++ b/kernel/sched/sched.h @@ -2047,7 +2047,6 @@ struct sched_group { unsigned int group_weight; unsigned int cores; struct sched_group_capacity *sgc; - int asym_prefer_cpu; /* CPU of highest priority in group */ int flags; =20 /* diff --git a/kernel/sched/topology.c b/kernel/sched/topology.c index 9c405f0e7b26..20aa087710f0 100644 --- a/kernel/sched/topology.c +++ b/kernel/sched/topology.c @@ -1302,7 +1302,7 @@ static void init_sched_groups_capacity(int cpu, struc= t sched_domain *sd) WARN_ON(!sg); =20 do { - int cpu, cores =3D 0, max_cpu =3D -1; + int cpu, cores =3D 0; =20 sg->group_weight =3D cpumask_weight(sched_group_span(sg)); =20 @@ -1314,19 +1314,6 @@ static void init_sched_groups_capacity(int cpu, stru= ct sched_domain *sd) #endif } sg->cores =3D cores; - - if (!(sd->flags & SD_ASYM_PACKING)) - goto next; - - for_each_cpu(cpu, sched_group_span(sg)) { - if (max_cpu < 0) - max_cpu =3D cpu; - else if (sched_asym_prefer(cpu, max_cpu)) - max_cpu =3D cpu; - } - sg->asym_prefer_cpu =3D max_cpu; - -next: sg =3D sg->next; } while (sg !=3D sd->groups); =20 --=20 2.34.1