From: "Michael S. Tsirkin" <mst@redhat.com>
To: "Gabriel L. Somlo" <gsomlo@gmail.com>
Cc: Paolo Bonzini <pbonzini@redhat.com>,
agraf@suse.de, afaerber@suse.de, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH v6 0/2] apic: bump emulated lapic version to 0x14
Date: Wed, 7 May 2014 18:05:58 +0300 [thread overview]
Message-ID: <20140507150558.GA17381@redhat.com> (raw)
In-Reply-To: <20140507143933.GC1655@ERROL.INI.CMU.EDU>
On Wed, May 07, 2014 at 10:39:36AM -0400, Gabriel L. Somlo wrote:
> On Wed, May 07, 2014 at 05:02:42PM +0300, Michael S. Tsirkin wrote:
> > > 2. e1000 flakiness:
> > >
> > > - link negotiation is flaky (I have a patch, but was told
> > > and agree that it morally belongs in the BIOS layer)
> >
> > Could you remind me what the issue and the patch are?
>
> See:
>
> http://lists.nongnu.org/archive/html/qemu-devel/2013-11/msg01046.html
>
> and the thread of messages around it.
>
> > > - Mavericks doesn't like the current e1000 model
> > > (E1000_DEV_ID_82540EM); there was a patch sent to the list
> > > to allow selecting among different e1000 models, I think
> > > it was:
> > >
> > > http://lists.nongnu.org/archive/html/qemu-devel/2014-02/msg04411.html
> > >
> > > Mavericks likes E1000_DEV_ID_82545EM_COPPER, but Windows
> > > won't work with that, so picking which model to use from
> > > the command line would be nice. The patch seems abandoned
> > > at this point, so I might contact the author and ask if
> > > he minds me picking it up and continuing to polish it until
> > > it's acceptable :)
> >
> > Yay, please do that!
>
> OK, seems like the low hanging fruit right now :)
>
> > > - virtio-net works with an "aftermarket" driver, verified to
> > > work both in MountainLion and Mavericks. It's what I'm using
> > > right now.
> > >
> > > AFAIK, that's everything :)
> >
> > For older OSX we need the mwait nop hack right?
> > http://www.contrib.andrew.cmu.edu/~somlo/OSXKVM/kvm-mwait-nop-20140205.patch
>
> So, the "path of least resistance", in increasing order of
> "resistance" :) would be:
>
> 1. On the guest:
>
> rm -rf System/Library/Extensions/AppleIntelCPUPowerManagement.kext
>
> This switches the guest back to using "hlt"-based idle. Of course,
> you need to do something about monitor/mwait to get the guest up
> and running long enough to remove the monitor/mwait based idle kext,
> (see below) :)
>
> 2. Emulate monitor and mwait as "nop", but continue to claim they are
> not supported via CPUID. That's the patch you cited. Not sure
> though whether that sort of "undocumented" functionality would be
> OK with the KVM crowd, though :)
I'd go for this one. It seems unlikely a guest wants to get
an exception intentionally.
Paolo?
> 3. Emulate monitor and mwait (at page-size granularity) by
> write-protecting a MONITORed page and triggering MWAITs from
> the ensuing write fault handler:
>
> http://www.contrib.andrew.cmu.edu/~somlo/OSXKVM/kvm-mwait-emu-20140205.patch
>
> This latter patch is still (pre-)alpha, and was really flaky last
> time I played with it. I also don't have a good intuition about how
> (in-)elegant and (un-)acceptable a hack like this would be for KVM :)
>
> Given that it's only a problem for OS X 10.7 and older, my intuition
> tells me the "bang per buck" is not very good, regardless of how cool
> I personally think messing around with the guts of KVM might be :) :)
>
> Thanks,
> --Gabriel
next prev parent reply other threads:[~2014-05-07 15:07 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 ` [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 [this message]
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=20140507150558.GA17381@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.