Re: [PATCH 2/2] mempool: use rte constant macro instead of GCC builtin

2024-07-03 Thread Patrick Robb
On Wed, Jul 3, 2024 at 10:22 AM David Marchand wrote: > > > > From the discussions on the retest mechanism, I understand we need to > ask for a rebase. > I sent a new retest. Let's see... > Hi, That makes sense that we need to re-apply on the latest mainline and retest. But, the "rebase" feature

Re: [PATCH 2/2] mempool: use rte constant macro instead of GCC builtin

2024-07-03 Thread David Marchand
On Wed, Jul 3, 2024 at 3:49 PM Morten Brørup wrote: > > > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > Sent: Wednesday, 3 July 2024 15.17 > > > > 14/06/2024 16:32, David Marchand: > > > On Wed, May 29, 2024 at 4:51 PM Thomas Monjalon > > > wrote: > > > > > > > > 20/03/2024 22:33, Tyler

RE: [PATCH 2/2] mempool: use rte constant macro instead of GCC builtin

2024-07-03 Thread Morten Brørup
> From: Thomas Monjalon [mailto:tho...@monjalon.net] > Sent: Wednesday, 3 July 2024 15.17 > > 14/06/2024 16:32, David Marchand: > > On Wed, May 29, 2024 at 4:51 PM Thomas Monjalon wrote: > > > > > > 20/03/2024 22:33, Tyler Retzlaff: > > > > Use newly introduced __rte_constant(e) macro instead of

Re: [PATCH 2/2] mempool: use rte constant macro instead of GCC builtin

2024-07-03 Thread Thomas Monjalon
14/06/2024 16:32, David Marchand: > On Wed, May 29, 2024 at 4:51 PM Thomas Monjalon wrote: > > > > 20/03/2024 22:33, Tyler Retzlaff: > > > Use newly introduced __rte_constant(e) macro instead of directly using > > > __builtin_constant_p() allowing mempool to be built by MSVC. > > > > Does it mean

Re: [PATCH 2/2] mempool: use rte constant macro instead of GCC builtin

2024-06-14 Thread David Marchand
On Wed, May 29, 2024 at 4:51 PM Thomas Monjalon wrote: > > 20/03/2024 22:33, Tyler Retzlaff: > > Use newly introduced __rte_constant(e) macro instead of directly using > > __builtin_constant_p() allowing mempool to be built by MSVC. > > Does it mean we should enable mempool build? > If yes, please

Re: [PATCH 2/2] mempool: use rte constant macro instead of GCC builtin

2024-05-29 Thread Thomas Monjalon
20/03/2024 22:33, Tyler Retzlaff: > Use newly introduced __rte_constant(e) macro instead of directly using > __builtin_constant_p() allowing mempool to be built by MSVC. Does it mean we should enable mempool build? If yes, please send a v2.

Re: [PATCH 2/2] mempool: use rte constant macro instead of GCC builtin

2024-05-29 Thread Andrew Rybchenko
On 3/26/24 12:57, Morten Brørup wrote: From: Tyler Retzlaff [mailto:roret...@linux.microsoft.com] Sent: Wednesday, 20 March 2024 22.34 Use newly introduced __rte_constant(e) macro instead of directly using __builtin_constant_p() allowing mempool to be built by MSVC. Signed-off-by: Tyler Retzlaf

RE: [PATCH 2/2] mempool: use rte constant macro instead of GCC builtin

2024-03-26 Thread Morten Brørup
> From: Tyler Retzlaff [mailto:roret...@linux.microsoft.com] > Sent: Wednesday, 20 March 2024 22.34 > > Use newly introduced __rte_constant(e) macro instead of directly using > __builtin_constant_p() allowing mempool to be built by MSVC. > > Signed-off-by: Tyler Retzlaff > --- Reviewed-by: Mort

[PATCH 2/2] mempool: use rte constant macro instead of GCC builtin

2024-03-20 Thread Tyler Retzlaff
Use newly introduced __rte_constant(e) macro instead of directly using __builtin_constant_p() allowing mempool to be built by MSVC. Signed-off-by: Tyler Retzlaff --- lib/mempool/rte_mempool.h | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/lib/mempool/rte_mempool.h b/l