qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] smbios: filter based on CONFIG_SMBIOS rather th


From: Paolo Bonzini
Subject: Re: [Qemu-devel] [PATCH] smbios: filter based on CONFIG_SMBIOS rather than TARGET
Date: Thu, 22 Dec 2016 16:25:31 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.5.1


On 22/12/2016 16:18, Leif Lindholm wrote:
> -smbios command line options were accepted but silently ignored on
> TARGET_ARM, due to a test for TARGET_I386 in arch_init.c.
> 
> Copy the mechanism of hw/pci/pci-stub.c to implement an smbios-stub
> instead, enabled for all targets without CONFIG_SMBIOS.
> 
> Signed-off-by: Leif Lindholm <address@hidden>
> ---
> 
> Not sure if Suggested-by: Paolo Bonzini <address@hidden>
> is semantically correct in this instance, but if it is, please
> add it.
> 
>  arch_init.c                |  8 --------
>  hw/Makefile.objs           |  2 +-
>  hw/smbios/Makefile.objs    |  3 +++
>  hw/smbios/smbios-stub.c    | 31 +++++++++++++++++++++++++++++++
>  hw/smbios/smbios.c         |  2 +-
>  include/hw/smbios/smbios.h |  2 +-
>  vl.c                       |  2 +-
>  7 files changed, 38 insertions(+), 12 deletions(-)
>  create mode 100644 hw/smbios/smbios-stub.c
> 
> diff --git a/arch_init.c b/arch_init.c
> index 5cc58b2..34e7694 100644
> --- a/arch_init.c
> +++ b/arch_init.c
> @@ -28,7 +28,6 @@
>  #include "sysemu/arch_init.h"
>  #include "hw/pci/pci.h"
>  #include "hw/audio/audio.h"
> -#include "hw/smbios/smbios.h"
>  #include "qemu/config-file.h"
>  #include "qemu/error-report.h"
>  #include "qmp-commands.h"
> @@ -248,13 +247,6 @@ void do_acpitable_option(const QemuOpts *opts)
>  #endif
>  }
>  
> -void do_smbios_option(QemuOpts *opts)
> -{
> -#ifdef TARGET_I386
> -    smbios_entry_add(opts);
> -#endif
> -}
> -
>  int kvm_available(void)
>  {
>  #ifdef CONFIG_KVM
> diff --git a/hw/Makefile.objs b/hw/Makefile.objs
> index 0ffd281..2a73ae5 100644
> --- a/hw/Makefile.objs
> +++ b/hw/Makefile.objs
> @@ -33,7 +33,7 @@ devices-dirs-$(CONFIG_VIRTIO) += virtio/
>  devices-dirs-$(CONFIG_SOFTMMU) += watchdog/
>  devices-dirs-$(CONFIG_SOFTMMU) += xen/
>  devices-dirs-$(CONFIG_MEM_HOTPLUG) += mem/
> -devices-dirs-$(CONFIG_SMBIOS) += smbios/
> +devices-dirs-$(CONFIG_SOFTMMU) += smbios/
>  devices-dirs-y += core/
>  common-obj-y += $(devices-dirs-y)
>  obj-y += $(devices-dirs-y)
> diff --git a/hw/smbios/Makefile.objs b/hw/smbios/Makefile.objs
> index c3d3753..ee0712b 100644
> --- a/hw/smbios/Makefile.objs
> +++ b/hw/smbios/Makefile.objs
> @@ -1,2 +1,5 @@
>  common-obj-$(CONFIG_SMBIOS) += smbios.o
>  common-obj-$(call land,$(CONFIG_SMBIOS),$(CONFIG_IPMI)) += smbios_type_38.o
> +
> +common-obj-$(call lnot,$(CONFIG_SMBIOS)) += smbios-stub.o
> +common-obj-$(CONFIG_ALL) += smbios-stub.o
> diff --git a/hw/smbios/smbios-stub.c b/hw/smbios/smbios-stub.c
> new file mode 100644
> index 0000000..3087394
> --- /dev/null
> +++ b/hw/smbios/smbios-stub.c
> @@ -0,0 +1,31 @@
> +/*
> + * SMBIOS stubs for platforms that don't support SMBIOS.
> + *
> + * Copyright (c) 2010 Isaku Yamahata <yamahata at valinux co jp>
> + *                    VA Linux Systems Japan K.K.
> + * Copyright (c) 2016 Leif Lindholm <address@hidden>
> + *                    Linaro Ltd.
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License as published by
> + * the Free Software Foundation; either version 2 of the License, or
> + * (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License along
> + * with this program; if not, see <http://www.gnu.org/licenses/>.
> + */
> +
> +#include "qemu/osdep.h"
> +#include "qapi/qmp/qerror.h"
> +#include "qmp-commands.h"
> +#include "hw/smbios/smbios.h"
> +
> +void smbios_entry_add(QemuOpts *opts, Error **errp)
> +{
> +    error_setg(errp, QERR_UNSUPPORTED);
> +}
> diff --git a/hw/smbios/smbios.c b/hw/smbios/smbios.c
> index 3a96ced..1a5437a 100644
> --- a/hw/smbios/smbios.c
> +++ b/hw/smbios/smbios.c
> @@ -882,7 +882,7 @@ static void save_opt(const char **dest, QemuOpts *opts, 
> const char *name)
>      }
>  }
>  
> -void smbios_entry_add(QemuOpts *opts)
> +void smbios_entry_add(QemuOpts *opts, Error **errp)
>  {
>      const char *val;
>  
> diff --git a/include/hw/smbios/smbios.h b/include/hw/smbios/smbios.h
> index 1cd53cc..31e8d5f 100644
> --- a/include/hw/smbios/smbios.h
> +++ b/include/hw/smbios/smbios.h
> @@ -257,7 +257,7 @@ struct smbios_type_127 {
>      struct smbios_structure_header header;
>  } QEMU_PACKED;
>  
> -void smbios_entry_add(QemuOpts *opts);
> +void smbios_entry_add(QemuOpts *opts, Error **errp);
>  void smbios_set_cpuid(uint32_t version, uint32_t features);
>  void smbios_set_defaults(const char *manufacturer, const char *product,
>                           const char *version, bool legacy_mode,
> diff --git a/vl.c b/vl.c
> index d77dd86..d2a0d75 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -3711,7 +3711,7 @@ int main(int argc, char **argv, char **envp)
>                  if (!opts) {
>                      exit(1);
>                  }
> -                do_smbios_option(opts);
> +                smbios_entry_add(opts, &error_abort);

This should be error_fatal.  Will fix and apply.

Paolo

>                  break;
>              case QEMU_OPTION_fwcfg:
>                  opts = qemu_opts_parse_noisily(qemu_find_opts("fw_cfg"),
> 



reply via email to

[Prev in Thread] Current Thread [Next in Thread]