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 {