From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from hera.kernel.org (hera.kernel.org [140.211.167.34]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id 3FBA1B6F0C for ; Fri, 25 Jun 2010 19:02:38 +1000 (EST) Message-ID: <4C2470AE.6090004@kernel.org> Date: Fri, 25 Jun 2010 13:02:38 +0400 From: Denis Kirjanov MIME-Version: 1.0 To: Alexander Graf Subject: Re: [PATCH] kvm/ppc: fix build warning References: <20100624194415.GA24107@hera.kernel.org> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Cc: mtosatti@redhat.com, kvm-ppc@vger.kernel.org, linuxppc-dev@ozlabs.org, Denis Kirjanov , avi@redhat.com List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On 06/25/2010 12:42 AM, Alexander Graf wrote: > > On 24.06.2010, at 21:44, Denis Kirjanov wrote: > >> Fix build warning: >> arch/powerpc/kvm/book3s_64_mmu.c: In function 'kvmppc_mmu_book3s_64_esid_to_vsid': >> arch/powerpc/kvm/book3s_64_mmu.c:446: warning: 'slb' may be used uninitialized in this function >> Signed-off-by: Denis Kirjanov > > Are you sure this isn't a broken compiler? I don't see where it could be used uninitialized. I'm using gcc version 4.3.4 (Gentoo 4.3.4 p1.1, pie-10.1.5) slb pointer initialized inside conditional branch and used later in the case case MSR_DR|MSR_IR