Hi Ferruh,
> -----Original Message----- > From: Yigit, Ferruh > Sent: Wednesday, December 7, 2016 11:49 PM > To: Lu, Wenzhuo; dev@dpdk.org > Cc: Chen, Jing D > Subject: Re: [dpdk-dev] [PATCH v2 32/32] app/testpmd: fix invalid port ID > > On 12/7/2016 3:32 AM, Wenzhuo Lu wrote: > > Some CLIs don't check the input port ID, it may cause segmentation > > fault (core dumped). > > Are these functions (that we are adding extra check) added with this > patchset? If > so why not add these checks where function implemented. > > If these functions are already implemented before this patchset, this patch > can > be standalone patch, instead of being part of this patchset, and can be before > this patchset so that it can be easily backported to stable trees. This patch is for the existing code. I think you're right. I'll remove it from this patch set and send an isolate patch for it. > > > > > Fixes: 425781ff5afe ("app/testpmd: add ixgbe VF management") > > > > Signed-off-by: Wenzhuo Lu <wenzhuo...@intel.com> > > Signed-off-by: Chen Jing D(Mark) <jing.d.c...@intel.com> > > --- > > <...>