Re: [PATCH] xfstests: fix compile error of src/fssum.c on bigendian host

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Oct 30, 2013 at 11:55:43AM +0800, Eryu Guan wrote:
> Definition of htonll() is needed on bigendian host too, otherwise
> src/fssum.c won't compile on ppc64/s390x hosts.
> 
> Signed-off-by: Eryu Guan <eguan@xxxxxxxxxx>
> ---
>  src/fssum.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/src/fssum.c b/src/fssum.c
> index c75ff8b..c26d32b 100644
> --- a/src/fssum.c
> +++ b/src/fssum.c
> @@ -50,6 +50,8 @@
>  
>  #if __BYTE_ORDER == __LITTLE_ENDIAN
>  #define htonll(x)     __bswap_64 (x)
> +#else
> +#define htonll(x)     (x)
>  #endif
>  #endif

Looks fine.

Acked-by: Dave Chinner <dchinner@xxxxxxxxxx>

-- 
Dave Chinner
david@xxxxxxxxxxxxx

_______________________________________________
xfs mailing list
xfs@xxxxxxxxxxx
http://oss.sgi.com/mailman/listinfo/xfs




[Index of Archives]     [Linux XFS Devel]     [Linux Filesystem Development]     [Filesystem Testing]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux