From 2a89fdba596f7afbad16509a8abf43c8bf7fec06 Mon Sep 17 00:00:00 2001 From: Xie XiuQi Date: Sun, 28 Apr 2019 22:18:14 +0800 Subject: [PATCH 2859/2944] openEuler: arm64/mpam: update group flags only when enable sucsses commit f654d1f36c1f2cf29afd1073731eb275483f1a8b openEuler-1.0 hulk inclusion category: bugfix bugzilla: 14212 CVE: NA The ctrlcon flag should be set only when we sucessful to enable ctrlmon, or the flag is mismatch with ctrlmon status. ctrlmon is indicate a group which could moniter the usage of L3/Memory bandwidth. Signed-off-by: Xie XiuQi Reviewed-by: Hanjun Guo Signed-off-by: Yang Yingliang Signed-off-by: Xin Hao Reviewed-by: Baolin Wang --- arch/arm64/kernel/mpam.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/arch/arm64/kernel/mpam.c b/arch/arm64/kernel/mpam.c index a9dab7e..dae2303 100644 --- a/arch/arm64/kernel/mpam.c +++ b/arch/arm64/kernel/mpam.c @@ -1013,9 +1013,10 @@ static ssize_t resctrl_group_ctrlmon_write(struct kernfs_open_file *of, rdtgrp->flags &= ~RDT_CTRLMON; resctrl_ctrlmon_disable(rdtgrp->mon.mon_data_kn, rdtgrp); } else if (!(rdtgrp->flags & RDT_CTRLMON) && ctrlmon) { - rdtgrp->flags |= RDT_CTRLMON; - resctrl_ctrlmon_enable(rdtgrp->kn, rdtgrp, - &rdtgrp->mon.mon_data_kn); + ret = resctrl_ctrlmon_enable(rdtgrp->kn, rdtgrp, + &rdtgrp->mon.mon_data_kn); + if (!ret) + rdtgrp->flags |= RDT_CTRLMON; } else { ret = -ENOENT; } -- 1.8.3.1