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

Nathan Lynch ntl at pobox.com
Thu May 7 14:17:27 PDT 2009


"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.


More information about the Containers mailing list