[PATCH 1/5] app/test-pm: add multiprocess test
    Stephen Hemminger 
    stephen at networkplumber.org
       
    Tue Dec 12 18:09:12 CET 2023
    
    
  
On Tue, 12 Dec 2023 06:25:12 +0200
Artemy Kovalyov <artemyko at nvidia.com> wrote:
> +rte_atomic32_t g_count;
> +
> +static int
> +done(const struct rte_mp_msg *msg __rte_unused, const void *arg __rte_unused)
> +{
> +	rte_atomic32_dec(&g_count);
> +	return 0;
> +}
Local variable, should be static.
Also, assert may not be the ideal way to report test failures.
The preferred way would be to use RTE_TEST_ASSERT() and RTE_TEST_ASSERT_EQUAL()
    
    
More information about the dev
mailing list