From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.remote-ip=198.175.65.16 ARC-Seal: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; cv=none; t=1792443069; b=htBff3UOQDG18WDsQk0y7Q4og3GUAGxA9ACHxpItdt+tYdQeLyYgt3zYFP2lFTW5pDz3 MCObiilsGPTgcQnvt1uGmEsp8DFAFGxThJSZYr7Wzr10st+ekCFn4FIe0LIMp0v64rJb1 QQfKf5cIEaSOC++CsEKzQPCK4EtvDhQvUpXcQHR+eOMsOVUMpSqkcHwqVZOttLhPujztT AeFKIrwJxJTpc8WGvGjRh9YbUoGGAYhcB6yE8FiKjU8BurwNUq6jk9Ti6CM1BPpfXRrrH LcXZ0912fsIMuJGd6rk2Is6J9Tc/YOT2mcWAM6E7fdnfKi2wI+tDQoHsfr6egVfLXtw== ARC-Message-Signature: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; c=relaxed/relaxed; t=1792443069; h=DMARC-Filter:DKIM-Signature:X-CSE-ConnectionGUID:X-CSE-MsgGUID: X-IronPort-AV:X-IronPort-AV:Received:X-CSE-ConnectionGUID: X-CSE-MsgGUID:X-ExtLoop1:X-IronPort-AV:Received:From:To:Cc:Subject: Date:Message-ID:X-Mailer:In-Reply-To:References:MIME-Version: Content-Transfer-Encoding; bh=5glsYhiJQvTewTue11QClnQyXGe14nbkKIVgpnHFobk=; b=dn0Y21Tl4VHqsEcA6U9eql86XZmq4g/y2xByZ8QZDTfi3N4gbx5UeVS9SwgrwZW8Qbpi aMhJrhL4MZIQgMUdYHDJ54xOXpnBcZgZB5l0SCyJdi8O+cMQ/KkPA4ItPSGVvXl0yMlAN OeDVEhHvqCPr/7KjYYlfZnMoWOBcFI0MGYyM4DtBpi8LWffhxFRZia/2fgXO4rQ0inXFx QAqEJ9DFpGWxYnz/Dl9rxaWrA7/r5f0ulCwDxKv79/0U5CsPscQYUo3JxzlO/IQlIxVU+ uswRUSz74Yp/Bn4j7hn/9xSc9NwvwUkpwAAwvfi1RCsjpvg8sRhJC214O7ervZqO1Uw== ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass header.from=intel.com; spf=pass smtp.mailfrom=intel.com; dkim=pass header.d=intel.com header.i=@intel.com header.a=rsa-sha256 header.s=Intel header.b=Th71LcTv; arc=none smtp.remote-ip=198.175.65.16 Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.16]) (using TLSv1.2 with cipher ECDHE-ECDSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 2D1A660A2615 for ; Mon, 19 Oct 2026 20:50:58 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.2 smtp.subspace.kernel.org 2D1A660A2615 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=intel.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key, unprotected) header.d=intel.com header.i=@intel.com header.a=rsa-sha256 header.s=Intel header.b=Th71LcTv DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1792443058; x=1823979058; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=coFynmodb4SSI9urQSNuuDWxQ3sofyrL/q0tMcF+blI=; b=Th71LcTv8JRyB/5dXW6PwHiCKIdMoR7cvQzbC+DqZUu1zApcSbTj70ly Vb6OcLfdrMg+KnoDGwO14OIPymgQfMNwNVX0RptgCQ4wOESLdaw+SWzYd 2933+SbuDZoXi9KmluD+rufsrmRLEbt8WioGsW+cAfLfSlYCM/Uf4gBju TuvS8Z6+Jy3uMQSISdCa41JmJOvZ/kznttcN0L1/KZqfAq6S5M//gCzNi vKr8O3NvFWYbXUcJgVqjyOBw75ad3hBrYtTpBomlc0VmO9MRc/w/cXZsg JnlzJlUBcHB8EmTditEvwHXafuNTut6A1zXuPknqSu4OhaJ5+6CIe66Vb g==; X-CSE-ConnectionGUID: Wn/pmBhrTeOeLPlUYbjPwA== X-CSE-MsgGUID: iwj/6uvsRX2AImMuT1SVVg== X-IronPort-AV: E=McAfee;i="6800,10657,11940"; a="1131224" X-IronPort-AV: E=Sophos;i="6.27,168,1787036400"; d="scan'208";a="1131224" Received: from fmviesa009.fm.intel.com ([10.60.135.149]) by orvoesa108.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 19 Oct 2026 13:50:50 -0700 X-CSE-ConnectionGUID: noYXbo5ORu6B1dFqiuvVhw== X-CSE-MsgGUID: oPLdZMh3SZKefgEuN2eUog== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.27,168,1787036400"; d="scan'208";a="3008663" Received: from spandruv-mobl5.amr.corp.intel.com (HELO agluck-desk3.intel.com) ([10.124.220.87]) by smtpauth.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 19 Oct 2026 13:50:50 -0700 From: Tony Luck To: Fenghua Yu , Reinette Chatre , Maciej Wieczor-Retman , Peter Newman , James Morse , Babu Moger , Drew Fustini , Dave Martin , Chen Yu , David E Box , x86@kernel.org Cc: Christoph Hellwig , linux-kernel@vger.kernel.org, patches@lists.linux.dev, Tony Luck Subject: [PATCH v14 14/24] x86,fs/resctrl: Handle systems where AET is the only monitoring feature Date: Mon, 19 Oct 2026 13:50:21 -0700 Message-ID: <20261019205031.16977-15-tony.luck@intel.com> X-Mailer: git-send-email 2.56.0 In-Reply-To: <20261019205031.16977-1-tony.luck@intel.com> References: <20261019205031.16977-1-tony.luck@intel.com> X-Mailing-List: patches@lists.linux.dev List-Id: List-Post: List-Help: List-Subscribe: List-Unsubscribe: List-Owner: Precedence: list MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Application Energy Telemetry (AET) is not enumerated via CPUID. Its presence is discovered on the first mount of the resctrl file system by querying the pmt_telemetry driver. resctrl is initialized at boot only if CPUID enumeration finds allocation or monitoring features. On a system that supports monitoring (X86_FEATURE_CQM) but no L3 monitoring features, get_rdt_mon_resources() reports that no monitoring is supported. If the system also does not support allocation, resctrl is not initialized and AET can never be used. Assume that AET may be supported whenever the system supports monitoring. If AET is not supported either, the mount finds neither allocation nor monitoring support. rdt_get_tree() then lets the mount succeed with nothing to allocate or monitor, but does not mark resctrl as mounted. Fail the mount with an error message instead, before any of the file system is set up. Signed-off-by: Tony Luck --- v14: New commit message. --- arch/x86/kernel/cpu/resctrl/internal.h | 2 +- arch/x86/kernel/cpu/resctrl/core.c | 22 +++++++++++++--------- arch/x86/kernel/cpu/resctrl/monitor.c | 4 +--- fs/resctrl/rdtgroup.c | 8 ++++++-- 4 files changed, 21 insertions(+), 15 deletions(-) diff --git a/arch/x86/kernel/cpu/resctrl/internal.h b/arch/x86/kernel/cpu/resctrl/internal.h index 086282a82fca..c038b7d80ce3 100644 --- a/arch/x86/kernel/cpu/resctrl/internal.h +++ b/arch/x86/kernel/cpu/resctrl/internal.h @@ -226,7 +226,7 @@ union l3_qos_abmc_cfg { void rdt_ctrl_update(void *arg); -int rdt_get_l3_mon_config(struct rdt_resource *r); +void rdt_get_l3_mon_config(struct rdt_resource *r); bool rdt_cpu_has(int flag); diff --git a/arch/x86/kernel/cpu/resctrl/core.c b/arch/x86/kernel/cpu/resctrl/core.c index b32fa143283e..0262174df7ad 100644 --- a/arch/x86/kernel/cpu/resctrl/core.c +++ b/arch/x86/kernel/cpu/resctrl/core.c @@ -991,7 +991,7 @@ static __init bool get_rdt_alloc_resources(void) static __init bool get_rdt_mon_resources(void) { struct rdt_resource *r = &rdt_resources_all[RDT_RESOURCE_L3].r_resctrl; - bool ret = false; + bool l3_mon_found = false; /* Any monitoring at all? */ if (!cpu_feature_enabled(X86_FEATURE_CQM)) @@ -1001,27 +1001,31 @@ static __init bool get_rdt_mon_resources(void) /* Any of the L3 monitoring features? */ if (!cpu_feature_enabled(X86_FEATURE_CQM_LLC)) - return false; + goto out; if (rdt_cpu_has(X86_FEATURE_CQM_OCCUP_LLC)) { resctrl_enable_mon_event(QOS_L3_OCCUP_EVENT_ID, false, 0, NULL); - ret = true; + l3_mon_found = true; } if (rdt_cpu_has(X86_FEATURE_CQM_MBM_TOTAL)) { resctrl_enable_mon_event(QOS_L3_MBM_TOTAL_EVENT_ID, false, 0, NULL); - ret = true; + l3_mon_found = true; } if (rdt_cpu_has(X86_FEATURE_CQM_MBM_LOCAL)) { resctrl_enable_mon_event(QOS_L3_MBM_LOCAL_EVENT_ID, false, 0, NULL); - ret = true; + l3_mon_found = true; } if (rdt_cpu_has(X86_FEATURE_ABMC)) - ret = true; + l3_mon_found = true; - if (!ret) - return false; + if (l3_mon_found) + rdt_get_l3_mon_config(r); + else + pr_info("CPUID reports L3 monitoring support, but no L3 monitoring features found\n"); - return !rdt_get_l3_mon_config(r); +out: + /* Non-L3 features, e.g. AET, may be supported */ + return true; } static __init void __check_quirks_intel(void) diff --git a/arch/x86/kernel/cpu/resctrl/monitor.c b/arch/x86/kernel/cpu/resctrl/monitor.c index 82a555855c1b..bd45d72d1d05 100644 --- a/arch/x86/kernel/cpu/resctrl/monitor.c +++ b/arch/x86/kernel/cpu/resctrl/monitor.c @@ -431,7 +431,7 @@ static void __init intel_rdt_mbm_apply_quirk(u32 num_rmid) mbm_cf = mbm_cf_table[cf_index].cf; } -int __init rdt_get_l3_mon_config(struct rdt_resource *r) +void __init rdt_get_l3_mon_config(struct rdt_resource *r) { struct rdt_hw_resource *hw_res = resctrl_to_arch_res(r); unsigned int mbm_offset; @@ -519,8 +519,6 @@ int __init rdt_get_l3_mon_config(struct rdt_resource *r) intel_rdt_mbm_apply_quirk(num_rmid); r->mon_capable = true; - - return 0; } static void resctrl_abmc_set_one_amd(void *arg) diff --git a/fs/resctrl/rdtgroup.c b/fs/resctrl/rdtgroup.c index 79d4ddc3d64d..ecdb5da50179 100644 --- a/fs/resctrl/rdtgroup.c +++ b/fs/resctrl/rdtgroup.c @@ -3193,6 +3193,11 @@ static int rdt_get_tree(struct fs_context *fc) goto out; } + if (!resctrl_alloc_capable() && !resctrl_mon_capable()) { + ret = invalfc(fc, "No allocation or monitoring features are available or enabled"); + goto out; + } + ret = setup_rmid_lru_list(); if (ret) goto out; @@ -3251,8 +3256,7 @@ static int rdt_get_tree(struct fs_context *fc) if (resctrl_mon_capable()) resctrl_arch_enable_mon(); - if (resctrl_alloc_capable() || resctrl_mon_capable()) - resctrl_mounted = true; + resctrl_mounted = true; if (resctrl_is_mbm_enabled()) { r = resctrl_arch_get_resource(RDT_RESOURCE_L3); -- 2.56.0