[Devel] Re: [PATCH 1/1] s390: cr: fix unterminated #ifndef

Serge E. Hallyn serue at us.ibm.com
Thu May 7 14:30:51 PDT 2009


Quoting Nathan Lynch (ntl at pobox.com):
> "Serge E. Hallyn" <serue at us.ibm.com> writes:
> > With this, v15 compiles and does checkpoint/restart just fine
> > on s390.
> >
> > Signed-off-by: Serge E. Hallyn <serue at us.ibm.com>
> > ---
> >  arch/s390/include/asm/checkpoint_hdr.h |    1 +
> >  1 files changed, 1 insertions(+), 0 deletions(-)
> >
> > diff --git a/arch/s390/include/asm/checkpoint_hdr.h b/arch/s390/include/asm/checkpoint_hdr.h
> > index 292dc33..48ee296 100644
> > --- a/arch/s390/include/asm/checkpoint_hdr.h
> > +++ b/arch/s390/include/asm/checkpoint_hdr.h
> > @@ -80,4 +80,5 @@ struct ckpt_hdr_header_arch {
> >  	struct ckpt_hdr h;
> >  };
> >  
> > +#endif /* __s390x__ */
> 
> I suggest just removing the #ifdef __s390x__.  32-bit builds may not be
> supported or tested yet, but just having the declarations isn't going to
> break things.  And it's one less thing to stumble over if someone does
> implement 32-bit support.

Actually, you're more right than you know - CONFIG_CHECKPOINT already is
off if not 64bit :)

So yeah, the ifdef should just be removed.

thanks,
-serge
_______________________________________________
Containers mailing list
Containers at lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/containers




More information about the Devel mailing list