Re: [PATCH v5 08/10] test-pmd: declare lcore_count atomic

2025-02-24 Thread Andre Muezerie
On Fri, Feb 21, 2025 at 09:58:28PM +0100, Morten Brørup wrote: > > From: Andre Muezerie [mailto:andre...@linux.microsoft.com] > > Sent: Friday, 21 February 2025 20.53 > > > > Compiling with MSVC results in the error below: > > > > app/test/test_ring_perf.c(197): error C7712: address argument to a

RE: [PATCH v5 08/10] test-pmd: declare lcore_count atomic

2025-02-24 Thread Konstantin Ananyev
> Subject: [PATCH v5 08/10] test-pmd: declare lcore_count atomic As a nit, the changes are not in testpmd, bit in the UT. Acked-by: Konstantin Ananyev > Compiling with MSVC results in the error below: > > app/test/test_ring_perf.c(197): error C7712: address argume

RE: [PATCH v5 08/10] test-pmd: declare lcore_count atomic

2025-02-21 Thread Morten Brørup
> From: Andre Muezerie [mailto:andre...@linux.microsoft.com] > Sent: Friday, 21 February 2025 20.53 > > Compiling with MSVC results in the error below: > > app/test/test_ring_perf.c(197): error C7712: address argument to atomic > operation must be a pointer to an atomic integer, > 'volati

[PATCH v5 08/10] test-pmd: declare lcore_count atomic

2025-02-21 Thread Andre Muezerie
Compiling with MSVC results in the error below: app/test/test_ring_perf.c(197): error C7712: address argument to atomic operation must be a pointer to an atomic integer, 'volatile unsigned int *' is not valid The fix is to mark lcore_count as atomic. Signed-off-by: Andre Muezerie Signed