Re: [tip: x86/cache] x86/resctrl: Fix incorrect local bandwidth when mba_sc is enabled

From: Xiaochen Shen
Date: Thu Dec 10 2020 - 11:42:26 EST


Hi Boris,

On 12/10/2020 18:26, Borislav Petkov wrote:
On Thu, Dec 10, 2020 at 12:45:11PM +0800, Xiaochen Shen wrote:
Thank you for clarifying this issue. It is not a 0-DAY CI issue.
Which begs the question: this patch should be Cc: stable and should go
in now, shouldn't it?

Because then the first submission applies cleanly ontop of
tip:x86/urgent.

I mean, it is fixing only reporting but that reporting is kinda waaay
off.

Hmmm?


Yes. Thank you for pointing out this, we'd better add Cc: stable tag.
But we have to backport the patch for -stable trees because the code base
is changed by following upstream commits:

(1) For all stable trees: in commit abe8f12b4425 ("x86/resctrl: Remove
unused struct mbm_state::chunks_bw"), removing unused struct
mbm_state::chunks_bw changes the code base for this patch.

(2) For 4.14 and 4.19 stable trees: in commit fa7d949337cc ("x86/resctrl:
Rename and move rdt files to a separate directory"), the file
arch/x86/kernel/cpu/intel_rdt_monitor.c has been renamed and moved to
arch/x86/kernel/cpu/resctrl/monitor.c. This patch needs to be rebased.

I plan to do backporting for all -stable trees after this patch is merged
into upstream.

Thank you.

--
Best regards,
Xiaochen