Hi David, On 05/13/2016 04:28 PM, David Marchand wrote: >> +#ifdef RTE_ARCH_PPC_64 >> + switch (length) { >> + case 4: >> + *(uint32_t *)dst = rte_be_to_cpu_32(*(uint32_t *)dst); >> + break; >> + case 2: >> + *(uint16_t *)dst = rte_be_to_cpu_16(*(uint16_t *)dst); >> + break; >> + default: >> + break; >> + } >> +#endif >> } > > I think that, in the original patch, I was handling lengths different > than 1, 2 and 4 ;-) > Idem for write.
Oh oh, spotted :) I'll fix that in the v2.