From mboxrd@z Thu Jan  1 00:00:00 1970
From: Gleb Natapov <gleb@redhat.com>
Subject: Re: [PATCH 0/7] Streamline arithmetic instruction emulation
Date: Sun, 23 Dec 2012 11:47:14 +0200
Message-ID: <20121223094714.GP17584@redhat.com>
References: <1356179217-5526-1-git-send-email-avi.kivity@gmail.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Cc: Marcelo Tosatti <mtosatti@redhat.com>, kvm@vger.kernel.org
To: Avi Kivity <avi.kivity@gmail.com>
Return-path: <kvm-owner@vger.kernel.org>
Received: from mx1.redhat.com ([209.132.183.28]:52054 "EHLO mx1.redhat.com"
	rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
	id S1751657Ab2LWJrR (ORCPT <rfc822;kvm@vger.kernel.org>);
	Sun, 23 Dec 2012 04:47:17 -0500
Content-Disposition: inline
In-Reply-To: <1356179217-5526-1-git-send-email-avi.kivity@gmail.com>
Sender: kvm-owner@vger.kernel.org
List-ID: <kvm.vger.kernel.org>

On Sat, Dec 22, 2012 at 02:26:50PM +0200, Avi Kivity wrote:
> The current arithmetic instruction emulation is fairly clumsy: after
> decode, each instruction gets a switch (size), and for every size
> we fetch the operands, prepare flags, emulate the instruction, then store
> back the flags and operands.
> 
> This patchset simplifies things by moving everything into common code
> except the instruction itself.  All the pre- and post- processing is
> coded just once.  The per-instrution code looks like:
> 
>   add %bl, %al
>   ret
> 
>   add %bx, %ax
>   ret
> 
>   add %ebx, %eax
>   ret
> 
>   add %rbx, %rax
>   ret
> 
> The savings in size, for the ten instructions converted in this patchset,
> are fairly large:
> 
>    text	   data	    bss	    dec	    hex	filename
>   63724	      0	      0	  63724	   f8ec	arch/x86/kvm/emulate.o.before
>   61268	      0	      0	  61268	   ef54	arch/x86/kvm/emulate.o.after
> 
> - around 2500 bytes.
> 
> Avi Kivity (7):
>   KVM: x86 emulator: framework for streamlining arithmetic opcodes
>   KVM: x86 emulator: Support for declaring single operand fastops
>   KVM: x86 emulator: introduce NoWrite flag
>   KVM: x86 emulator: mark CMP, CMPS, SCAS, TEST as NoWrite
>   KVM: x86 emulator: convert NOT, NEG to fastop
>   KVM: x86 emulator: add macros for defining 2-operand fastop emulation
>   KVM: x86 emulator: convert basic ALU ops to fastop
> 
>  arch/x86/kvm/emulate.c | 215 +++++++++++++++++++++++++++----------------------
>  1 file changed, 120 insertions(+), 95 deletions(-)
> 
Looks good to me.

--
			Gleb.