Re: [PATCH] x86,fs/resctrl: Add event_group information for Intel Diamond Rapids

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

Hi Tony,

On 9/21/26 2:32 PM, Tony Luck wrote:
> Intel published XML descriptions for the Application Energy Telemetry
> events on Diamond Rapids here: https://github.com/intel/Intel-PMT.

See "Link:" in Documentation/process/maintainer-tip.rst

> 
> Add event_group structures for both new GUIDs.

"Add event_group structures" describes actual code and is clear from the patch. 

What does "both" refer to?

> 
> There are two new "perf" events. Add them to the resctrl file system.

Here it becomes confusing since patch adds two event groups
as well as two events.

Below is an attempt at a changelog that follows tip requirements. Could
you please consider this as a draft and build on it to use accurate
terminology and be accurate for this patch? 

  x86,fs/resctrl: Add event group descriptions for Intel Diamond Rapids

  Intel Diamond Rapids exposes Application Energy Telemetry through two
  new PMT aggregator GUIDs: one covering the "energy" feature, and one
  covering the "perf" feature, both documented in the Intel-PMT XML
  repository [1].

  The "perf" aggregator references two events not previously enumerated
  by resctrl: retired instructions (INST_RETIRED), and a count of
  unhalted core clock cycles perceived by hardware as contributing to
  instruction execution (PCNT).

  Add definitions for both aggregators, and add the two new events to
  the resctrl filesystem so they can be exposed to user space.

  Signed-off-by: Tony Luck <tony.luck@intel.com>
  Link: https://github.com/intel/Intel-PMT # [1]

> 
> Signed-off-by: Tony Luck <tony.luck@intel.com>
> ---
patch looks good.

Reinette