From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH] KVM: fix mov immediate emulation for 64-bit operands Date: Sun, 08 Jan 2012 16:08:04 +0200 Message-ID: <4F09A344.8020000@redhat.com> References: <1325967346-12539-1-git-send-email-namit@cs.technion.ac.il> <4F08AA3C.9090306@zytor.com> <20120108102657.9a5dd6534ea4ae829073ff6e@gmail.com> <4F09A2A0.1060801@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: Takuya Yoshikawa , "H. Peter Anvin" , Marcelo Tosatti , Thomas Gleixner , Ingo Molnar , x86@kernel.org, kvm@vger.kernel.org, linux-kernel@vger.kernel.org To: Nadav Amit Return-path: In-Reply-To: <4F09A2A0.1060801@redhat.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org On 01/08/2012 04:05 PM, Avi Kivity wrote: > > > > Avi, please acknowledge adding OpImm64. > > Yes, OpImm64 is the cleanest IMO. Note it doesn't even cost us a bit. Note, usually I'd ask for a unit test to accompany the fix, but the current framework only supports testing instructions that have a memroy operand. So you're off the hook for now, unless you'd like to extend the framework. -- error compiling committee.c: too many arguments to function