[PATCH 0/2] x86/resctrl: Fix Platform QoS issues for Hygon

Xiaochen Shen posted 2 patches 2 weeks, 1 day ago
There is a newer version of this series
arch/x86/kernel/cpu/resctrl/core.c     | 21 +++++++++++++++++----
arch/x86/kernel/cpu/resctrl/internal.h |  3 +++
2 files changed, 20 insertions(+), 4 deletions(-)
[PATCH 0/2] x86/resctrl: Fix Platform QoS issues for Hygon
Posted by Xiaochen Shen 2 weeks, 1 day ago
This patch series fixes two issues regarding Platform QoS (resctrl)
support on Hygon CPUs.

The first patch adds the missing initialization sequence required for
proper resctrl functionality. The second patch corrects the bit-width
configuration for the Memory Bandwidth Monitoring (MBM) counter.

Xiaochen Shen (2):
  x86/resctrl: Add missing resctrl initialization for Hygon
  x86/resctrl: Fix memory bandwidth counter width for Hygon

 arch/x86/kernel/cpu/resctrl/core.c     | 21 +++++++++++++++++----
 arch/x86/kernel/cpu/resctrl/internal.h |  3 +++
 2 files changed, 20 insertions(+), 4 deletions(-)

-- 
2.47.3
Re: [PATCH 0/2] x86/resctrl: Fix Platform QoS issues for Hygon
Posted by Reinette Chatre 2 weeks ago
Hi Xiaochen!

On 12/3/25 6:37 PM, Xiaochen Shen wrote:
> This patch series fixes two issues regarding Platform QoS (resctrl)
> support on Hygon CPUs.
> 
> The first patch adds the missing initialization sequence required for
> proper resctrl functionality. The second patch corrects the bit-width
> configuration for the Memory Bandwidth Monitoring (MBM) counter.
> 
> Xiaochen Shen (2):
>   x86/resctrl: Add missing resctrl initialization for Hygon
>   x86/resctrl: Fix memory bandwidth counter width for Hygon
> 
>  arch/x86/kernel/cpu/resctrl/core.c     | 21 +++++++++++++++++----
>  arch/x86/kernel/cpu/resctrl/internal.h |  3 +++
>  2 files changed, 20 insertions(+), 4 deletions(-)
> 

This series looks good to me. Thank you.

One nit is that tip prefers the Cc: tag to be after the author's SoB. For details you can
refer to section "Ordering of commit tags" in Documentation/process/maintainer-tip.rst

Reviewed-by: Reinette Chatre <reinette.chatre@intel.com>

Reinette
Re: [PATCH 0/2] x86/resctrl: Fix Platform QoS issues for Hygon
Posted by Xiaochen Shen 2 weeks ago
Hi Reinette,

On 12/5/2025 7:30 AM, Reinette Chatre wrote:
> This series looks good to me. Thank you.
> 
> One nit is that tip prefers the Cc: tag to be after the author's SoB. For details you can
> refer to section "Ordering of commit tags" in Documentation/process/maintainer-tip.rst
> 
> Reviewed-by: Reinette Chatre <reinette.chatre@intel.com>
> 
> Reinette

Thank you very much for code review!

I will fix this nit in the v2 patch series to comply with tip tree rules.


Best regards,
Xiaochen Shen