diff mbox series

[v2,23/25] x86: mtrr: Update 'mtrr' to allow setting MTRRs on any CPU

Message ID 20200614165958.159716-20-sjg@chromium.org
State Superseded
Headers show
Series x86: Enhance MTRR functionality to support multiple CPUs | expand

Commit Message

Simon Glass June 14, 2020, 4:59 p.m. UTC
Add a -c option to mtrr to allow any CPU to be updated with this command.

Signed-off-by: Simon Glass <sjg at chromium.org>
Reviewed-by: Wolfgang Wallner <wolfgang.wallner at br-automation.com>
---

(no changes since v1)

 cmd/x86/mtrr.c | 18 ++++++++++++++++--
 1 file changed, 16 insertions(+), 2 deletions(-)

Comments

Bin Meng June 28, 2020, 8:40 a.m. UTC | #1
On Mon, Jun 15, 2020 at 1:00 AM Simon Glass <sjg at chromium.org> wrote:
>
> Add a -c option to mtrr to allow any CPU to be updated with this command.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> Reviewed-by: Wolfgang Wallner <wolfgang.wallner at br-automation.com>
> ---
>
> (no changes since v1)
>
>  cmd/x86/mtrr.c | 18 ++++++++++++++++--
>  1 file changed, 16 insertions(+), 2 deletions(-)
>
> diff --git a/cmd/x86/mtrr.c b/cmd/x86/mtrr.c
> index b51b1cd7e2..d88a875d92 100644
> --- a/cmd/x86/mtrr.c
> +++ b/cmd/x86/mtrr.c
> @@ -104,6 +104,17 @@ static int do_mtrr(struct cmd_tbl *cmdtp, int flag, int argc,
>         int ret;
>
>         cpu_select = MP_SELECT_BSP;
> +       if (argc >= 3 && !strcmp("-c", argv[1])) {
> +               const char *cpustr;
> +
> +               cpustr = argv[2];
> +               if (*cpustr == 'a')
> +                       cpu_select = MP_SELECT_ALL;
> +               else
> +                       cpu_select = simple_strtol(cpustr, NULL, 16);
> +               argc -= 2;
> +               argv += 2;
> +       }
>         argc--;
>         argv++;
>         cmd = argv[0] ? *argv[0] : 0;
> @@ -145,11 +156,14 @@ static int do_mtrr(struct cmd_tbl *cmdtp, int flag, int argc,
>  }
>
>  U_BOOT_CMD(
> -       mtrr,   6,      1,      do_mtrr,
> +       mtrr,   8,      1,      do_mtrr,
>         "Use x86 memory type range registers (32-bit only)",
>         "[list]        - list current registers\n"
>         "set <reg> <type> <start> <size>   - set a register\n"
>         "\t<type> is Uncacheable, Combine, Through, Protect, Back\n"
>         "disable <reg>      - disable a register\n"
> -       "enable <reg>       - enable a register"
> +       "enable <reg>       - enable a register\n"
> +       "\n"
> +       "Precede command with '-c <n>|all' to access a particular CPU, e.g.\n"
> +       "   mtrr -c all list; mtrr -c 2e list"

nits: better to mention cpu number is interpreted as a hex number

>  );
> --

Reviewed-by: Bin Meng <bmeng.cn at gmail.com>
diff mbox series

Patch

diff --git a/cmd/x86/mtrr.c b/cmd/x86/mtrr.c
index b51b1cd7e2..d88a875d92 100644
--- a/cmd/x86/mtrr.c
+++ b/cmd/x86/mtrr.c
@@ -104,6 +104,17 @@  static int do_mtrr(struct cmd_tbl *cmdtp, int flag, int argc,
 	int ret;
 
 	cpu_select = MP_SELECT_BSP;
+	if (argc >= 3 && !strcmp("-c", argv[1])) {
+		const char *cpustr;
+
+		cpustr = argv[2];
+		if (*cpustr == 'a')
+			cpu_select = MP_SELECT_ALL;
+		else
+			cpu_select = simple_strtol(cpustr, NULL, 16);
+		argc -= 2;
+		argv += 2;
+	}
 	argc--;
 	argv++;
 	cmd = argv[0] ? *argv[0] : 0;
@@ -145,11 +156,14 @@  static int do_mtrr(struct cmd_tbl *cmdtp, int flag, int argc,
 }
 
 U_BOOT_CMD(
-	mtrr,	6,	1,	do_mtrr,
+	mtrr,	8,	1,	do_mtrr,
 	"Use x86 memory type range registers (32-bit only)",
 	"[list]        - list current registers\n"
 	"set <reg> <type> <start> <size>   - set a register\n"
 	"\t<type> is Uncacheable, Combine, Through, Protect, Back\n"
 	"disable <reg>      - disable a register\n"
-	"enable <reg>       - enable a register"
+	"enable <reg>       - enable a register\n"
+	"\n"
+	"Precede command with '-c <n>|all' to access a particular CPU, e.g.\n"
+	"   mtrr -c all list; mtrr -c 2e list"
 );