[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