[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 04/33] smbios: filter based on CONFIG_SMBIOS rather t
From: |
Paolo Bonzini |
Subject: |
[Qemu-devel] [PULL 04/33] smbios: filter based on CONFIG_SMBIOS rather than TARGET |
Date: |
Wed, 11 Jan 2017 20:34:58 +0100 |
From: Leif Lindholm <address@hidden>
-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>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>
---
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 +-
include/sysemu/arch_init.h | 1 -
vl.c | 2 +-
8 files changed, 38 insertions(+), 13 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/include/sysemu/arch_init.h b/include/sysemu/arch_init.h
index 1c9dad1..88dcf77 100644
--- a/include/sysemu/arch_init.h
+++ b/include/sysemu/arch_init.h
@@ -29,7 +29,6 @@ extern const uint32_t arch_type;
void select_soundhw(const char *optarg);
void do_acpitable_option(const QemuOpts *opts);
-void do_smbios_option(QemuOpts *opts);
void audio_init(void);
int kvm_available(void);
int xen_available(void);
diff --git a/vl.c b/vl.c
index a23de18..1b81340 100644
--- a/vl.c
+++ b/vl.c
@@ -3712,7 +3712,7 @@ int main(int argc, char **argv, char **envp)
if (!opts) {
exit(1);
}
- do_smbios_option(opts);
+ smbios_entry_add(opts, &error_fatal);
break;
case QEMU_OPTION_fwcfg:
opts = qemu_opts_parse_noisily(qemu_find_opts("fw_cfg"),
--
1.8.3.1
- [Qemu-devel] [PULL 00/33] Misc patches for 2017-01-11, Paolo Bonzini, 2017/01/11
- [Qemu-devel] [PULL 01/33] bugfix: vm halt when in reset looping, Paolo Bonzini, 2017/01/11
- [Qemu-devel] [PULL 02/33] megasas: fix guest-triggered memory leak, Paolo Bonzini, 2017/01/11
- [Qemu-devel] [PULL 03/33] qom: Make all interface types abstract, Paolo Bonzini, 2017/01/11
- [Qemu-devel] [PULL 04/33] smbios: filter based on CONFIG_SMBIOS rather than TARGET,
Paolo Bonzini <=
- [Qemu-devel] [PULL 05/33] stubs: merge all monitor stubs in one file, remove monitor_cur_is_qmp stub, Paolo Bonzini, 2017/01/11
- [Qemu-devel] [PULL 06/33] stubs: move smbios stubs to hw/smbios, Paolo Bonzini, 2017/01/11
- [Qemu-devel] [PULL 07/33] stubs: move acpi stubs to hw/acpi, Paolo Bonzini, 2017/01/11
- [Qemu-devel] [PULL 08/33] stubs: remove unused stub for serial_hd, Paolo Bonzini, 2017/01/11
- [Qemu-devel] [PULL 09/33] hw: move reset handlers from vl.c to hw/core, Paolo Bonzini, 2017/01/11
- [Qemu-devel] [PULL 10/33] stubs: group stubs for user-mode emulation, Paolo Bonzini, 2017/01/11
- [Qemu-devel] [PULL 14/33] build: remove --enable-colo/--disable-colo, Paolo Bonzini, 2017/01/11
- [Qemu-devel] [PULL 12/33] stubs: move vhost stubs to stubs/vhost.o, Paolo Bonzini, 2017/01/11
- [Qemu-devel] [PULL 13/33] event_notifier: cleanups around event_notifier_set_handler, Paolo Bonzini, 2017/01/11
- [Qemu-devel] [PULL 11/33] stubs: group all monitor_fdset_* functions in a single file, Paolo Bonzini, 2017/01/11