From: Paul Mackerras <paulus@samba.org>
To: Segher Boessenkool <segher@kernel.crashing.org>
Cc: Jimi Xenidis <jimix@pobox.com>, KVM list <kvm@vger.kernel.org>,
Alexander Graf <agraf@suse.de>,
Linuxppc-dev <linuxppc-dev@ozlabs.org>,
kvm-ppc@vger.kernel.org
Subject: Re: [PATCH 10/13] kvm/powerpc: Add support for Book3S processors in
Date: Wed, 01 Jun 2011 05:11:45 +0000 [thread overview]
Message-ID: <20110601051145.GB6773@brick.ozlabs.ibm.com> (raw)
In-Reply-To: <c71b71560e43813012b6f293230dc78e@kernel.crashing.org>
On Wed, Jun 01, 2011 at 12:34:43AM +0200, Segher Boessenkool wrote:
> There's a loop of 128 iterations of 3 insns.
>
> I'm not saying it is actually bad, just that that 50 is slightly off ;-)
That would be the TLB invalidation. On POWER7 we only need to do that
if the virtual cpu last ran on a different physical cpu, or if this
physical cpu last ran a different virtual cpu in the same partition.
So hopefully we don't have to do it very often.
The reason we have to do it in that case is to allow the optimization
where we use tlbiel for TLB invalidations if the guest claims that
the translation being invalidated was only ever used on this virtual
cpu. That means that we have to guard against stale TLB entries left
behind when a virtual cpu moves from one physical cpu to another.
Paul.
next prev parent reply other threads:[~2011-06-01 5:11 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20110511103443.GA2837@brick.ozlabs.ibm.com>
[not found] ` <20110511104456.GK2837@brick.ozlabs.ibm.com>
2011-05-15 21:58 ` [PATCH 10/13] kvm/powerpc: Add support for Book3S processors in hypervisor mode Alexander Graf
2011-05-16 5:58 ` [PATCH 10/13] kvm/powerpc: Add support for Book3S processors in Paul Mackerras
2011-05-17 10:17 ` [PATCH 10/13] kvm/powerpc: Add support for Book3S processors in hypervisor mode Alexander Graf
2011-05-27 10:33 ` [PATCH 10/13] kvm/powerpc: Add support for Book3S processors in Paul Mackerras
2011-05-27 10:43 ` [PATCH 10/13] kvm/powerpc: Add support for Book3S processors in hypervisor mode Alexander Graf
2011-05-27 20:59 ` Segher Boessenkool
2011-05-27 23:19 ` Alexander Graf
2011-05-28 1:07 ` Segher Boessenkool
2011-05-31 20:26 ` Jimi Xenidis
2011-05-31 22:34 ` Segher Boessenkool
2011-06-01 5:11 ` Paul Mackerras [this message]
[not found] ` <764651B6-6DDA-49DC-AA95-52CD86FF82D9@suse.de>
[not found] ` <20110517111536.GC7924@brick.ozlabs.ibm.com>
[not found] ` <14CFAA70-1747-4902-8CC1-4BE924CAD031@suse.de>
[not found] ` <4DD25F10.4030707@redhat.com>
[not found] ` <20110519052246.GA8165@drongo>
2011-05-19 6:01 ` [PATCH 0/13] Hypervisor-mode KVM on POWER7 Alexander Graf
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=20110601051145.GB6773@brick.ozlabs.ibm.com \
--to=paulus@samba.org \
--cc=agraf@suse.de \
--cc=jimix@pobox.com \
--cc=kvm-ppc@vger.kernel.org \
--cc=kvm@vger.kernel.org \
--cc=linuxppc-dev@ozlabs.org \
--cc=segher@kernel.crashing.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