diff options
author | Xiaochen Shen <xiaochen.shen@intel.com> | 2019-03-29 17:50:38 -0400 |
---|---|---|
committer | Borislav Petkov <bp@suse.de> | 2019-04-01 12:45:27 -0400 |
commit | faa3604eda325588451c9c1eb4c8a8d04c1cd633 (patch) | |
tree | c6c9efdcda75a9efa5cdd607ca38f1f01be6b925 | |
parent | f560bd19d2fe0e54851d706b72acbc6f2eed3567 (diff) |
x86/resctrl: Fix typos in the mba_sc mount option
The user can control the MBA memory bandwidth in MBps (Mega
Bytes per second) units of the MBA Software Controller (mba_sc)
by using the "mba_MBps" mount option. For details, see
Documentation/x86/resctrl_ui.txt.
However, commit
23bf1b6be9c2 ("kernfs, sysfs, cgroup, intel_rdt: Support fs_context")
changed the mount option name from "mba_MBps" to "mba_mpbs" by mistake.
Change it back from to "mba_MBps" because it is user-visible, and
correct "Opt_mba_mpbs" spelling to "Opt_mba_mbps".
[ bp: massage commit message. ]
Fixes: 23bf1b6be9c2 ("kernfs, sysfs, cgroup, intel_rdt: Support fs_context")
Signed-off-by: Xiaochen Shen <xiaochen.shen@intel.com>
Signed-off-by: Borislav Petkov <bp@suse.de>
Cc: dhowells@redhat.com
Cc: Fenghua Yu <fenghua.yu@intel.com>
Cc: "H. Peter Anvin" <hpa@zytor.com>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: pei.p.jia@intel.com
Cc: Reinette Chatre <reinette.chatre@intel.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Tony Luck <tony.luck@intel.com>
Cc: x86-ml <x86@kernel.org>
Link: https://lkml.kernel.org/r/1553896238-22130-1-git-send-email-xiaochen.shen@intel.com
-rw-r--r-- | arch/x86/kernel/cpu/resctrl/rdtgroup.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/arch/x86/kernel/cpu/resctrl/rdtgroup.c b/arch/x86/kernel/cpu/resctrl/rdtgroup.c index 399601eda8e4..54b9eef3eea9 100644 --- a/arch/x86/kernel/cpu/resctrl/rdtgroup.c +++ b/arch/x86/kernel/cpu/resctrl/rdtgroup.c | |||
@@ -2039,14 +2039,14 @@ out: | |||
2039 | enum rdt_param { | 2039 | enum rdt_param { |
2040 | Opt_cdp, | 2040 | Opt_cdp, |
2041 | Opt_cdpl2, | 2041 | Opt_cdpl2, |
2042 | Opt_mba_mpbs, | 2042 | Opt_mba_mbps, |
2043 | nr__rdt_params | 2043 | nr__rdt_params |
2044 | }; | 2044 | }; |
2045 | 2045 | ||
2046 | static const struct fs_parameter_spec rdt_param_specs[] = { | 2046 | static const struct fs_parameter_spec rdt_param_specs[] = { |
2047 | fsparam_flag("cdp", Opt_cdp), | 2047 | fsparam_flag("cdp", Opt_cdp), |
2048 | fsparam_flag("cdpl2", Opt_cdpl2), | 2048 | fsparam_flag("cdpl2", Opt_cdpl2), |
2049 | fsparam_flag("mba_mpbs", Opt_mba_mpbs), | 2049 | fsparam_flag("mba_MBps", Opt_mba_mbps), |
2050 | {} | 2050 | {} |
2051 | }; | 2051 | }; |
2052 | 2052 | ||
@@ -2072,7 +2072,7 @@ static int rdt_parse_param(struct fs_context *fc, struct fs_parameter *param) | |||
2072 | case Opt_cdpl2: | 2072 | case Opt_cdpl2: |
2073 | ctx->enable_cdpl2 = true; | 2073 | ctx->enable_cdpl2 = true; |
2074 | return 0; | 2074 | return 0; |
2075 | case Opt_mba_mpbs: | 2075 | case Opt_mba_mbps: |
2076 | if (boot_cpu_data.x86_vendor != X86_VENDOR_INTEL) | 2076 | if (boot_cpu_data.x86_vendor != X86_VENDOR_INTEL) |
2077 | return -EINVAL; | 2077 | return -EINVAL; |
2078 | ctx->enable_mba_mbps = true; | 2078 | ctx->enable_mba_mbps = true; |