Matthew Chapman wrote: > Jes, > > Glad you tracked it down. Can I suggest rather than using memcpy, a > more efficient way might be something like... > > #define ZERO_EXTEND(x,bits) ((x) & (~0UL >> (64-(bits)))) > > *dest = ZERO_EXTEND(p->u.ioreq.data, 8*s); Much nicer indeed! Here's a pretty version - Tony will you apply this one instead. Cheers, Jes