[RFC PATCH 0/4] x86/resctrl: Refactor resctrl enumeration

Tony Luck <[email protected]>
Newsgroups dev.linux.lists.patches,org.kernel.vger.linux-kernel
Message-ID <[email protected]>
This series is posted as RFC to get some feedback before I incorporate
it into the next version of my patches to fix Kconfig issues[1]. It was
prompted by Reinette's comment[2]:

   "It looks to me that moving monitoring enumeration done in resctrl_cpu_detect()
    into resctrl proper where the RMID range checks can be centralized on features
    gated by rdt_cpu_has() would be simpler and accurate?"

Patch 1 adds a missing test that RDT monitoring is enabled in any form.
This test has been missing from the very start of Linux implementation.
I added a Fixes: tag abd it could be backported to stable and long term
kernels, but there might not be much value in doing so.

The remaining patches just relocate the enumeration from an early call on
the BSP, to the appropriate point in the normal "late_init" initialization
of resctrl. No functional changes intended in these patches.

Signed-off-by: Tony Luck <[email protected]>

Link: https://lore.kernel.org/all/[email protected]/ [1]
Link: https://lore.kernel.org/all/[email protected]/ [2]

Tony Luck (4):
  x86/resctrl: Check if monitoring features are enabled
  x86/resctrl: Enumerate monitor features in rdt_get_l3_mon_config()
  x86/resctrl: Apply Intel MBM quirk from rdt_get_l3_mon_config()
  x86/resctrl: Delete resctrl_cpu_detect()

 arch/x86/include/asm/processor.h       |  4 --
 arch/x86/include/asm/resctrl.h         | 12 ++---
 arch/x86/kernel/cpu/resctrl/internal.h |  4 +-
 arch/x86/kernel/cpu/amd.c              |  3 --
 arch/x86/kernel/cpu/hygon.c            |  3 --
 arch/x86/kernel/cpu/intel.c            |  7 ---
 arch/x86/kernel/cpu/resctrl/core.c     | 47 ++----------------
 arch/x86/kernel/cpu/resctrl/monitor.c  | 67 +++++++++++++++++++-------
 8 files changed, 59 insertions(+), 88 deletions(-)


base-commit: bd5f485f3f026225b86573e559af0b7254ef4184
-- 
2.55.0
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.