[PATCH] testpmd: stop pkt forwarding when quitting testpmd

[PATCH] testpmd: stop pkt forwarding when quitting testpmd

[Pablo de Lara]

When user quits testpmd, and there is traffic being forwarded,
that may produce a segmentation fault, due to ports being closed,
while they are still transmitting packets.

This patch prevents the issue from happening,
by stopping packet forwarding before closing the ports.

Signed-off-by: Pablo de Lara <pablo.de.lara.guarch-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
---
 app/test-pmd/testpmd.c |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c
index 61291be..7be96ad 100644
--- a/app/test-pmd/testpmd.c
+++ b/app/test-pmd/testpmd.c
@@ -1623,6 +1623,9 @@ pmd_test_exit(void)
 {
 	portid_t pt_id;
 
+	if (test_done == 0)
+		stop_packet_forwarding();
+
 	FOREACH_PORT(pt_id, ports) {
 		printf("Stopping port %d...", pt_id);
 		fflush(stdout);
-- 
1.7.4.1

Re: [PATCH] testpmd: stop pkt forwarding when quitting testpmd

[Thomas Monjalon]

> When user quits testpmd, and there is traffic being forwarded,
> that may produce a segmentation fault, due to ports being closed,
> while they are still transmitting packets.
> 
> This patch prevents the issue from happening,
> by stopping packet forwarding before closing the ports.
> 
> Signed-off-by: Pablo de Lara <pablo.de.lara.guarch-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>

Applied, thanks