On Wed, Feb 27, 2013 at 01:49:12PM -0500, Theodore Ts'o wrote: > -#define EXTENT_STATUS_WRITTEN 0x80000000 /* written extent */ > -#define EXTENT_STATUS_UNWRITTEN 0x40000000 /* unwritten extent */ > -#define EXTENT_STATUS_DELAYED 0x20000000 /* delayed extent */ > -#define EXTENT_STATUS_HOLE 0x10000000 /* hole */ > +#define EXTENT_STATUS_WRITTEN (((unsigned long long) 1) << 63) > +#define EXTENT_STATUS_UNWRITTEN (((unsigned long long) 1) << 62) > +#define EXTENT_STATUS_DELAYED (((unsigned long long) 1) << 61) > +#define EXTENT_STATUS_HOLE (((unsigned long long) 1) << 60) Just a nitpick: 1ULL << ... is shorter and probably the standard way we do flags. -- Regards/Gruss, Boris. Sent from a fat crate under my desk. Formatting is fine. -- -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html