| Message ID | 20231124120907.2044651-1-michal.orzel@amd.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | automation: Switch u-boot boot command to bootz for arm32 tests | expand |
On 24/11/2023 12:09 pm, Michal Orzel wrote: > Thanks to recent changes added to ImageBuilder to support the bootz > command, which allows obtaining the effective image size (including NOLOAD > sections) from the zImage header, switch the BOOT_CMD for arm32 tests to > bootz. Among other scenarios, this change will enable us, in the future, > to add tests with UBSAN enabled Xen, which would otherwise fail due to > incorrect image placement resulting in overlapping. > > Signed-off-by: Michal Orzel <michal.orzel@amd.com> > --- > Before adding UBSAN CI tests, we still need to decide if we want to add support > to panic on UBSAN epilogue guarded by some config option or to just grep for > UBSAN message. CONFIG_UBSAN_FATAL please (name subject to taste.) Grepping dmesg is never a reliable option (see how many recent fixes we've had to console handling in the smoke tests), and reliability is important. ~Andrew
On Fri, 24 Nov 2023, Michal Orzel wrote: > Thanks to recent changes added to ImageBuilder to support the bootz > command, which allows obtaining the effective image size (including NOLOAD > sections) from the zImage header, switch the BOOT_CMD for arm32 tests to > bootz. Among other scenarios, this change will enable us, in the future, > to add tests with UBSAN enabled Xen, which would otherwise fail due to > incorrect image placement resulting in overlapping. > > Signed-off-by: Michal Orzel <michal.orzel@amd.com> Acked-by: Stefano Stabellini <sstabellini@kernel.org> > --- > Before adding UBSAN CI tests, we still need to decide if we want to add support > to panic on UBSAN epilogue guarded by some config option or to just grep for > UBSAN message. > > CI pipeline: > https://gitlab.com/xen-project/people/morzel/xen-orzelmichal/-/pipelines/1083821754 > --- > automation/scripts/qemu-smoke-dom0-arm32.sh | 2 +- > automation/scripts/qemu-smoke-dom0less-arm32.sh | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/automation/scripts/qemu-smoke-dom0-arm32.sh b/automation/scripts/qemu-smoke-dom0-arm32.sh > index a4b487b08055..d91648905669 100755 > --- a/automation/scripts/qemu-smoke-dom0-arm32.sh > +++ b/automation/scripts/qemu-smoke-dom0-arm32.sh > @@ -68,7 +68,7 @@ XEN_CMD="console=dtuart dom0_mem=1024M bootscrub=0 console_timestamps=boot" > NUM_DOMUS=0 > > LOAD_CMD="tftpb" > -BOOT_CMD="bootm" > +BOOT_CMD="bootz" > UBOOT_SOURCE="boot.source" > UBOOT_SCRIPT="boot.scr"' > config > > diff --git a/automation/scripts/qemu-smoke-dom0less-arm32.sh b/automation/scripts/qemu-smoke-dom0less-arm32.sh > index 7e3cfbe9c4d0..e31b6b9014e1 100755 > --- a/automation/scripts/qemu-smoke-dom0less-arm32.sh > +++ b/automation/scripts/qemu-smoke-dom0less-arm32.sh > @@ -101,7 +101,7 @@ DOMU_MEM[0]="512" > NUM_DOMUS=1 > > LOAD_CMD="tftpb" > -BOOT_CMD="bootm" > +BOOT_CMD="bootz" > UBOOT_SOURCE="boot.source" > UBOOT_SCRIPT="boot.scr"' > config > > -- > 2.25.1 >
diff --git a/automation/scripts/qemu-smoke-dom0-arm32.sh b/automation/scripts/qemu-smoke-dom0-arm32.sh index a4b487b08055..d91648905669 100755 --- a/automation/scripts/qemu-smoke-dom0-arm32.sh +++ b/automation/scripts/qemu-smoke-dom0-arm32.sh @@ -68,7 +68,7 @@ XEN_CMD="console=dtuart dom0_mem=1024M bootscrub=0 console_timestamps=boot" NUM_DOMUS=0 LOAD_CMD="tftpb" -BOOT_CMD="bootm" +BOOT_CMD="bootz" UBOOT_SOURCE="boot.source" UBOOT_SCRIPT="boot.scr"' > config diff --git a/automation/scripts/qemu-smoke-dom0less-arm32.sh b/automation/scripts/qemu-smoke-dom0less-arm32.sh index 7e3cfbe9c4d0..e31b6b9014e1 100755 --- a/automation/scripts/qemu-smoke-dom0less-arm32.sh +++ b/automation/scripts/qemu-smoke-dom0less-arm32.sh @@ -101,7 +101,7 @@ DOMU_MEM[0]="512" NUM_DOMUS=1 LOAD_CMD="tftpb" -BOOT_CMD="bootm" +BOOT_CMD="bootz" UBOOT_SOURCE="boot.source" UBOOT_SCRIPT="boot.scr"' > config
Thanks to recent changes added to ImageBuilder to support the bootz command, which allows obtaining the effective image size (including NOLOAD sections) from the zImage header, switch the BOOT_CMD for arm32 tests to bootz. Among other scenarios, this change will enable us, in the future, to add tests with UBSAN enabled Xen, which would otherwise fail due to incorrect image placement resulting in overlapping. Signed-off-by: Michal Orzel <michal.orzel@amd.com> --- Before adding UBSAN CI tests, we still need to decide if we want to add support to panic on UBSAN epilogue guarded by some config option or to just grep for UBSAN message. CI pipeline: https://gitlab.com/xen-project/people/morzel/xen-orzelmichal/-/pipelines/1083821754 --- automation/scripts/qemu-smoke-dom0-arm32.sh | 2 +- automation/scripts/qemu-smoke-dom0less-arm32.sh | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)