Re: [PATCH 07/19] compat: add endianness helpers
- From
Jeff King <peff@peff.net>
- Date
- Oct 30, 2013, 08:25 UTC
- Message-ID
- <20131030082516.GN11317@sigill.intra.peff.net>
- In-Reply-To
- <87txg4sbyf.fsf@linux-k42r.v.cablecom.net>
On Sat, Oct 26, 2013 at 09:55:36AM +0200, Thomas Rast wrote:
> > The POSIX standard doesn't currently define a `nothll`/`htonll` > > typo: ntohll
Thanks.
Show 8 quoted lines
> > function pair to perform network-to-host and host-to-network > > swaps of 64-bit data. These 64-bit swaps are necessary for the on-disk > > storage of EWAH bitmaps if they are not in native byte order. > [...] > > +# include <byteswap.h> > > Do we need a hack on top similar to what ntoh_l and hton_l do, for > platforms that do not support unaligned access?
Ugh, probably. I didn't even know about those. But we do use them when reading the ewah bitmaps, which I believe can be at random offsets. We should be able to use the same ntoh_l solution.
-Peff