Hi, Ferruh, > -----Original Message----- > From: Yigit, Ferruh > Sent: Thursday, December 08, 2016 6:43 PM > To: Chen, Jing D <jing.d.c...@intel.com>; Lu, Wenzhuo <wenzhuo...@intel.com>; > dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2 15/32] net/i40e: add VF vlan strip func > > On 12/8/2016 9:10 AM, Chen, Jing D wrote: > > HI, Ferruh, > > > > Best Regards, > > Mark > > > > > >> -----Original Message----- > >> From: Yigit, Ferruh > >> Sent: Wednesday, December 07, 2016 10:18 PM > >> To: Lu, Wenzhuo <wenzhuo...@intel.com>; dev@dpdk.org > >> Cc: Chen, Jing D <jing.d.c...@intel.com> > >> Subject: Re: [dpdk-dev] [PATCH v2 15/32] net/i40e: add VF vlan strip func > >> > >> On 12/7/2016 3:31 AM, Wenzhuo Lu wrote: > >>> Add a function to configure vlan strip enable/disable for specific > >>> SRIOV VF device. > >>> > >>> Signed-off-by: Chen Jing D(Mark) <jing.d.c...@intel.com> > >>> --- > >> > >> <...> > >> > >>> + > >>> +/* Set vlan strip on/off for specific VF from host */ > >>> +int > >>> +rte_pmd_i40e_set_vf_vlan_stripq(uint8_t port, uint16_t vf_id, uint8_t on) > >>> +{ > >>> + struct rte_eth_dev *dev; > >>> + struct i40e_pf *pf; > >>> + struct i40e_vsi *vsi; > >>> + > >>> + RTE_ETH_VALID_PORTID_OR_ERR_RET(port, -ENODEV); > >>> + > >>> + dev = &rte_eth_devices[port]; > >>> + pf = I40E_DEV_PRIVATE_TO_PF(dev->data->dev_private); > >>> + > >>> + if (vf_id > pf->vf_num - 1 || !pf->vfs) { > >>> + PMD_DRV_LOG(ERR, "Invalid argument."); > >>> + return -EINVAL; > >>> + } > >>> + > >>> + vsi = pf->vfs[vf_id].vsi; > >>> + > >>> + if (vsi) > >>> + return i40e_vsi_config_vlan_stripping(vsi, !!on); > >>> + else > >> > >> if vd_if is valid, can vsi be NULL? If so this check may be required in > >> some prev patches too. > > > > It's a little impossible. This sanity check just make the code stronger. > > > > If it is impossible, do you agree to remove this? And if this can be > possible we must update other patches, almost all other patches assume > this can't be NULL.
I'll recommend other patches to add it, too. The reason is we can't image if there is some code change have impact in future, the necessary sanity check in slow patch make code stronger.