From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH 0/5][RESEND] Split the emulator: decode & execute Date: Tue, 18 Sep 2007 12:16:51 +0200 Message-ID: <46EFA593.2010706@qumranet.com> References: <46EF99C1.4070801@bull.net> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: kvm-devel To: Laurent Vivier Return-path: In-Reply-To: <46EF99C1.4070801-6ktuUTfB/bM@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: kvm-devel-bounces-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org Errors-To: kvm-devel-bounces-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org List-Id: kvm.vger.kernel.org Laurent Vivier wrote: > These patches split the emulator in two parts: one to decode the instruction, > the other to execute it. The decode part is then called only when needed. > > Applied all, thanks. I changed the name of the variable 'decode' to 'c' to reduce the number of line splits. This patchset, in addition to the correctness issues, also enables further cleaning up of x86_emulate.c: since most variables are now in a structure, chunks of the code can be moved to a separate function that takes just the decode cache. This will reduce the high indent level and increase readability. -- error compiling committee.c: too many arguments to function ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2005. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/