On Thu, Oct 3, 2024 at 8:57 AM David Marchand <david.march...@redhat.com> wrote:
>
> From: Mattias Rönnblom <mattias.ronnb...@ericsson.com>
>
> Calling rte_exit() from a worker lcore thread causes a deadlock in
> rte_service_finalize().
>
> This patch makes rte_service_finalize() deadlock-free by avoiding the
> need to synchronize with service lcore threads, which in turn is
> achieved by moving service and per-lcore state from the heap to being
> statically allocated.
>
> The BSS segment increases with ~156 kB (on x86_64 with default
> RTE_MAX_LCORE and RTE_SERVICE_NUM_MAX).
>
> According to the service perf autotest, this change also results in a
> slight reduction of service framework overhead.
>
> Fixes: 33666b448f15 ("service: fix crash on exit")
> Cc: sta...@dpdk.org
>
> Signed-off-by: Mattias Rönnblom <mattias.ronnb...@ericsson.com>
> Acked-by: Tyler Retzlaff <roret...@linux.microsoft.com>
> ---
> Changes since v1:
> - rebased,I can't merge this patch in its current state. At the moment, two CI report a problem with the eal_flags_file_prefix_autotest unit test. -------------------------------------stdout------------------------------------- RTE>>eal_flags_file_prefix_autotest Running binary with argv[]:'/home/zhoumin/gh_dpdk/build/app/dpdk-test' '--proc-type=secondary' '-m' '18' '--file-prefix=memtest' Running binary with argv[]:'/home/zhoumin/gh_dpdk/build/app/dpdk-test' '-m' '18' '--file-prefix=memtest1' Error - hugepage files for memtest1 were not deleted! Test Failed RTE>> Can you have a look? Thanks. -- David marchand
