From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:56343) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZF7YT-0002MM-0c for qemu-devel@nongnu.org; Tue, 14 Jul 2015 17:15:57 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZF7YL-0004Ek-JU for qemu-devel@nongnu.org; Tue, 14 Jul 2015 17:15:56 -0400 Received: from mx1.redhat.com ([209.132.183.28]:53206) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZF7YL-0004EW-4W for qemu-devel@nongnu.org; Tue, 14 Jul 2015 17:15:49 -0400 Message-ID: <55A57C00.9050807@redhat.com> Date: Tue, 14 Jul 2015 23:15:44 +0200 From: Laszlo Ersek MIME-Version: 1.0 References: <1436905391-19262-1-git-send-email-wei@redhat.com> In-Reply-To: <1436905391-19262-1-git-send-email-wei@redhat.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [ARM SMBIOS RFC PATCH 0/5] SMBIOS Support for ARM List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Wei Huang Cc: peter.maydell@linaro.org, drjones@redhat.com, ehabkost@redhat.com, mst@redhat.com, Ard Biesheuvel , zhaoshenglong@huawei.co, qemu-devel@nongnu.org, imammedo@redhat.com, pbonzini@redhat.com, rth@twiddle.net Cc'ing Ard. On 07/14/15 22:23, Wei Huang wrote: > 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, awaiting > OVMF (AAVMF) to parse/present SMBIOS entry. > > NOTE: This is RFC and the patches haven't been fully tested yet. > Laszlo is kind enough to offer help on fixing AAVMF to enable support > for SMBIOS. I intend to start working on this tomorrow. (Well, not in ~50 minutes for sure, but you get the idea.) Thanks! Laszlo > Any comments are welcomed for next release. > > Thanks, > -Wei > > Wei Huang (5): > smbios: extract x86 smbios building code into a function > smbios: remove dependency on x86 e820 tables > smbios: pass ram size as a parameter to build smbios tables > smbios: move smbios code into a common folder > smbios: implement smbios support for mach-virt > > arch_init.c | 2 +- > default-configs/arm-softmmu.mak | 1 + > default-configs/i386-softmmu.mak | 1 + > default-configs/x86_64-softmmu.mak | 1 + > hw/Makefile.objs | 1 + > hw/arm/virt.c | 24 + > hw/i386/Makefile.objs | 2 +- > hw/i386/pc.c | 56 +- > hw/i386/pc_piix.c | 2 +- > hw/i386/pc_q35.c | 2 +- > hw/i386/smbios.c | 1102 ------------------------------------ > hw/smbios/Makefile.objs | 1 + > hw/smbios/smbios.c | 1102 ++++++++++++++++++++++++++++++++++++ > include/hw/arm/virt-acpi-build.h | 1 + > include/hw/i386/smbios.h | 227 -------- > include/hw/smbios/smbios.h | 237 ++++++++ > tests/bios-tables-test.c | 2 +- > vl.c | 2 +- > 18 files changed, 1414 insertions(+), 1352 deletions(-) > delete mode 100644 hw/i386/smbios.c > create mode 100644 hw/smbios/Makefile.objs > create mode 100644 hw/smbios/smbios.c > delete mode 100644 include/hw/i386/smbios.h > create mode 100644 include/hw/smbios/smbios.h >