[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] tpm: add stubs for qdev-prop-tpm
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH] tpm: add stubs for qdev-prop-tpm |
Date: |
Mon, 18 Dec 2017 13:27:09 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.5.0 |
On 18/12/2017 13:16, Cornelia Huck wrote:
> Building with --disable-tpm yields
>
> ../hw/core/qdev-properties-system.o: In function `set_tpm':
> /home/cohuck/git/qemu/hw/core/qdev-properties-system.c:274: undefined
> reference to `qemu_find_tpm_be'
> /home/cohuck/git/qemu/hw/core/qdev-properties-system.c:278: undefined
> reference to `tpm_backend_init'
> ../hw/core/qdev-properties-system.o: In function `release_tpm':
> /home/cohuck/git/qemu/hw/core/qdev-properties-system.c:291: undefined
> reference to `tpm_backend_reset'
>
> Add some proper stubs for those functions.
>
> Fixes: 493b78303532 ("qdev: add DEFINE_PROP_TPMBE")
> Reported-by: Thomas Huth <address@hidden>
> Signed-off-by: Cornelia Huck <address@hidden>
Maybe all the new code can be moved from qdev-properties-system.c to
hw/tpm/qdev.c instead? (Adding stubs should be the last resort,
especially if the stubs are never used outside system emulation).
Thanks,
Paolo
> ---
> stubs/tpm.c | 15 +++++++++++++++
> 1 file changed, 15 insertions(+)
>
> diff --git a/stubs/tpm.c b/stubs/tpm.c
> index c18aac1c73..97457e0646 100644
> --- a/stubs/tpm.c
> +++ b/stubs/tpm.c
> @@ -7,6 +7,7 @@
> #include "qemu/osdep.h"
> #include "sysemu/tpm.h"
> #include "qmp-commands.h"
> +#include "sysemu/tpm_backend.h"
>
> int tpm_init(void)
> {
> @@ -31,3 +32,17 @@ TpmModelList *qmp_query_tpm_models(Error **errp)
> {
> return NULL;
> }
> +
> +TPMBackend *qemu_find_tpm_be(const char *id)
> +{
> + return NULL;
> +}
> +
> +int tpm_backend_init(TPMBackend *s, TPMIf *tpmif, Error **errp)
> +{
> + return -1;
> +}
> +
> +void tpm_backend_reset(TPMBackend *s)
> +{
> +}
>