qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Gabriel L. Somlo" <gsomlo@gmail.com>
To: qemu-devel@nongnu.org
Cc: pbonzini@redhat.com, agraf@suse.de, afaerber@suse.de, mst@redhat.com
Subject: [Qemu-devel] [PATCH v6 0/2] apic: bump emulated lapic version to 0x14
Date: Tue,  6 May 2014 11:17:23 -0400	[thread overview]
Message-ID: <1399389445-1607-1-git-send-email-somlo@cmu.edu> (raw)

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

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

             reply	other threads:[~2014-05-06 15:17 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-06 15:17 Gabriel L. Somlo [this message]
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 ` [Qemu-devel] [PATCH v6 0/2] apic: bump emulated lapic version to 0x14 Michael S. Tsirkin
2014-05-07 10:37   ` 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=1399389445-1607-1-git-send-email-somlo@cmu.edu \
    --to=gsomlo@gmail.com \
    --cc=afaerber@suse.de \
    --cc=agraf@suse.de \
    --cc=mst@redhat.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 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).