From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marcelo Tosatti Subject: Re: [PATCH 00/13] Emulator group simplification Date: Tue, 27 Jul 2010 20:21:04 -0300 Message-ID: <20100727232104.GA4116@amt.cnet> References: <1280144271-328-1-git-send-email-avi@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: kvm@vger.kernel.org To: Avi Kivity Return-path: Received: from mx1.redhat.com ([209.132.183.28]:2374 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755258Ab0G1PFh (ORCPT ); Wed, 28 Jul 2010 11:05:37 -0400 Received: from int-mx02.intmail.prod.int.phx2.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) by mx1.redhat.com (8.13.8/8.13.8) with ESMTP id o6SF5bAs025900 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Wed, 28 Jul 2010 11:05:37 -0400 Content-Disposition: inline In-Reply-To: <1280144271-328-1-git-send-email-avi@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: On Mon, Jul 26, 2010 at 02:37:38PM +0300, Avi Kivity wrote: > This patchset simplifies the x86 emulator group decoding, cleans up the decoder > tables (and adds a missing Lock marker as well). > > Avi Kivity (13): > KVM: x86 emulator: add macros for repetitive instructions > KVM: x86 emulator: consolidate inc/dec reg decoding > KVM: x86 emulator: consolidate push/pop reg decoding > KVM: X86 emulator: consolidate Jcc rel8 decoding > KVM: x86 emulator: consolidate MOV reg, imm decoding > KVM: x86 emulator: consolidate CMOVcc decoding > KVM: x86 emulator: consolidate Jcc rel32 decoding > KVM: x86 emulator: Make group storage bits separate from operand bits > KVM: x86 emulator: add Undefined decode flag > KVM: x86 emulator: mix decode bits from opcode and group decode > tables > KVM: x86 emulator: simplify Group 1 decoding > KVM: x86 emulator: Allow LOCK prefix for NEG and NOT > KVM: x86 emulator: unify the two Group 3 variants > > arch/x86/kvm/emulate.c | 143 ++++++++++++++++-------------------------------- > 1 files changed, 47 insertions(+), 96 deletions(-) Applied, thanks.