[Devel] Re: [PATCH 2/2] user-cr: s390: Detect s390x arch too

[Serge E. Hallyn]

Quoting Matt Helsley (matthltc at us.ibm.com):
> Signed-off-by: Matt Helsley <matthltc at us.ibm.com>

We don't actually support s390, but if/when we do it seems reasonable
to assume it'll be handled the same as s390x, so

Acked-by: Serge Hallyn <serue at us.ibm.com>

(Certainly the original code, checking only defined(__s390)) wasn't right)

> ---
>  ckptinfo.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/ckptinfo.c b/ckptinfo.c
> index b2101e6..6859742 100644
> --- a/ckptinfo.c
> +++ b/ckptinfo.c
> @@ -63,7 +63,7 @@ static int image_parse_error(struct ckpt_hdr *h, int fd, struct args *args);
>  #define __HAVE_image_parse_cpu
>  #define image_parse_cpu   image_parse_cpu_X86
>  static int image_parse_cpu_X86(struct ckpt_hdr *h, int fd, struct args *args);
> -#elif defined(__s390__)
> +#elif defined(__s390__) || defined(__s390x__)
>  #define __HAVE_image_parse_cpu
>  #define image_parse_cpu   image_parse_cpu_s390
>  static int image_parse_cpu_s390(struct ckpt_hdr *h, int fd, struct args *args);
> @@ -301,7 +301,7 @@ static int image_parse_cpu_X86(struct ckpt_hdr *h, int fd, struct args *args)
>  }
>  #endif
> 
> -#ifdef __s390__
> +#if defined(__s390__) || defined(__s390x__)
>  static int image_parse_cpu_s390(struct ckpt_hdr *h, int fd, struct args *args)
>  {
>  	struct ckpt_hdr_cpu *hh = (struct ckpt_hdr_cpu *) h;
> -- 
> 1.5.6.3
> 
> _______________________________________________
> Containers mailing list
> Containers at lists.linux-foundation.org
> https://lists.linux-foundation.org/mailman/listinfo/containers
_______________________________________________
Containers mailing list
Containers at lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/containers