On 4/22/22 17:13, Eelco Chaudron wrote: > > > On 22 Apr 2022, at 15:45, Harry van Haaren wrote: > >> The code changes here are to handle (1 << i) shifts where 'i' is the >> packet index in the batch, and 1 << 31 is an overflow of the signed '1'. >> >> Fixed by adding UINT32_C() around the 1 character, ensuring compiler knows >> the 1 is unsigned (and 32-bits). Undefined Behaviour sanitizer is now happy >> with the bit-shifts at runtime. >> >> Suggested-by: Ilya Maximets <i.maxim...@ovn.org> >> Signed-off-by: Harry van Haaren <harry.van.haa...@intel.com> >> >> --- > > Compile tested as I do not have an AVX machine. > > Acked-by: Eelco Chaudron <echau...@redhat.com> >
Thanks! Applied to master and branch-2.17. Best regards, Ilya Maximets. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
