From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:49882) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XqIDy-0002Sc-HD for qemu-devel@nongnu.org; Mon, 17 Nov 2014 04:03:59 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XqIDt-0005FT-DY for qemu-devel@nongnu.org; Mon, 17 Nov 2014 04:03:54 -0500 Received: from mx3-phx2.redhat.com ([209.132.183.24]:55117) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XqIDt-0005FL-74 for qemu-devel@nongnu.org; Mon, 17 Nov 2014 04:03:49 -0500 Date: Mon, 17 Nov 2014 04:03:45 -0500 (EST) From: Gal Hammer Message-ID: <1863093929.11346847.1416215025616.JavaMail.zimbra@redhat.com> In-Reply-To: <20141116194937.GA13687@redhat.com> References: <1416132959-25905-1-git-send-email-ghammer@redhat.com> <20141116194937.GA13687@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH V8 0/3] Virtual Machine Generation ID Reply-To: Gal Hammer List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Michael S. Tsirkin" Cc: pbonzini@redhat.com, qemu-devel@nongnu.org ----- Original Message ----- > From: "Michael S. Tsirkin" > To: "Gal Hammer" > Cc: pbonzini@redhat.com, qemu-devel@nongnu.org > Sent: Sunday, November 16, 2014 9:49:37 PM > Subject: Re: [Qemu-devel] [PATCH V8 0/3] Virtual Machine Generation ID > > On Sun, Nov 16, 2014 at 12:15:56PM +0200, Gal Hammer wrote: > > Hi, > > > > The patch grow to three parts now. Although it is still add a QEmu > > support for Microsoft's Virtual Machine Generation ID device. > > > > The first is a short device's description, then the ACPI tables > > changes and the actual device and the last patch updates the tests' > > ACPI tables. > > > > Your comment are welcomed. > > > > Thanks, > > > > Gal. > > Looks good, but how about a unit test? > Should be easy: give device a physical address, > read back memory to see that it's been written to. I thought it would make sense to add the tests only after the patch is accepted and the device's implementation is final. > > > > > V8 - Add a device's description file. > > - GUID is stored in fw cfg file and the guest writes the > > physical address to the device (reduces vmexits). > > > > V7 - Move the device's description back to the static SSDT table. > > - The GUID is store in a "hard coded" physical address and not > > in the ACPI table itself. > > - ACPI notification is triggered when the GUID is changed. > > > > V6 - include the pre-compiled ASL file > > - remove an empty line at end of files. > > > > V5 - Move device's description to SSDT table (dynamic). > > > > V4 - Fix a typo in error message string. > > - Move device's description from DSDT back to SSDT table. > > > > V3 - Remove "-uuid" command line parameter. > > - Move device's description from SSDT to DSDT table. > > - Add new "vmgenid" sysbus device. > > > > Gal Hammer (3): > > docs: vm generation id device's description > > i386: Add a Virtual Machine Generation ID device > > tests: update acpi tables after adding the vmgenid device > > > > default-configs/i386-softmmu.mak | 1 + > > default-configs/x86_64-softmmu.mak | 1 + > > docs/specs/vmgenid.txt | 27 ++++++++ > > hw/acpi/core.c | 8 +++ > > hw/acpi/ich9.c | 8 +++ > > hw/acpi/piix4.c | 8 +++ > > hw/i386/acpi-build.c | 26 +++++++ > > hw/i386/acpi-dsdt.dsl | 4 +- > > hw/i386/acpi-dsdt.hex.generated | 6 +- > > hw/i386/pc.c | 8 +++ > > hw/i386/q35-acpi-dsdt.dsl | 5 +- > > hw/i386/q35-acpi-dsdt.hex.generated | 8 +-- > > hw/i386/ssdt-misc.dsl | 43 ++++++++++++ > > hw/i386/ssdt-misc.hex.generated | 8 +-- > > hw/isa/lpc_ich9.c | 1 + > > hw/misc/Makefile.objs | 1 + > > hw/misc/vmgenid.c | 131 > > +++++++++++++++++++++++++++++++++++ > > include/hw/acpi/acpi.h | 2 + > > include/hw/acpi/acpi_dev_interface.h | 4 ++ > > include/hw/acpi/ich9.h | 2 + > > include/hw/i386/pc.h | 3 + > > include/hw/misc/vmgenid.h | 21 ++++++ > > tests/acpi-test-data/pc/DSDT | Bin 2807 -> 2820 bytes > > tests/acpi-test-data/pc/SSDT | Bin 3065 -> 3268 bytes > > tests/acpi-test-data/q35/DSDT | Bin 7397 -> 7410 bytes > > tests/acpi-test-data/q35/SSDT | Bin 1346 -> 1549 bytes > > 26 files changed, 313 insertions(+), 13 deletions(-) > > create mode 100644 docs/specs/vmgenid.txt > > create mode 100644 hw/misc/vmgenid.c > > create mode 100644 include/hw/misc/vmgenid.h > > > > -- > > 1.9.3 > >