Messages in this thread | | | Date | Mon, 25 May 2009 08:21:32 +0300 | From | Avi Kivity <> | Subject | Re: [PATCH -tip] x86: kvm/paging_tmpl.h intialize the variable before using it |
| |
Jaswinder Singh Rajput wrote: >> It's a false alarm. Isn't there a macro to shut up the warning? >> >> > > In arch/x86/kvm/paging_tmpl.h sptep is initialize only in : > > for_each_shadow_entry(vcpu, addr, iterator) { > > If we skip this, then we end up with wild sptep and returning it. > > Do you still think it is a false alarm ? >
That loop always has at least one iteration.
-- I have a truly marvellous patch that fixes the bug which this signature is too narrow to contain.
| |