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.

