lkml.org 
[lkml]   [2009]   [May]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH -tip] x86: kvm/paging_tmpl.h intialize the variable before using it
    From
    Date
    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



    \
     
     \ /
      Last update: 2009-05-25 07:43    [W:3.145 / U:0.564 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site