diff options
author | Xiao Yang <xiaox.yang@intel.com> | 2019-07-30 13:40:16 +0800 |
---|---|---|
committer | Philip Li <philip.li@intel.com> | 2019-07-30 15:01:16 +0800 |
commit | 6f0cb3662ddf97f6110050af93fb31762d727809 (patch) | |
tree | 731e444f0ec7b171d3107f144abed664b8c24288 | |
parent | c53b636ac2b2940400e69681fd73850d761fae6b (diff) | |
download | lkp-tests-6f0cb3662ddf97f6110050af93fb31762d727809.tar.gz |
setup/boot_params: Rename bp_mem to bp_memmap
1) Some jobs uses bp_memmap as boot parameter but
setup/boot_params doesn't check bp_memmap, so
setup/boot_params cannot report error when the
value of bp_memmap in job file mismatches the
actual value of bp_memmap in cmdline. For example:
----------------------------------------------
# grep bp_memmap ./chromeswap-hsw-d01-1-5-uniform-0.25.yaml
bp_memmap: 1G!7G
# grep -o 'memmap=[0-9G!]*' /proc/cmdline
memmap=1G!5G
Actual result:
Don't report error when running job
Expected result:
Report the following error when running job
Missing parameter: memmap=1G!7G, actual command line:
root=PARTUUID=69cab1c6-7b3b-7c43-bd47-2542bc62c38c
console=tty0 console=ttyS0,115200n8 console=hvc0 quiet
memmap=1G!5G ...
----------------------------------------------
2) No job uses bp_mem as boot parameter
Add check for bp_memmap in setup/boot_params by renaming
bp_mem to bp_memmap
Signed-off-by: Xiao Yang <xiaox.yang@intel.com>
Signed-off-by: Philip Li <philip.li@intel.com>
-rwxr-xr-x | setup/boot_params | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/setup/boot_params b/setup/boot_params index 1f557b64..55924b8b 100755 --- a/setup/boot_params +++ b/setup/boot_params @@ -1,5 +1,5 @@ #!/bin/sh -# - bp_mem +# - bp_memmap # - bp1_memmap # - bp2_memmap # - bp3_memmap @@ -26,7 +26,7 @@ check_param() fi } -check_param "bp_mem" +check_param "bp_memmap" check_param "bp1_memmap" check_param "bp2_memmap" check_param "bp3_memmap" |