When the secondary process testpmd application running any fwd_engine
and after that primary has exited will cause a crash.
This patch forces secondary process forwarding lcores
in fwd_engine to stop before the primary process exits.
Fixes: a550baf24af90f ("app/testpmd: support multi-process")
Cc: sta...@dpdk.org
Signed-off-by: Khadem Ullah <14pwcse1...@uetpeshawar.edu.pk>
Signed-off-by: Stephen Hemminger <step...@networkplumber.org>
---
app/test-pmd/testpmd.c | 19 ++++++++++++++++---
lib/eal/common/hotplug_mp.c | 3 +++
lib/eal/common/hotplug_mp.h | 2 ++
3 files changed, 21 insertions(+), 3 deletions(-)
diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c
index 0f58a31eb5..eda3a0264d 100644
--- a/app/test-pmd/testpmd.c
+++ b/app/test-pmd/testpmd.c
@@ -11,6 +11,7 @@
#include <fcntl.h>
#ifndef RTE_EXEC_ENV_WINDOWS
#include <sys/mman.h>
+#include "hotplug_mp.h"
#endif
#include <sys/types.h>
#include <errno.h>
@@ -103,7 +104,7 @@ uint16_t verbose_level = 0; /**< Silent by default. */
int testpmd_logtype; /**< Log type for testpmd logs */
/* Maximum delay for exiting after primary process. */
-#define MONITOR_INTERVAL (500 * 1000)
+#define MONITOR_INTERVAL (500)
/* use main core for command line ? */
uint8_t interactive = 0;
@@ -2563,7 +2564,8 @@ stop_packet_forwarding(void)
for (lc_id = 0; lc_id < cur_fwd_config.nb_fwd_lcores; lc_id++)
fwd_lcores[lc_id]->stopped = 1;
printf("\nWaiting for lcores to finish...\n");
- rte_eal_mp_wait_lcore();
+ if (rte_eal_process_type() == RTE_PROC_PRIMARY)
+ rte_eal_mp_wait_lcore();
port_fwd_end = cur_fwd_config.fwd_eng->port_fwd_end;
if (port_fwd_end != NULL) {
for (i = 0; i < cur_fwd_config.nb_fwd_ports; i++) {
@@ -3634,6 +3636,11 @@ pmd_test_exit(void)
printf("\nStopping port %d...\n", pt_id);
fflush(stdout);
stop_port(pt_id);
+ printf("Stopping secondary process...\n");
+ struct eal_dev_mp_req req;
+ memset(&req, 0, sizeof(req));
+ req.t = EAL_DEV_REQ_TYPE_STOP;
+ eal_dev_hotplug_request_to_secondary(&req);
}
RTE_ETH_FOREACH_DEV(pt_id) {
printf("\nShutting down port %d...\n", pt_id);
@@ -4371,7 +4378,13 @@ static void
monitor_primary(void *arg __rte_unused)
{
if (rte_eal_primary_proc_alive(NULL)) {
- rte_eal_alarm_set(MONITOR_INTERVAL, monitor_primary, NULL);
+ if (f_exit) {
+ stop_packet_forwarding();
+ fprintf(stderr, "\nPrimary process is no longer active,
exiting...\n");
+ exit(EXIT_FAILURE);
+ } else {
+ rte_eal_alarm_set(MONITOR_INTERVAL, monitor_primary,
NULL);
+ }
} else {
/*
* If primary process exits, then all the device information
diff --git a/lib/eal/common/hotplug_mp.c b/lib/eal/common/hotplug_mp.c
index 17089ca3db..538e6eaf4e 100644
--- a/lib/eal/common/hotplug_mp.c
+++ b/lib/eal/common/hotplug_mp.c
@@ -239,6 +239,9 @@ static void __handle_primary_request(void *param)
memset(&mp_resp, 0, sizeof(mp_resp));
switch (req->t) {
+ case EAL_DEV_REQ_TYPE_STOP:
+ f_exit = 1;
+ break;
case EAL_DEV_REQ_TYPE_ATTACH:
case EAL_DEV_REQ_TYPE_DETACH_ROLLBACK:
ret = local_dev_probe(req->devargs, &dev);
diff --git a/lib/eal/common/hotplug_mp.h b/lib/eal/common/hotplug_mp.h
index 7221284286..b0755bcf51 100644
--- a/lib/eal/common/hotplug_mp.h
+++ b/lib/eal/common/hotplug_mp.h
@@ -9,6 +9,7 @@
#define EAL_DEV_MP_ACTION_REQUEST "eal_dev_mp_request"
#define EAL_DEV_MP_ACTION_RESPONSE "eal_dev_mp_response"
+volatile uint8_t f_exit;
#define EAL_DEV_MP_DEV_NAME_MAX_LEN RTE_DEV_NAME_MAX_LEN
#define EAL_DEV_MP_BUS_NAME_MAX_LEN 32
@@ -19,6 +20,7 @@ enum eal_dev_req_type {
EAL_DEV_REQ_TYPE_DETACH,
EAL_DEV_REQ_TYPE_ATTACH_ROLLBACK,
EAL_DEV_REQ_TYPE_DETACH_ROLLBACK,
+ EAL_DEV_REQ_TYPE_STOP,
};
struct eal_dev_mp_req {
--
2.27.0
