On 6 June 2017 at 16:13, Ben Pfaff <[email protected]> wrote:
> On Fri, May 26, 2017 at 02:11:31PM -0700, Joe Stringer wrote:
>> Clang 4.0 complains:
>>
>> ../tests/test-hash.c:160:16: error: taking address of packed member 'b' of
>> class or structure 'offset_ovs_u128' may result in an unaligned pointer value
>>       [-Werror,-Waddress-of-packed-member]
>>         in0 = &in0_data.b;
>>
>> Set the bit in the aligned u128 first then copy the contents into the
>> offset u128 so that we don't have to take the address of the non-aligned
>> u128 and pass it to set_bit128.
>>
>> For the 256byte_hash, fix it up so that it's actually testing the 256B
>> hash inside a 32-bit offset u128 as well.
>>
>> Suggested-by: Ben Pfaff <[email protected]>
>> Signed-off-by: Joe Stringer <[email protected]>
>
> Thanks!
>
> Acked-by: Ben Pfaff <[email protected]>

Thanks, applied.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to