On Sat, Jan 12, 2013 at 04:32:50PM +0200, Avi Kivity wrote:
> Signed-off-by: Avi Kivity <avi.kiv...@gmail.com>
> ---
> arch/x86/kvm/emulate.c | 33 +++++++++++++++++++++------------
> 1 file changed, 21 insertions(+), 12 deletions(-)
>
> diff --git a/arch/x86/kvm/emulate.c b/arch/x86/kvm/emulate.c
> index 619a33d..2189c6a 100644
> --- a/arch/x86/kvm/emulate.c
> +++ b/arch/x86/kvm/emulate.c
> @@ -454,6 +454,8 @@ static void invalidate_registers(struct x86_emulate_ctxt
> *ctxt)
> #define FOP_END \
> ".popsection")
>
> +#define FOPNOP() FOP_ALIGN FOP_RET
> +
> #define FOP1E(op, dst) \
> FOP_ALIGN #op " %" #dst " \n\t" FOP_RET
>
> @@ -476,6 +478,18 @@ static void invalidate_registers(struct x86_emulate_ctxt
> *ctxt)
> ON64(FOP2E(op##q, rax, rbx)) \
> FOP_END
>
> +#define FOP3E(op, dst, src, src2) \
> + FOP_ALIGN #op " %" #src2 ", %" #src ", %" #dst " \n\t" FOP_RET
> +
> +/* 3-operand, word-only, src2=cl */
> +#define FASTOP3WCL(op) \
> + FOP_START(op) \
> + FOPNOP() \
> + FOP3E(op, ax, bx, cl) \
> + FOP3E(op, eax, ebx, cl) \
> + ON64(FOP3E(op, rax, rbx, cl)) \
All other FASTOPs have explicit operand size. It is not strictly required
since assembler can figure it out from register name, but for consistency
why not put it there too?
--
Gleb.
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
