Re: [PATCH v6 2/3] test: add dispatcher test suite

[David Marchand <david.marchand@redhat.com>]

On Mon, Oct 9, 2023 at 8:22 PM Mattias Rönnblom
<mattias.ronnblom@ericsson.com> wrote:
> +static int
> +test_dispatcher(void)
> +{
> +       return unit_test_suite_runner(&test_suite);
> +}
> +
> +REGISTER_FAST_TEST(dispatcher_autotest, false, true, test_dispatcher);

Since this test expects some lcores, wdyt of adding:

@@ -1044,6 +1044,12 @@ static struct unit_test_suite test_suite = {
 static int
 test_dispatcher(void)
 {
+       if (rte_lcore_count() < NUM_SERVICE_CORES + 1) {
+               printf("Not enough cores for dispatcher_autotest,
expecting at least %u\n",
+                       NUM_SERVICE_CORES + 1);
+               return TEST_SKIPPED;
+       }
+
        return unit_test_suite_runner(&test_suite);
 }

This should avoid the failures we get with some CI env.
(additionnally, I tested this on my laptop and the test runs fine)


-- 
David Marchand