Messages in this thread | | | Subject | Re: [PATCH -tip] x86: kvm/paging_tmpl.h intialize the variable before using it | From | Jaswinder Singh Rajput <> | Date | Mon, 25 May 2009 11:07:32 +0530 |
| |
On Mon, 2009-05-25 at 08:21 +0300, Avi Kivity wrote: > 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. >
hmm, In that case we should use a do-while flavor function so that the compiler will also happy with it.
-- JSR
| |