[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-ppc] [Qemu-devel] [PATCH] Bug fix for #986241: spell env corre

From: Andreas Färber
Subject: Re: [Qemu-ppc] [Qemu-devel] [PATCH] Bug fix for #986241: spell env correctly
Date: Sun, 22 Apr 2012 22:41:34 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:11.0) Gecko/20120328 Thunderbird/11.0.1

Am 22.04.2012 22:06, schrieb Peter Maydell:
> On 22 April 2012 20:56, Andreas Färber <address@hidden> wrote:
>> Commit ed120055c7f9b26b5707d3ceabbe5a3f06aaf937 (Implement PAPR VPA
>> functions for pSeries shared processor partitions) introduced the
>> register_dtl() function and typo "emv" as name of its argument.
>> This went unnoticed because the code in that function can access the
>> global variable "env" so that no build failure resulted.
>> Fix the argument to read "env". Resolves LP#986241.
> Incidentally, I had a quick look through that file and couldn't
> see any uses of 'env' outside of functions which pass it in --
> is it possible to drop the dyngen-exec.h include altogether?

A quick build test confirms that, yes. Blue's AREG0 series does not seem
to touch this file, so we could do it independently as a follow-up to
this one.


SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

reply via email to

[Prev in Thread] Current Thread [Next in Thread]