On Fri, Feb 24, 2017 at 09:23:26PM +0000, Weiny, Ira wrote: > > Perhaps... > > The question is do you want to rely on BSD or gnu extensions? My > reading of the various man pages leads me to the same path as you. > That the BSD htobe* has become the standard in glibc. If you go down that rat hole then __BYTE_ORDER__ isn't portable either.. Which ever you choose, both places should use an identical construct, introducing two different definitions is evil. bswap_64 is a tiny bit better because it does not rely on __USE_MISC which means the public header might compile with -std=c11/-ansi/etc However I did a little check and some of our public headers already do not support that mode... So my conclusion was to go with the simple solution, and mark it deprecated so we can remove it some day and avoid this worry. I guess it could get wrappered in '#ifdef htobe64' though - then it would still support -ansi mode. > > > +#define ntohll(x) x > > > > missing () around the x. > > Yea missed that... It is also problematic in that it doesn't force the output to a consistent 64 bit type, so big/little builds get different types which is never fun. 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
