On Sat, Jan 28, 2017 at 05:13:40PM -0500, Devesh Sharma wrote: > +#define upper_32_bits(n) ((__u32)(((n) >> 16) >> 16)) This can be >> 16 > +#define lower_32_bits(n) ((__u32)((n) & 0xFFFFFFFFUL)) Do not define macros as functions. Use static inline instead. Jason -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
