From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.12]) (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 CBE5F39281F for ; Wed, 29 Jul 2026 17:28:13 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=198.175.65.12 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785346096; cv=none; b=EyELyGn46p78QP8ZZwyd/aaN1NVdOusfYAQjAC8iwB/GqTtOEcWC9UQcxFDyYBHqIpF/NSHdUV171LmnGEvxTBHKi+S7AoVTIAaB1d91U3/UACzOiOFcSmJNVgAxn3NEn3mqmAxESMadpGDfV+2RLJBcC6utK6hQe2TsaZ/D3rQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785346096; c=relaxed/simple; bh=iInLczJVZpheVwDr4rEXHLOGHettNkNVQv8dhk6hc0A=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=VWCwy0E3D56IUHW0JUQQTX8Invhr4WqgM/K41bkc0zRa0ZPp0E2wgRfEVYlh4Qvj9cfGeQg31gZ+od2GM5uhxm8aUPp4fkgsoxuKVvmZCwSNm8936JPK9JyeFKB5wTcbaeme/J2XsrijJenJZdDOOwgmwhvt1QQP/7+0YatTXEk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com; spf=pass smtp.mailfrom=intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=XPzkymFI; arc=none smtp.client-ip=198.175.65.12 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) header.d=intel.com header.i=@intel.com header.b="XPzkymFI" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1785346093; x=1816882093; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=iInLczJVZpheVwDr4rEXHLOGHettNkNVQv8dhk6hc0A=; b=XPzkymFI/fRN+ho4SXP+lnxQUbp4X++Qa3I50Ftzx2cD9DLZADKMW88v RMJ3N8MNf/NsFp8JnedLWX8Y1mRy55sBXlnzO+GnbLkSyr7UOSpFXEdmK e0k7/SfSu+kK45wN9zKJVg7xT5lMWkUeY5xORducFmMx8RzPNljL6Im0s XZtj+qbyfs4hDg0yJgeBf3HiuHMw/zrSkVYtYdYBXUy6Czoz0nwaBt4Dl /i7U6xg7S8U3bFEBUQ5N4XIMgU2vOq2nFMWD69zCNpX2i+yJzAFVfY36i xOb9WZn3xIXoSdoSBnrotYgy979cV8bjaoDJFw17FSl2Sh3+9Ok/uCkL0 w==; X-CSE-ConnectionGUID: a2yZXE91TPOL+jsMjufgbA== X-CSE-MsgGUID: vg6afZ/OSgy+EV43hMsb2A== X-IronPort-AV: E=McAfee;i="6800,10657,11859"; a="97473679" X-IronPort-AV: E=Sophos;i="6.25,192,1779174000"; d="scan'208";a="97473679" Received: from fmviesa004.fm.intel.com ([10.60.135.144]) by orvoesa104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 29 Jul 2026 10:28:03 -0700 X-CSE-ConnectionGUID: 8uUz5WhWTYaYOu+dWOl00A== X-CSE-MsgGUID: jP+rPX0FTDOGVSkBS6O51A== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.25,192,1779174000"; d="scan'208";a="262024615" Received: from iherna2-mobl4.amr.corp.intel.com (HELO agluck-desk3.home.arpa) ([10.124.223.238]) by fmviesa004-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 29 Jul 2026 10:28:03 -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 v10 14/17] x86/resctrl: Export interface to report telemetry unbind/remove Date: Wed, 29 Jul 2026 10:27:49 -0700 Message-ID: <20260729172752.11561-15-tony.luck@intel.com> X-Mailer: git-send-email 2.55.0 In-Reply-To: <20260729172752.11561-1-tony.luck@intel.com> References: <20260729172752.11561-1-tony.luck@intel.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Linux allows devices to be unbound from drivers, even if the modules cannot be unloaded because of reference counts. For telemetry this results in teardown of the virtual address mappings of the MMIO regions which hold the event counters. Subsequent access to these counters will page fault. Export intel_aet_invalidate() for the pmt_telemetry module to inform resctrl when invalidation is about to happen. Signed-off-by: Tony Luck --- v10: New patch include/linux/intel_vsec.h | 2 ++ arch/x86/kernel/cpu/resctrl/intel_aet.c | 34 +++++++++++++++++++++++++ 2 files changed, 36 insertions(+) diff --git a/include/linux/intel_vsec.h b/include/linux/intel_vsec.h index c04b1d0f3143..4da15ec03d20 100644 --- a/include/linux/intel_vsec.h +++ b/include/linux/intel_vsec.h @@ -262,11 +262,13 @@ void intel_aet_register_enumeration(struct module *module, struct pmt_feature_group *(*get)(enum pmt_feature_id id), void (*put)(struct pmt_feature_group *p)); void intel_aet_unregister_enumeration(void); +void intel_aet_invalidate(u64 pkgmask); #else static inline void intel_aet_register_enumeration(struct module *module, struct pmt_feature_group *(*get)(enum pmt_feature_id id), void (*put)(struct pmt_feature_group *p)) { } static inline void intel_aet_unregister_enumeration(void) { } +static inline void intel_aet_invalidate(u64 pkgmask) { } #endif /* CONFIG_X86_CPU_RESCTRL_INTEL_AET */ #endif diff --git a/arch/x86/kernel/cpu/resctrl/intel_aet.c b/arch/x86/kernel/cpu/resctrl/intel_aet.c index 65b70cc8d0a5..eab27e487da9 100644 --- a/arch/x86/kernel/cpu/resctrl/intel_aet.c +++ b/arch/x86/kernel/cpu/resctrl/intel_aet.c @@ -177,6 +177,11 @@ static bool skip_telem_region(struct telemetry_region *tr, struct event_group *e tr->guid); return true; } + if (tr->plat_info.package_id >= BITS_PER_LONG_LONG) { + pr_warn("Package %u out of range in guid 0x%x\n", tr->plat_info.package_id, + tr->guid); + return true; + } if (tr->size != e->mmio_size) { pr_warn("MMIO space wrong size (%zu bytes) for guid 0x%x. Expected %zu bytes.\n", tr->size, e->guid, e->mmio_size); @@ -299,6 +304,13 @@ static struct module *pmt_module; static struct pmt_feature_group *(*get_feature)(enum pmt_feature_id id); static void (*put_feature)(struct pmt_feature_group *p); +/* + * Bitmask of packages where MMIO virtual mappings have been invalidated + * by device unbind/remove operations. May be set to ~0ULL to indicate that + * all mappings may have been removed. + */ +static u64 invalid_pkg_mask; + /* * Request a copy of struct pmt_feature_group for each event group. If there is * one, the returned structure has an array of telemetry_region structures, @@ -373,6 +385,18 @@ void intel_aet_unregister_enumeration(void) } EXPORT_SYMBOL_NS_GPL(intel_aet_unregister_enumeration, "INTEL_PMT"); +/* + * pmt_telemetry driver calls this for unbind/remove operations that + * will invalidate the virtual addresses of MMIO registers provided + * by intel_pmt_get_regions_by_feature(). + */ +void intel_aet_invalidate(u64 pkgmask) +{ + guard(mutex)(&aet_register_lock); + invalid_pkg_mask |= pkgmask; +} +EXPORT_SYMBOL_NS_GPL(intel_aet_invalidate, "INTEL_PMT"); + bool intel_aet_pre_mount(void) { guard(mutex)(&aet_register_lock); @@ -389,6 +413,11 @@ bool intel_aet_pre_mount(void) } pmt_in_use = true; + /* + * All the regions enumerated by intel_pmt_get_regions_by_feature() + * have valid MMIO mappings. + */ + invalid_pkg_mask = 0ULL; return true; } @@ -436,6 +465,11 @@ int intel_aet_read_event(int domid, u32 rmid, void *arch_priv, u64 *val) void *pevt0; u32 idx; + guard(mutex)(&aet_register_lock); + + if (invalid_pkg_mask == ~0ULL || (invalid_pkg_mask & BIT_ULL(domid))) + return -EINVAL; + pevt0 = pevt - pevt->idx; e = container_of(pevt0, struct event_group, evts); idx = rmid * e->num_events; -- 2.55.0