From: Paolo Bonzini <pbonzini@redhat.com>
To: Nadav Amit <namit@cs.technion.ac.il>
Cc: kvm@vger.kernel.org
Subject: Re: [PATCH 0/2] KVM: x86: Fix REP-string effect on RCX/RSI/RDI
Date: Sun, 10 May 2015 17:28:39 +0200 [thread overview]
Message-ID: <554F7927.6040303@redhat.com> (raw)
In-Reply-To: <1430215561-6677-1-git-send-email-namit@cs.technion.ac.il>
On 28/04/2015 12:05, Nadav Amit wrote:
> This patch-set fixes KVM behavior when handling a REP-string instruction that
> runs with an address-size of 32-bit. In this case ECX/EDI/ESI are used as
> counter and pointers, and the high 32-bits should be cleared.
>
> The first patch handles with the simple case. The second one handles the
> corner-case in which ECX is initially zero. It appears that Intel and AMD
> behave differently in this case (and some experiments suggest even different
> Intel generations act differently), and I could not find any documentation that
> describes it. Yet, the behavior of INS/OUTS can be observed by the guest and
> VMware appears to get it right.
>
> Thanks for reviewing the patches.
>
> Nadav Amit (2):
> KVM: x86: Fix update RCX/RDI/RSI on REP-string
> KVM: x86: Fix zero iterations REP-string
>
> arch/x86/kvm/emulate.c | 33 +++++++++++++++++++++++++++------
> 1 file changed, 27 insertions(+), 6 deletions(-)
>
Applied, thanks.
Paolo
prev parent reply other threads:[~2015-05-10 15:28 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-04-28 10:05 [PATCH 0/2] KVM: x86: Fix REP-string effect on RCX/RSI/RDI Nadav Amit
2015-04-28 10:06 ` [PATCH 1/2] KVM: x86: Fix update RCX/RDI/RSI on REP-string Nadav Amit
2015-04-28 10:06 ` [PATCH 2/2] KVM: x86: Fix zero iterations REP-string Nadav Amit
2015-05-10 15:28 ` Paolo Bonzini [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=554F7927.6040303@redhat.com \
--to=pbonzini@redhat.com \
--cc=kvm@vger.kernel.org \
--cc=namit@cs.technion.ac.il \
/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.