[v6,0/2] app/pdump: enhance to support unique cores

52262 mbox series

The patch series enhances application to support pdump capture to run
on unique cores.

Motivation
==========

DPDK pdump capture tool currently runs as secondary on the default core 0.
For all --pdump, core 0 iterates and capture packets. This leads to drops
and delay in the capture. This introduces skew in result and performance
bottleneck.

In order to allow --pdump to unique cores, the pdump application has to be
support multi-core in EAL arguments and each --pdump has to be mapped to
unique cores.

Status
======

 - Default c_flag option is removed.
 - with option --multi, the application can run on unique cores.

Change Log:
===========

V6:
 - change MACRO value - David Marchand
 - use MACRO in usage function - David Marchand
 - correct spelling in documentation - Reshma Pathan

V5:
 - Re-use MACRO and replace strncmp - David Marchand

V4:
 - spelling correction - Reshma Pathan
 - remove c_falg - Reshma Pathan

V3:
 - correct the parse_usage - Vipin Varghese
 - add change log - Vipin Varghese

V2:
 - Replace option '-c' to '-l' - Keith Wiles

Vipin Varghese (2):
  app/pdump: remove core restriction
  app/pdump: enhance to support multi-core capture

 app/pdump/main.c           | 113 +++++++++++++++++++++++++++----------
 doc/guides/tools/pdump.rst |   8 ++-
 2 files changed, 91 insertions(+), 30 deletions(-)

Future:
=======
 - Ehance multi mode, allow master to stop --pdump for desired file size.
 - Ehance multi mode, allow to suggest NUMA aware file locations.
 - Stop unique pdump.

04/04/2019 10:55, Vipin Varghese:
> The patch series enhances application to support pdump capture to run
> on unique cores.
> 
> Motivation
> ==========
> 
> DPDK pdump capture tool currently runs as secondary on the default core 0.
> For all --pdump, core 0 iterates and capture packets. This leads to drops
> and delay in the capture. This introduces skew in result and performance
> bottleneck.
> 
> In order to allow --pdump to unique cores, the pdump application has to be
> support multi-core in EAL arguments and each --pdump has to be mapped to
> unique cores.

Applied, thanks