> -----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. Should I change the state of this patch into 'Not Applicable' or other?
