On 26/3/24 13:44, Philippe Mathieu-Daudé wrote:
On 26/3/24 13:26, Igor Mammedov wrote:
QEMU build fails with
hw/i386/fw_cfg.c:74: undefined reference to `smbios_get_table_legacy'
when it's built with only 'microvm' enabled i.e. with config patch
+++ b/configs/devices/i386-softmmu/default.mak
@@ -26,7 +26,7 @@
# Boards:
#
-CONFIG_ISAPC=y
-CONFIG_I440FX=y
-CONFIG_Q35=y
+CONFIG_ISAPC=n
+CONFIG_I440FX=n
+CONFIG_Q35=n
it happens because I've fogotten/lost smbios_get_table_legacy() stub.
Fix it by adding missing stub as Philippe suggested.
Fixes: b42b0e4daaa5 "smbios: build legacy mode code only for 'pc'
machine"
Reported-by: Michael Tokarev <m...@tls.msk.ru>
Singned-off-by: Philippe Mathieu-Daudé <phi...@linaro.org>
Signed-off-by: Igor Mammedov <imamm...@redhat.com>
---
Compile tested only.
While it's fixing bug for off-tree usecase with non-upstream config,
it's trivial enough to go into 9.0 if time frame allows.
Benefit of it going into 9.0 is that folks who play with minimal builds
won't have to carry the patch in their tree.
hw/smbios/smbios_legacy_stub.c | 5 +++++
1 file changed, 5 insertions(+)
Thanks, patch queued.
Tested-by: Philippe Mathieu-Daudé <phi...@linaro.org>
BTW I tested using the following patch:
-- >8 --
commit 8be7b26b430d3ab192a2d22215ee512072bd88fb
Author: Philippe Mathieu-Daudé <phi...@linaro.org>
Date: Tue Mar 26 13:52:17 2024 +0100
hw/i386: Add a config to only build the microvm machine
Add a config file to build a binary only containing the
microvm machine.
As suggested in commit d1d5e9eefd ("configure: allow the
selection of alternate config in the build"), it can be
built using:
$ ../configure --without-default-features \
--target-list=x86_64-softmmu \
--with-devices-x86_64=microvm
Signed-off-by: Philippe Mathieu-Daudé <phi...@linaro.org>
diff --git a/configs/devices/x86_64-softmmu/microvm.mak
b/configs/devices/x86_64-softmmu/microvm.mak
new file mode 100644
index 0000000000..fe48b5b4a7
--- /dev/null
+++ b/configs/devices/x86_64-softmmu/microvm.mak
@@ -0,0 +1,20 @@
+# SPDX-FileCopyrightText: 2024 Linaro Ltd.
+#
+# Config that only supports the 64-bit microvm machine.
+# This avoids bringing in any of numerous legacy features from
+# the legacy machines or the 32bit platform.
+#
+
+CONFIG_MICROVM=y
+CONFIG_PCI_DEVICES=n
+CONFIG_SMBIOS=y
+CONFIG_SMBIOS_LEGACY=n
+CONFIG_VIRTIO_BALLOON=y
+CONFIG_VIRTIO_BLK=y
+CONFIG_VIRTIO_CRYPTO=y
+CONFIG_VIRTIO_GPU=y
+CONFIG_VIRTIO_INPUT=y
+CONFIG_VIRTIO_NET=y
+CONFIG_VIRTIO_RNG=y
+CONFIG_VIRTIO_SCSI=y
+CONFIG_VIRTIO_SERIAL=y
---
Before:
Undefined symbols for architecture arm64:
"_smbios_get_table_legacy", referenced from:
_fw_cfg_build_smbios in hw_i386_fw_cfg.c.o
ld: symbol(s) not found for architecture arm64
After:
$ ./qemu-system-x86_64 -S -monitor stdio -M microvm
QEMU 8.2.90 monitor - type 'help' for more information
(qemu) info qom-tree
/machine (microvm-machine)
...
