Re: [PATCH v3 2/2] net: have checksum routines accept unaligned data

2022-09-20 Thread Thomas Monjalon
20/09/2022 14:09, Mattias Rönnblom: > On 2022-07-11 15:25, Olivier Matz wrote: > > On Mon, Jul 11, 2022 at 02:11:32PM +0200, Mattias Rönnblom wrote: > >> __rte_raw_cksum() (used by rte_raw_cksum() among others) accessed its > >> data through an uint16_t pointer, which allowed the compiler to assume

Re: [PATCH v3 2/2] net: have checksum routines accept unaligned data

2022-09-20 Thread Mattias Rönnblom
On 2022-07-11 15:25, Olivier Matz wrote: > On Mon, Jul 11, 2022 at 02:11:32PM +0200, Mattias Rönnblom wrote: >> __rte_raw_cksum() (used by rte_raw_cksum() among others) accessed its >> data through an uint16_t pointer, which allowed the compiler to assume >> the data was 16-bit aligned. This in tur

Re: [PATCH v3 2/2] net: have checksum routines accept unaligned data

2022-08-08 Thread Mattias Rönnblom
On 2022-07-11 15:25, Olivier Matz wrote: > On Mon, Jul 11, 2022 at 02:11:32PM +0200, Mattias Rönnblom wrote: >> __rte_raw_cksum() (used by rte_raw_cksum() among others) accessed its >> data through an uint16_t pointer, which allowed the compiler to assume >> the data was 16-bit aligned. This in tur

Re: [PATCH v3 2/2] net: have checksum routines accept unaligned data

2022-07-11 Thread Olivier Matz
On Mon, Jul 11, 2022 at 02:11:32PM +0200, Mattias Rönnblom wrote: > __rte_raw_cksum() (used by rte_raw_cksum() among others) accessed its > data through an uint16_t pointer, which allowed the compiler to assume > the data was 16-bit aligned. This in turn would, with certain > architectures and comp

[PATCH v3 2/2] net: have checksum routines accept unaligned data

2022-07-11 Thread Mattias Rönnblom
__rte_raw_cksum() (used by rte_raw_cksum() among others) accessed its data through an uint16_t pointer, which allowed the compiler to assume the data was 16-bit aligned. This in turn would, with certain architectures and compiler flag combinations, result in code with SIMD load or store instruction