On 28/9/23 09:49, Paolo Bonzini wrote:
tls-cipher-suites is an object that is used to inject TLS configuration
into the guest (via fw_cfg).  It is never used for host-side TLS
operation, and therefore it need not be available in the tools.

Signed-off-by: Paolo Bonzini <pbonz...@redhat.com>
---
  crypto/meson.build   | 3 ++-
  hw/nvram/meson.build | 6 +-----
  2 files changed, 3 insertions(+), 6 deletions(-)

diff --git a/crypto/meson.build b/crypto/meson.build
index 9ac1a898027..c46f9c22a7f 100644
--- a/crypto/meson.build
+++ b/crypto/meson.build
@@ -46,7 +46,8 @@ endif
  if have_afalg
    crypto_ss.add(if_true: files('afalg.c', 'cipher-afalg.c', 'hash-afalg.c'))
  endif
-crypto_ss.add(when: gnutls, if_true: files('tls-cipher-suites.c'))
+
+system_ss.add(when: gnutls, if_true: files('tls-cipher-suites.c'))
util_ss.add(files(
    'aes.c',
diff --git a/hw/nvram/meson.build b/hw/nvram/meson.build
index 988dff6f8e5..75e415b1a01 100644
--- a/hw/nvram/meson.build
+++ b/hw/nvram/meson.build
@@ -1,8 +1,4 @@
-if have_system or have_tools
-  # QOM interfaces must be available anytime QOM is used.
-  qom_ss.add(files('fw_cfg-interface.c'))
-endif
-
+system_ss.add(files('fw_cfg-interface.c'))

Thanks, Richard mentioned it and I had it tagged as TODO,
but forgot about it...
https://lore.kernel.org/qemu-devel/8c26026f-26b4-08c0-0e77-ca9970dca...@linaro.org/

FTR the original issue was:
https://lore.kernel.org/qemu-devel/20200929154651.gc4...@linux.fritz.box/

$ qemu-storage-daemon --object help
List of user creatable objects:
qemu-storage-daemon: missing interface 'fw_cfg-data-generator' for object 'tls-creds'
Aborted (core dumped)

Reviewed-by: Philippe Mathieu-Daudé <phi...@linaro.org>


Reply via email to