IPv6 APIs rework
Robin Jarry
rjarry at redhat.com
Sat Jul 20 22:33:25 CEST 2024
Stephen Hemminger, Jul 20, 2024 at 22:26:
> There is no need for packing or alignment in in6_addr or current DPDK,
> what would be the benefit? Compilers generate worse code if
> a structure is marked packed.
The only benefit is to maintain current behaviour.
At first, I had not packed nor aligned anything and I had tons of test
errors because the compiler added padding in structures that contained
IPv6 addresses.
I don't want to mix things together. In my opinion, removing that
alignof(1) constraint is an optimization which has nothing to do with
the IPv6 API functional rework.
So my proposal is: add a structure *packed and unaligned* first so that
*all tests are passing*.
And *then*, after the changes have been applied on the main branch and
no critical issues have been reported, see if we need to remove these
packed and unaligned constraints.
More information about the dev
mailing list