From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:42001) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UoilW-0008LK-Ld for qemu-devel@nongnu.org; Mon, 17 Jun 2013 19:23:16 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UoilV-0004aB-IA for qemu-devel@nongnu.org; Mon, 17 Jun 2013 19:23:14 -0400 Sender: fluxion Date: Mon, 17 Jun 2013 18:22:53 -0500 From: mdroth Message-ID: <20130617232253.GB17657@vm> References: <1370610036-10577-1-git-send-email-armbru@redhat.com> <87a9mpdl5n.fsf@blackfin.pond.sub.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <87a9mpdl5n.fsf@blackfin.pond.sub.org> Subject: Re: [Qemu-devel] [Qemu-stable] [PATCH v2 0/6] Some -smbios work List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Markus Armbruster Cc: aliguori@us.ibm.com, lersek@redhat.com, qemu-devel@nongnu.org, qemu-stable@nongnu.org On Mon, Jun 17, 2013 at 01:36:04PM +0200, Markus Armbruster wrote: > Would this make sense for -stable? Impact is modest: fix a rather > obscure feature of x86 targets on bigendian hosts, and improve error > messages around it). On the other hand, the patches look pretty safe to > me. 5/6 was certainly a good candidate. I was going to cherry-pick it but I agree the other patches looked safe to I went ahead and applied the series: https://github.com/mdroth/qemu/commits/stable-1.5-staging > > Markus Armbruster writes: > > > Better error messages, a bit of code cleanup, and a big endian fix. > > > > Not addressed: qemu_uuid_parse() sets an SMBIOS field by side effect. > > Gross! > > > > Testing: > > * Verify error messages improve for > > -smbios gaga > > -smbios file=gaga > > -smbios type=42 > > -smbios type=1,uuid=gaga > > -smbios type=0,release=gaga > > * Verify SMBIOS table remains unchanged > > -smbios type=0,vendor=me,version=42,date=today,release=1.2 -smbios type=1,manufacturer=me,product=crap,version=6,serial=77,uuid=988bc9dd-0986-440f-ac24-cf9626c5aa88,sku=888,family=flintstones > > by sticking > > qemu_hexdump(smbios_entries, stdout, "SMBIOS", smbios_entries_len); > > into smbios_get_table() > > > > v2: Address "Hawkeye" Laszlo's review > > * 1-3/7 unchanged > > * Drop 4/7 because it's buggy, and the fixed version isn't worthwhile > > * Spelling fix in commit message of 5/7 > > * Correct scanf format in 5-6/7 > > > > Markus Armbruster (6): > > error-report.h: Supply missing include > > log.h: Supply missing includes > > smbios: Convert to error_report() > > smbios: Clean up smbios_add_field() parameters > > smbios: Fix -smbios type=0,release=... for big endian hosts > > smbios: Check R in -smbios type=0,release=R parses okay > > > > arch_init.c | 3 +-- > > hw/i386/smbios.c | 57 ++++++++++++++++++++++++--------------------- > > include/hw/i386/smbios.h | 2 +- > > include/qemu/error-report.h | 1 + > > include/qemu/log.h | 3 +++ > > 5 files changed, 37 insertions(+), 29 deletions(-) >