|
 |
9ae3a8 |
From d2bfd3e4bb4a6115dc40b56e885a68887df30f94 Mon Sep 17 00:00:00 2001
|
|
|
9ae3a8 |
From: Markus Armbruster <armbru@redhat.com>
|
|
|
9ae3a8 |
Date: Sat, 2 Nov 2013 10:01:20 +0100
|
|
|
9ae3a8 |
Subject: [PATCH 20/29] smbios: Normalize smbios_entry_add()'s error handling to exit(1)
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
RH-Author: Markus Armbruster <armbru@redhat.com>
|
|
|
9ae3a8 |
Message-id: <1383386488-29789-4-git-send-email-armbru@redhat.com>
|
|
|
9ae3a8 |
Patchwork-id: 55241
|
|
|
9ae3a8 |
O-Subject: [PATCH 7.0 qemu-kvm 03/11] smbios: Normalize smbios_entry_add()'s error handling to exit(1)
|
|
|
9ae3a8 |
Bugzilla: 994490
|
|
|
9ae3a8 |
RH-Acked-by: Laszlo Ersek <lersek@redhat.com>
|
|
|
9ae3a8 |
RH-Acked-by: Michael S. Tsirkin <mst@redhat.com>
|
|
|
9ae3a8 |
RH-Acked-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
From: Markus Armbruster <armbru@redhat.com>
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
It exits on all error conditions but one, where it returns -1.
|
|
|
9ae3a8 |
Normalize, and return void.
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
Signed-off-by: Markus Armbruster <armbru@redhat.com>
|
|
|
9ae3a8 |
Reviewed-by: Eric Blake <eblake@redhat.com>
|
|
|
9ae3a8 |
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
|
|
|
9ae3a8 |
(cherry picked from commit 351a6a73ca7a9123f0dfd6c6f85fd01e82fe3741)
|
|
|
9ae3a8 |
---
|
|
|
9ae3a8 |
arch_init.c | 4 +---
|
|
|
9ae3a8 |
hw/i386/smbios.c | 10 +++++-----
|
|
|
9ae3a8 |
include/hw/i386/smbios.h | 2 +-
|
|
|
9ae3a8 |
3 files changed, 7 insertions(+), 9 deletions(-)
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
9ae3a8 |
---
|
|
|
9ae3a8 |
arch_init.c | 4 +---
|
|
|
9ae3a8 |
hw/i386/smbios.c | 10 +++++-----
|
|
|
9ae3a8 |
include/hw/i386/smbios.h | 2 +-
|
|
|
9ae3a8 |
3 files changed, 7 insertions(+), 9 deletions(-)
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
diff --git a/arch_init.c b/arch_init.c
|
|
|
9ae3a8 |
index cd2bb5a..7834d36 100644
|
|
|
9ae3a8 |
--- a/arch_init.c
|
|
|
9ae3a8 |
+++ b/arch_init.c
|
|
|
9ae3a8 |
@@ -1080,9 +1080,7 @@ void do_acpitable_option(const QemuOpts *opts)
|
|
|
9ae3a8 |
void do_smbios_option(const char *optarg)
|
|
|
9ae3a8 |
{
|
|
|
9ae3a8 |
#ifdef TARGET_I386
|
|
|
9ae3a8 |
- if (smbios_entry_add(optarg) < 0) {
|
|
|
9ae3a8 |
- exit(1);
|
|
|
9ae3a8 |
- }
|
|
|
9ae3a8 |
+ smbios_entry_add(optarg);
|
|
|
9ae3a8 |
#endif
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
diff --git a/hw/i386/smbios.c b/hw/i386/smbios.c
|
|
|
9ae3a8 |
index e708cb8..0608aee 100644
|
|
|
9ae3a8 |
--- a/hw/i386/smbios.c
|
|
|
9ae3a8 |
+++ b/hw/i386/smbios.c
|
|
|
9ae3a8 |
@@ -183,7 +183,7 @@ static void smbios_build_type_1_fields(const char *t)
|
|
|
9ae3a8 |
buf, strlen(buf) + 1);
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
-int smbios_entry_add(const char *t)
|
|
|
9ae3a8 |
+void smbios_entry_add(const char *t)
|
|
|
9ae3a8 |
{
|
|
|
9ae3a8 |
char buf[1024];
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
@@ -222,7 +222,7 @@ int smbios_entry_add(const char *t)
|
|
|
9ae3a8 |
smbios_entries_len += sizeof(*table) + size;
|
|
|
9ae3a8 |
(*(uint16_t *)smbios_entries) =
|
|
|
9ae3a8 |
cpu_to_le16(le16_to_cpu(*(uint16_t *)smbios_entries) + 1);
|
|
|
9ae3a8 |
- return 0;
|
|
|
9ae3a8 |
+ return;
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
if (get_param_value(buf, sizeof(buf), "type", t)) {
|
|
|
9ae3a8 |
@@ -230,10 +230,10 @@ int smbios_entry_add(const char *t)
|
|
|
9ae3a8 |
switch (type) {
|
|
|
9ae3a8 |
case 0:
|
|
|
9ae3a8 |
smbios_build_type_0_fields(t);
|
|
|
9ae3a8 |
- return 0;
|
|
|
9ae3a8 |
+ return;
|
|
|
9ae3a8 |
case 1:
|
|
|
9ae3a8 |
smbios_build_type_1_fields(t);
|
|
|
9ae3a8 |
- return 0;
|
|
|
9ae3a8 |
+ return;
|
|
|
9ae3a8 |
default:
|
|
|
9ae3a8 |
error_report("Don't know how to build fields for SMBIOS type %ld",
|
|
|
9ae3a8 |
type);
|
|
|
9ae3a8 |
@@ -242,5 +242,5 @@ int smbios_entry_add(const char *t)
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
error_report("Must specify type= or file=");
|
|
|
9ae3a8 |
- return -1;
|
|
|
9ae3a8 |
+ exit(1);
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
diff --git a/include/hw/i386/smbios.h b/include/hw/i386/smbios.h
|
|
|
9ae3a8 |
index 9babeaf..56c6108 100644
|
|
|
9ae3a8 |
--- a/include/hw/i386/smbios.h
|
|
|
9ae3a8 |
+++ b/include/hw/i386/smbios.h
|
|
|
9ae3a8 |
@@ -13,7 +13,7 @@
|
|
|
9ae3a8 |
*
|
|
|
9ae3a8 |
*/
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
-int smbios_entry_add(const char *t);
|
|
|
9ae3a8 |
+void smbios_entry_add(const char *t);
|
|
|
9ae3a8 |
void smbios_add_field(int type, int offset, const void *data, size_t len);
|
|
|
9ae3a8 |
uint8_t *smbios_get_table(size_t *length);
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
--
|
|
|
9ae3a8 |
1.7.1
|
|
|
9ae3a8 |
|