From: "Michael S. Tsirkin" <mst@redhat.com>
To: Gal Hammer <ghammer@redhat.com>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH V10 0/3] Virtual Machine Generation ID
Date: Mon, 26 Jan 2015 20:33:02 +0200 [thread overview]
Message-ID: <20150126183302.GA8022@redhat.com> (raw)
In-Reply-To: <1418546511-13063-1-git-send-email-ghammer@redhat.com>
On Sun, Dec 14, 2014 at 10:41:48AM +0200, Gal Hammer wrote:
> Hi,
>
> Resending patches after the release of version 2.2.
>
> Please note that this patch set doesn't include the *.hex.generated
> files and the binary ACPI tables (make check will fail).
>
> Thanks,
>
> Gal.
Hi Gal,
I had to revert this - pls address the issues found by Igor.
Looking at spec, it also seems safer to reserve a full 4K bytes of
memory, to guarantee that the memory won't be shared with some
uncacheable page.
> VX - Fixed typos in docs and a few clarification.
>
> V9 - Add a unit test.
> - Rebased to version 2.2.
> - Removed hex.generated the binary files from patch.
>
> 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: add a unit test for the vmgenid device.
>
> default-configs/i386-softmmu.mak | 1 +
> default-configs/x86_64-softmmu.mak | 1 +
> docs/specs/vmgenid.txt | 38 ++++++++++++++++++++++++++++
> 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/pc.c | 8 ++++++
> hw/i386/q35-acpi-dsdt.dsl | 5 +++-
> hw/i386/ssdt-misc.dsl | 43 ++++++++++++++++++++++++++++++++
> hw/isa/lpc_ich9.c | 1 +
> hw/misc/Makefile.objs | 1 +
> 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 +++
> tests/Makefile | 2 ++
> tests/vmgenid-test.c | 48 ++++++++++++++++++++++++++++++++++++
> 19 files changed, 211 insertions(+), 2 deletions(-)
> create mode 100644 docs/specs/vmgenid.txt
> create mode 100644 tests/vmgenid-test.c
>
> --
> 1.9.3
prev parent reply other threads:[~2015-01-26 18:33 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-12-14 8:41 [Qemu-devel] [PATCH V10 0/3] Virtual Machine Generation ID Gal Hammer
2014-12-14 8:41 ` [Qemu-devel] [PATCH V10 1/3] docs: vm generation id device's description Gal Hammer
2014-12-14 8:41 ` [Qemu-devel] [PATCH V10 2/3] i386: Add a Virtual Machine Generation ID device Gal Hammer
2015-01-26 18:36 ` Eric Blake
2014-12-14 8:41 ` [Qemu-devel] [PATCH V10 3/3] tests: add a unit test for the vmgenid device Gal Hammer
2015-01-26 18:33 ` Michael S. Tsirkin [this message]
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=20150126183302.GA8022@redhat.com \
--to=mst@redhat.com \
--cc=ghammer@redhat.com \
--cc=qemu-devel@nongnu.org \
/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).