h_crc is __le32 but cpu_to_be32() is... __be32. So sparse complains, even though it's harmless. Although sparse is smart about bare 0s, and we could drop the swap, other places explicitly swap to keep things clear (I guess?) so "swap" the 0 with the proper routine. Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxx> --- diff --git a/libxfs/rdwr.c b/libxfs/rdwr.c index 9ee89d3..dd8baa6 100644 --- a/libxfs/rdwr.c +++ b/libxfs/rdwr.c @@ -187,7 +187,7 @@ libxfs_log_header( head->h_len = cpu_to_be32(sunit - BBSIZE); else head->h_len = cpu_to_be32(20); - head->h_crc = cpu_to_be32(0); + head->h_crc = cpu_to_le32(0); head->h_prev_block = cpu_to_be32(-1); head->h_num_logops = cpu_to_be32(1); head->h_cycle_data[0] = cpu_to_be32(0xb0c0d0d0); _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs