kata-containers / tests

Kata Containers tests, CI, and metrics
https://katacontainers.io/
Apache License 2.0
139 stars 196 forks source link

install_rom_aarch64.sh failing #5817

Open ad-zsolt-imre opened 7 months ago

ad-zsolt-imre commented 7 months ago

Description of problem

The install_rom_aarch64.sh is failing for multiple reasons:

make[1]: Entering directory '/tmp/tmp.xKw73XBZTB/acpica/generate/unix/acpiexec'
obj/acpiexec ../../../source/common/acgetline.c
...
obj/acpiexec ../../../source/components/dispatcher/dsinit.c
obj/acpiexec ../../../source/components/utilities/utdebug.c
../../../source/components/utilities/utdebug.c: In function ‘AcpiUtInitStackPtrTrace’:
../../../source/components/utilities/utdebug.c:188:31: error: storing the address of local variable ‘CurrentSp’ in ‘AcpiGbl_EntryStackPointer’ [-Werror=dangling-pointer=]
  188 |     AcpiGbl_EntryStackPointer = &CurrentSp;
      |     ~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~
../../../source/components/utilities/utdebug.c:185:29: note: ‘CurrentSp’ declared here
  185 |     ACPI_SIZE               CurrentSp;
      |                             ^~~~~~~~~
In file included from ../../../source/include/acpi.h:173,
                 from ../../../source/components/utilities/utdebug.c:154:
../../../source/include/acglobal.h:334:41: note: ‘AcpiGbl_EntryStackPointer’ declared here
  334 | ACPI_GLOBAL (ACPI_SIZE *,               AcpiGbl_EntryStackPointer);
      |                                         ^~~~~~~~~~~~~~~~~~~~~~~~~
../../../source/include/acpixf.h:188:17: note: in definition of macro ‘ACPI_GLOBAL’
  188 |     extern type name
      |                 ^~~~
../../../source/components/utilities/utdebug.c: In function ‘AcpiUtTrackStackPtr’:
../../../source/components/utilities/utdebug.c:213:36: error: storing the address of local variable ‘CurrentSp’ in ‘AcpiGbl_LowestStackPointer’ [-Werror=dangling-pointer=]
  213 |         AcpiGbl_LowestStackPointer = &CurrentSp;
      |         ~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~
../../../source/components/utilities/utdebug.c:208:29: note: ‘CurrentSp’ declared here
  208 |     ACPI_SIZE               CurrentSp;
      |                             ^~~~~~~~~
../../../source/include/acglobal.h:335:41: note: ‘AcpiGbl_LowestStackPointer’ declared here
  335 | ACPI_GLOBAL (ACPI_SIZE *,               AcpiGbl_LowestStackPointer);
      |                                         ^~~~~~~~~~~~~~~~~~~~~~~~~~
../../../source/include/acpixf.h:188:17: note: in definition of macro ‘ACPI_GLOBAL’
  188 |     extern type name
      |                 ^~~~
cc1: all warnings being treated as errors
make[1]: *** [../Makefile.rules:20: obj/utdebug.o] Error 1
make[1]: Leaving directory '/tmp/tmp.xKw73XBZTB/acpica/generate/unix/acpiexec'
make: *** [generate/unix/Makefile.common:7: acpiexec] Error 2
make: Leaving directory '/tmp/tmp.xKw73XBZTB/acpica'

Expected result

It produces the two UEFI ROM image files as promised.

Actual result

The process fails at multiple stages. (See description)

Environment

Linux node1 6.5.0-1014-raspi #17-Ubuntu SMP PREEMPT_DYNAMIC Thu Mar 21 11:24:03 UTC 2024 aarch64 aarch64 aarch64 

Ubuntu 23.10

gcc version 13.2.0 (Ubuntu 13.2.0-4ubuntu3) 
ad-zsolt-imre commented 7 months ago

Is the build still needed, btw? I changed the script to pull https://releases.linaro.org/components/kernel/uefi-linaro/16.02/release/qemu64/QEMU_EFI.fd and then it completed. Still, I ended up with the same error reported by k3s:

Error: failed to create containerd task: failed to create shim task: failed to query hotpluggable CPUs: QMP command failed: The feature 'query-hotpluggable-cpus' is not enabled: unknown

Then, after some digging I landed here: https://wiki.qemu.org/Features/CPUHotplug - the page says:

dedicated legacy interface: cpu-add QMP command (removed in QEMU v5.2)

Then, I checked the qemu version on my node and I found:

QEMU emulator version 7.2.0 (kata-static)

Is it possible that the problem is that the version of qemu shipped with the Kata no longer supports the cpu-add QMP command?