Re: [PATCH v2] mbuf: replace GCC marker extension with C11 anonymous unions

2024-02-13 Thread Tyler Retzlaff
On Tue, Feb 13, 2024 at 08:27:52PM +0100, Morten Brørup wrote: > > From: Tyler Retzlaff [mailto:roret...@linux.microsoft.com] > > Sent: Tuesday, 13 February 2024 19.48 > > > > On Tue, Feb 13, 2024 at 05:58:21PM +0100, Morten Brørup wrote: > > > > From: Tyler Retzlaff [mailto:roret...@linux.microso

RE: [PATCH v2] mbuf: replace GCC marker extension with C11 anonymous unions

2024-02-13 Thread Morten Brørup
> From: Tyler Retzlaff [mailto:roret...@linux.microsoft.com] > Sent: Tuesday, 13 February 2024 19.48 > > On Tue, Feb 13, 2024 at 05:58:21PM +0100, Morten Brørup wrote: > > > From: Tyler Retzlaff [mailto:roret...@linux.microsoft.com] > > > Sent: Tuesday, 13 February 2024 07.46 > > > > > > Replace t

Re: [PATCH v2] mbuf: replace GCC marker extension with C11 anonymous unions

2024-02-13 Thread Tyler Retzlaff
On Tue, Feb 13, 2024 at 05:58:21PM +0100, Morten Brørup wrote: > > From: Tyler Retzlaff [mailto:roret...@linux.microsoft.com] > > Sent: Tuesday, 13 February 2024 07.46 > > > > Replace the use of RTE_MARKER with C11 anonymous unions to improve > > code portability between toolchains. > > How about

RE: [PATCH v2] mbuf: replace GCC marker extension with C11 anonymous unions

2024-02-13 Thread Morten Brørup
> From: Tyler Retzlaff [mailto:roret...@linux.microsoft.com] > Sent: Tuesday, 13 February 2024 07.46 > > Replace the use of RTE_MARKER with C11 anonymous unions to improve > code portability between toolchains. How about combining the cacheline 0 marker and padding, like this: struct rte_mbuf {

[PATCH v2] mbuf: replace GCC marker extension with C11 anonymous unions

2024-02-12 Thread Tyler Retzlaff
Replace the use of RTE_MARKER with C11 anonymous unions to improve code portability between toolchains. Update use of rte_mbuf rearm_data field in net/ionic, net/sfc, net/ixgbe and net/virtio which were accessing field as a zero-length array. Signed-off-by: Tyler Retzlaff --- drivers/net/ionic/