[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [QEMU v8 PATCH 0/7] SMBIOS: build full tables in QEMU
From: |
Gabriel L. Somlo |
Subject: |
[Qemu-devel] [QEMU v8 PATCH 0/7] SMBIOS: build full tables in QEMU |
Date: |
Wed, 23 Apr 2014 09:42:35 -0400 |
On Wed, Apr 23, 2014 at 12:13:48PM +0200, Gerd Hoffmann wrote:
> Series looks good to me.
Awesome, thanks! Here's one more iteration, rearranging things
to start with the most benign and uncontroversial patches, and
progressing toward the "payload" at the end :)
> /me wonders though how you've tested the final revision and the compat
> stuff. Patch #6 doesn't actually add a new machine type where the
> non-legacy mode is active ...
Why, by commenting and uncommenting "smbios_legacy_mode = true;"
in the *compat_2_0() functions :) I realized I was being silly
after sending out the v7 series, and added something more or less
similar to what you wrote, and was saving it for v8. I squashed it
on top of the compat functions, see patch 2/7.
> I'll go wait for a few days for additional feedback, then [...]
> send out a pull request.
Sounds like a plan! BTW, would it make sense to have Kevin apply
this: http://www.seabios.org/pipermail/seabios/2014-April/007856.html
to SeaBIOS and then pull a new blob from there around the same
time (or before ?) this patch set is applied to QEMU ?
Thanks much,
Gabriel
Gabriel L. Somlo (7):
E820: Add interface for accessing e820 table
PC: Add machine version 2.1 for piix and q35
SMBIOS: Rename symbols to better reflect future use
SMBIOS: Update header file definitions
SMBIOS: Use macro to set smbios defaults
SMBIOS: Use bitmaps to prevent incompatible comand line options
SMBIOS: Build aggregate smbios tables and entry point
hw/i386/pc.c | 39 ++-
hw/i386/pc_piix.c | 43 ++-
hw/i386/pc_q35.c | 37 ++-
hw/i386/smbios.c | 789 +++++++++++++++++++++++++++++++++++++++++++----
include/hw/i386/pc.h | 2 +
include/hw/i386/smbios.h | 99 ++++--
6 files changed, 902 insertions(+), 107 deletions(-)
--
1.9.0
- [Qemu-devel] [QEMU v8 PATCH 0/7] SMBIOS: build full tables in QEMU,
Gabriel L. Somlo <=
- [Qemu-devel] [QEMU v8 PATCH 1/7] E820: Add interface for accessing e820 table, Gabriel L. Somlo, 2014/04/23
- [Qemu-devel] [QEMU v8 PATCH 4/7] SMBIOS: Update header file definitions, Gabriel L. Somlo, 2014/04/23
- [Qemu-devel] [QEMU v8 PATCH 7/7] SMBIOS: Build aggregate smbios tables and entry point, Gabriel L. Somlo, 2014/04/23
- [Qemu-devel] [QEMU v8 PATCH 5/7] SMBIOS: Use macro to set smbios defaults, Gabriel L. Somlo, 2014/04/23
- [Qemu-devel] [QEMU v8 PATCH 3/7] SMBIOS: Rename symbols to better reflect future use, Gabriel L. Somlo, 2014/04/23
- [Qemu-devel] [QEMU v8 PATCH 2/7] PC: Add machine version 2.1 for piix and q35, Gabriel L. Somlo, 2014/04/23
- [Qemu-devel] [QEMU v8 PATCH 6/7] SMBIOS: Use bitmaps to prevent incompatible comand line options, Gabriel L. Somlo, 2014/04/23
- Re: [Qemu-devel] [QEMU v8 PATCH 0/7] SMBIOS: build full tables in QEMU, Gerd Hoffmann, 2014/04/23