On 25.06.2010, at 11:02, Denis Kirjanov wrote:

On 06/25/2010 12:42 AM, Alexander Graf wrote:

quoted

=20
On 24.06.2010, at 21:44, Denis Kirjanov wrote:
=20

quoted

Fix build warning:
arch/powerpc/kvm/book3s_64_mmu.c: In function =

'kvmppc_mmu_book3s_64_esid_to_vsid':

quoted

quoted

arch/powerpc/kvm/book3s_64_mmu.c:446: warning: 'slb' may be used =

uninitialized in this function

quoted

quoted

Signed-off-by: Denis Kirjanov<dkirjanov@kernel.org>

=20
Are you sure this isn't a broken compiler? I don't see where it could =

be used uninitialized.

=20
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
=20

Oh, I'm apparently looking at completely different code. The same =
function in git://git.kernel.org/pub/scm/virt/kvm/kvm.git is good. Which =
tree did you use?


Alex