2018-03-23 16:36 GMT+01:00 Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>: > On Thu, Mar 15, 2018 at 12:18:58PM +0100, Salvatore Mesoraca wrote: >> >> +#define MAX_BLOCKSIZE 16 >> + >> +#ifdef CONFIG_HAVE_EFFICIENT_UNALIGNED_ACCESS >> +#define MAX_ALIGNMASK 15 >> +#else >> +#define MAX_ALIGNMASK 0 >> +#endif >> + > > Hmm, this won't work. Just because you have efficient unaligned > access in general doesn't mean that every implementation can live > with unaligned access. In particular, on x86 there are quite a > few implementations that require alignment or they will fault. > > So please just make it 15 unconditionally. Oh, thank you for pointing it out. I'll fix this in v3. Salvatore