Re: [dpdk-dev] [PATCH v2] app/testpmd: fix testpmd exit for ctrl+d

2018-07-26 Thread Thomas Monjalon
24/07/2018 16:46, Pattan, Reshma: > + CC: Olivier and Mordechay Haimovsky. > > > Testpmd should be existed gracefully when ctrl+d is typed. > > This behaviour is not handled properly, fix this by calling > > pmd_test_exit() instead of rte_panic. > > > > Fixes: 0ad778b398c6 ("app/testpmd: rework

Re: [dpdk-dev] [PATCH v2] app/testpmd: fix testpmd exit for ctrl+d

2018-07-24 Thread Pattan, Reshma
Hi , Can you check if v2 version is working ? And it would be great if you add Tested-by: if test passes. Else, please let me know the steps to reproduce the issue. Thanks, Reshma

Re: [dpdk-dev] [PATCH v2] app/testpmd: fix testpmd exit for ctrl+d

2018-07-24 Thread Iremonger, Bernard
Hi Reshma, > -Original Message- > From: Pattan, Reshma > Sent: Tuesday, July 24, 2018 3:47 PM > To: dev@dpdk.org > Cc: Iremonger, Bernard ; > tho...@monjalon.net; Singh, Jasvinder ; > olivier.m...@6wind.com; Mordechay Haimovsky > Subject: RE: [PATCH v2] app/testpmd: fix testpmd exit for c

Re: [dpdk-dev] [PATCH v2] app/testpmd: fix testpmd exit for ctrl+d

2018-07-24 Thread Pattan, Reshma
+ CC: Olivier and Mordechay Haimovsky. > -Original Message- > From: Pattan, Reshma > Sent: Tuesday, July 24, 2018 3:17 PM > To: dev@dpdk.org > Cc: Iremonger, Bernard ; > tho...@monjalon.net; Singh, Jasvinder ; > Pattan, Reshma > Subject: [PATCH v2] app/testpmd: fix testpmd exit for ctrl+

[dpdk-dev] [PATCH v2] app/testpmd: fix testpmd exit for ctrl+d

2018-07-24 Thread Reshma Pattan
Testpmd should be existed gracefully when ctrl+d is typed. This behaviour is not handled properly, fix this by calling pmd_test_exit() instead of rte_panic. Fixes: 0ad778b398c6 ("app/testpmd: rework softnic forward mode") Reported-by: Mordechay Haimovsky Signed-off-by: Jasvinder Singh Signed-of