qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Wei Huang <wei@redhat.com>
To: qemu-devel@nongnu.org
Cc: wei@redhat.com, peter.maydell@linaro.org, drjones@redhat.com,
	ard.biesheuvel@linaro.org, ehabkost@redhat.com,
	ivan.khoronzhuk@linaro.org, mst@redhat.com, somlo@cmu.edu,
	zhaoshenglong@huawei.com, leif.lindholm@linaro.org,
	roy.franz@linaro.org, pbonzini@redhat.com, imammedo@redhat.com,
	lersek@redhat.com, jdelvare@suse.de, rth@twiddle.net
Subject: [Qemu-devel] [ARM SMBIOS V5 PATCH 0/2]SMBIOS Support for ARM
Date: Wed, 26 Aug 2015 15:04:28 -0400	[thread overview]
Message-ID: <1440615870-9518-1-git-send-email-wei@redhat.com> (raw)

SMBIOS tables present userful system hardware info to management
applications, such as DMI tools. Even though SMBIOS was originally
developed for Intel x86, it has been extended to both Itanium and 
ARM (32bit & 64bit). More and more ARM server releases, such as 
RHEL Server for ARM, start to integrate support for SMBIOS.

This patchset is intendted to provid SMBIOS tables for ARM mach-virt
machine. The SMBIOS tables are created and stored in fw_cfg, relying on
OVMF (AAVMF) to parse/present SMBIOS entry.

Given that refractoring patches have been accepted by upstream. This new
version (V5) integrates SMBIOS 3.0 support for ARM mach-virt. I have
tested this version using a customized AAVMF created by Laszlo, who
has submitted his patches to OVMF mailing list. Compatibility tests 
for x86 VMs (Linux & Windows) also passed.

V4->V5:
 * Fixed tests/bios-tables-test
 * Validated with ARM64/x86-64 VMs and passed "make test"

V3->V4:
 * Patch 1 - 3 accepted by mst tree. So start from Patch 4 & 5
 * Remove ep_length; Instead infer anchor_length from anchor string (mst)

V2->V3:
 * Removed unncessary ram_size paramemter (patch 3 in V2, Laszlo)
 * Fixed UUID encode (Laszlo)
 * Added -smbios option (Leif)
 * Fixed misc variable defintion (Laszlo)
 * V2 regression tested on x86 (Gabriel and Leif)

V1->V2:
 * Add NULL checking for fw_cfg (Shannon Zhao)
 * Init 3.0 entry point table max size to smbios_tables_len (Laszlo)
 * Minor re-arrangement of smbios.h layout with function headers to the bottom
 * Validated SMBIOS 3.0 tables with a customized AAVMF created by Laszlo

RFC->V1:
 * Add SMBIOS 3.0 support for buidling SMBIOS
 * Switch from SMBIOS 2.1 to 3.0 for ARM mach-virt
 * RFC version Tested-by Laszlo Ersek and Acked-by Gabriel Somlo

Thanks,
-Wei

Wei Huang (2):
  smbios: add smbios 3.0 support
  smbios: implement smbios support for mach-virt

 default-configs/arm-softmmu.mak |  1 +
 hw/arm/virt.c                   | 25 ++++++++++++
 hw/i386/pc_piix.c               |  3 +-
 hw/i386/pc_q35.c                |  3 +-
 hw/smbios/smbios.c              | 84 ++++++++++++++++++++++++++++++-----------
 include/hw/smbios/smbios.h      | 62 +++++++++++++++++++++++-------
 qemu-options.hx                 |  2 +-
 tests/bios-tables-test.c        |  6 +--
 8 files changed, 143 insertions(+), 43 deletions(-)

-- 
1.8.3.1

             reply	other threads:[~2015-08-26 19:04 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-08-26 19:04 Wei Huang [this message]
2015-08-26 19:04 ` [Qemu-devel] [ARM SMBIOS V5 PATCH 1/2] smbios: add smbios 3.0 support Wei Huang
2015-08-27  6:15   ` Laszlo Ersek
2015-09-01 15:56     ` Peter Maydell
2015-08-26 19:04 ` [Qemu-devel] [ARM SMBIOS V5 PATCH 2/2] smbios: implement smbios support for mach-virt Wei Huang
2015-09-01 15:58   ` Peter Maydell
2015-08-27  6:18 ` [Qemu-devel] [ARM SMBIOS V5 PATCH 0/2]SMBIOS Support for ARM Laszlo Ersek
2015-09-01 16:15 ` Peter Maydell

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1440615870-9518-1-git-send-email-wei@redhat.com \
    --to=wei@redhat.com \
    --cc=ard.biesheuvel@linaro.org \
    --cc=drjones@redhat.com \
    --cc=ehabkost@redhat.com \
    --cc=imammedo@redhat.com \
    --cc=ivan.khoronzhuk@linaro.org \
    --cc=jdelvare@suse.de \
    --cc=leif.lindholm@linaro.org \
    --cc=lersek@redhat.com \
    --cc=mst@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=roy.franz@linaro.org \
    --cc=rth@twiddle.net \
    --cc=somlo@cmu.edu \
    --cc=zhaoshenglong@huawei.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).