On Wed, Jul 15, 2020 at 2:56 PM Aaron Conole <acon...@redhat.com> wrote:
> I guess the service_lcore_attr_get failed, but with no useful
> information. We should have the test suite app turn the log level all
> the way up. I realize it will increase the log data even higher, but
> without it, we have no idea why this test failed.
>
> Something like the following
> ---
>
> diff --git a/app/test/test.c b/app/test/test.c
> index 94d26ab1f6..c47cb075f9 100644
> --- a/app/test/test.c
> +++ b/app/test/test.c
> @@ -150,6 +150,9 @@ main(int argc, char **argv)
>
> prgname = argv[0];
>
> + rte_log_set_global_level(RTE_LOG_DEBUG);
> + rte_log_set_level(RTE_LOGTYPE_EAL, RTE_LOG_DEBUG);
> +
> recursive_call = getenv(RECURSIVE_ENV_VAR);
> if (recursive_call != NULL) {
> ret = do_recursive_call();
> ---
>
> That way we can at least debug when it happens.
Debugging is one thing.
But here service_lcore_attr_get() has a lot of asserts that should
trigger a straight error.
--
David Marchand
