Re: [PATCH v12 23/25] x86/resctrl: Require 64-bit x86 for resctrl support

[Reinette Chatre <reinette.chatre@intel.com>]

Hi Tony,

On 9/16/26 4:13 PM, Tony Luck wrote:
> Intel Application Energy Telemetry (AET) provides event counts in 64-bit
> registers in MMIO space. For accurate results these should be read with a
> single 64-bit load operation from the CPU.
> 
> In preparation for the removal of the separate CONFIG_X86_CPU_RESCTRL_INTEL_AET
> configuration option, move the X86_64 dependency to X86_CPU_RESCTRL.
> 
> This in line with Dave Hansen's proposal[1] for an addition to Linux

"This in line" -> "This is in line"?

> documentation that details the state of 32-bit x86 support.

Instead of requiring reader to follow link to learn what the state of support is
this could just briefly summarize it and reader can then choose to follow link
for the details. Does not need to be detailed itself, that is what the link is for,
just a small change would already help a lot, for example:

"details the state of 32-bit x86 support." -> "details reduced 32-bit x86 support."

> 
> Link: https://lore.kernel.org/all/c6ab0c24-8931-4f0d-9be6-23498ab4c38b@intel.com/ # [1]
> Signed-off-by: Tony Luck <tony.luck@intel.com>

See "Ordering of commit tags" in Documentation/process/maintainer-tip.rst

> ---

Reinette