> -----Original Message-----
> From: Stephen Hemminger <step...@networkplumber.org>
> Sent: Saturday, May 6, 2023 11:48 PM
> To: Chaoyong He <chaoyong...@corigine.com>
> Cc: dev@dpdk.org; oss-drivers <oss-driv...@corigine.com>; Niklas
> Soderlund <niklas.soderl...@corigine.com>; Nole Zhang
> <peng.zh...@nephogine.com>
> Subject: Re: [PATCH] app/pdump: make dpdk-pdump can run multiple
> simultaneously
> 
> On Sat,  6 May 2023 16:52:06 +0800
> Chaoyong He <chaoyong...@corigine.com> wrote:
> 
> > From: Peng Zhang <peng.zh...@corigine.com>
> >
> > The pdump doesn't support start the multiple separate process for
> > different queues or ports to monitor the packets, because it apply the
> > same name for memory space. This commit will use the device_id, port
> > and queue to name the memory space and it will support this function.
> >
> > Adjust the value of RTE_MEMZONE_NAMESIZE to adapt the name length.
> >
> > Signed-off-by: Peng Zhang <peng.zh...@corigine.com>
> > Reviewed-by: Chaoyong He <chaoyong...@corigine.com>
> > Reviewed-by: Niklas Söderlund <niklas.soderl...@corigine.com>
> 
> FYI - app/dumpcap does support multiple ports already.
> Why not use that?
Yes, you are right, I will abandon this patch, thanks for your reply

Reply via email to