> -----Original Message----- > From: Stephen Hemminger <[email protected]> > Sent: Monday, September 7, 2026 8:31 PM > To: [email protected] > Cc: Stephen Hemminger <[email protected]>; Dariusz Sosnowski > <[email protected]>; Slava Ovsiienko <[email protected]>; Bing > Zhao <[email protected]>; Ori Kam <[email protected]>; Suanming Mou > <[email protected]>; Matan Azrad <[email protected]> > Subject: [PATCH v3 4/5] net/mlx5: drop unnecessary STRICT_ALIGN > > The transmit inline copy splits the 8 byte case into two 32 bit moves when > RTE_ARCH_STRICT_ALIGN is set. Only armv8 aarch32 ever set that flag, and > ARMv8 does unaligned access in hardware, so the split gains nothing. Use a > single 64 bit move. > > The destination is inline_data, at offset 4 of a 16 byte aligned dseg, so the > 8 > byte store is always misaligned. Write it through the unaligned type; a plain > uint64_t store there is undefined behaviour and is reported by UBSAN. > > The debug assertion on the inline data offset goes away with the strict > alignment path since the wider move has no such requirement. > > Signed-off-by: Stephen Hemminger <[email protected]>
Acked-by: Dariusz Sosnowski <[email protected]>
