From: "Michael S. Tsirkin" <mst@redhat.com>
To: "Gabriel L. Somlo" <gsomlo@gmail.com>
Cc: pbonzini@redhat.com, agraf@suse.de, qemu-devel@nongnu.org,
afaerber@suse.de
Subject: Re: [Qemu-devel] [PATCH v6 0/2] apic: bump emulated lapic version to 0x14
Date: Wed, 7 May 2014 12:03:24 +0300 [thread overview]
Message-ID: <20140507090324.GA18283@redhat.com> (raw)
In-Reply-To: <1399389445-1607-1-git-send-email-somlo@cmu.edu>
On Tue, May 06, 2014 at 11:17:23AM -0400, Gabriel L. Somlo wrote:
> This patch set changes the software-emulated local apic version
> to 0x14 starting with pc machine types 2.1 and newer. This should
> be particularly helpful when running OS X guests with TCG, since
> XNU appears to have a hardcoded requirement that lapic version >= 0x14.
>
> Changelog:
>
> v6: - rebased to apply cleanly (no fuzz) against latest qemu git
> - opportunity to practice dealing with Acked-by and Reviewed-by :)
>
> v5: convert lapic version to uint8_t (only 8 bits dedicated to
> "implementation version" in the apic version register, according to
> the Intel spec).
>
> v4: - split into a two-patch series with cover letter
> - 1/2: - introduces empty 2.0 compat_props
> - depends on 3458b2b075f92f163ccb9a1f24733eb5705947f0 to add
> 2.1 machine type and move aliases (now already upstream, but
> not at the time v4 went out :)
> - 2/2: - adds lapic version as a machine property defaulting to 0x14
> - set to 0x11 in compat_props for machines 2.0 and older
>
> v3 and older: single patch, lapic version is global, no cover letter
>
> Thanks again,
> Gabriel
Applied, thanks for your patience.
> PS. Funny, now that I'm getting close to having figured out the qemu
> contributor netiquette, I'm just about done submitting all the changes
> I set out to contribute... :)
>
> Gabriel L. Somlo (2):
> pc: add compat_props placeholder for 2.0 machine type
> pic: use emulated lapic version 0x14 on pc machines >= 2.1
>
> hw/i386/pc_piix.c | 4 ++++
> hw/i386/pc_q35.c | 4 ++++
> hw/intc/apic.c | 2 +-
> hw/intc/apic_common.c | 1 +
> include/hw/i386/apic_internal.h | 1 +
> include/hw/i386/pc.h | 12 ++++++++++++
> 6 files changed, 23 insertions(+), 1 deletion(-)
>
> --
> 1.9.0
next prev parent reply other threads:[~2014-05-07 9:04 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-06 15:17 [Qemu-devel] [PATCH v6 0/2] apic: bump emulated lapic version to 0x14 Gabriel L. Somlo
2014-05-06 15:17 ` [Qemu-devel] [PATCH v6 1/2] pc: add compat_props placeholder for 2.0 machine type Gabriel L. Somlo
2014-05-06 15:17 ` [Qemu-devel] [PATCH v6 2/2] pic: use emulated lapic version 0x14 on pc machines >= 2.1 Gabriel L. Somlo
2014-05-06 17:54 ` Gabriel L. Somlo
2014-05-07 9:03 ` Michael S. Tsirkin [this message]
2014-05-07 10:37 ` [Qemu-devel] [PATCH v6 0/2] apic: bump emulated lapic version to 0x14 Paolo Bonzini
2014-05-07 13:45 ` Gabriel L. Somlo
2014-05-07 13:49 ` Paolo Bonzini
2014-05-07 14:02 ` Michael S. Tsirkin
2014-05-07 14:39 ` Gabriel L. Somlo
2014-05-07 15:05 ` Michael S. Tsirkin
2014-05-07 15:30 ` Paolo Bonzini
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=20140507090324.GA18283@redhat.com \
--to=mst@redhat.com \
--cc=afaerber@suse.de \
--cc=agraf@suse.de \
--cc=gsomlo@gmail.com \
--cc=pbonzini@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.