netdev - RE: [Intel-wired-lan] [PATCH net] ice: fix enabling RX VLAN filtering

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <SJ0PR11MB5865AD8922F34A36327CC7248F3D2@SJ0PR11MB5865.namprd11.prod.outlook.com>
Date: Wed, 3 Apr 2024 09:43:44 +0000
From: "Romanowski, Rafal" <rafal.romanowski@...el.com>
To: "intel-wired-lan@...ts.osuosl.org" <intel-wired-lan@...ts.osuosl.org>
CC: "Drewek, Wojciech" <wojciech.drewek@...el.com>, "netdev@...r.kernel.org"
	<netdev@...r.kernel.org>, "kuba@...nel.org" <kuba@...nel.org>,
	"pabeni@...hat.com" <pabeni@...hat.com>, "davem@...emloft.net"
	<davem@...emloft.net>, poros <poros@...hat.com>, Michal Swiatkowski
	<michal.swiatkowski@...ux.intel.com>, "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>, "edumazet@...gle.com" <edumazet@...gle.com>,
	"Nguyen, Anthony L" <anthony.l.nguyen@...el.com>, "horms@...nel.org"
	<horms@...nel.org>
Subject: RE: [Intel-wired-lan] [PATCH net] ice: fix enabling RX VLAN filtering

> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces@...osl.org> On Behalf Of
> Michal Swiatkowski
> Sent: Monday, March 25, 2024 10:20 PM
> To: poros <poros@...hat.com>
> Cc: Drewek, Wojciech <wojciech.drewek@...el.com>;
> netdev@...r.kernel.org; linux-kernel@...r.kernel.org;
> edumazet@...gle.com; Nguyen, Anthony L <anthony.l.nguyen@...el.com>;
> horms@...nel.org; intel-wired-lan@...ts.osuosl.org; kuba@...nel.org;
> pabeni@...hat.com; davem@...emloft.net
> Subject: Re: [Intel-wired-lan] [PATCH net] ice: fix enabling RX VLAN filtering
> 
> On Mon, Mar 25, 2024 at 09:19:01PM +0100, Petr Oros wrote:
> > ice_port_vlan_on/off() was introduced in commit 2946204b3fa8 ("ice:
> > implement bridge port vlan"). But ice_port_vlan_on() incorrectly
> > assigns ena_rx_filtering to inner_vlan_ops in DVM mode.
> > This causes an error when rx_filtering cannot be enabled in legacy mode.
> >
> > Reproducer:
> >  echo 1 > /sys/class/net/$PF/device/sriov_numvfs
> >  ip link set $PF vf 0 spoofchk off trust on vlan 3
> > dmesg:
> >  ice 0000:41:00.0: failed to enable Rx VLAN filtering for VF 0 VSI 9
> > during VF rebuild, error -95
> >
> > Fixes: 2946204b3fa8 ("ice: implement bridge port vlan")
> > Signed-off-by: Petr Oros <poros@...hat.com>
> > ---
> >  .../ethernet/intel/ice/ice_vf_vsi_vlan_ops.c   | 18 ++++++++----------
> >  1 file changed, 8 insertions(+), 10 deletions(-)
> >
> > diff --git a/drivers/net/ethernet/intel/ice/ice_vf_vsi_vlan_ops.c
> > b/drivers/net/ethernet/intel/ice/ice_vf_vsi_vlan_ops.c
> > index 80dc4bcdd3a41c..b3e1bdcb80f84d 100644
> > --- a/drivers/net/ethernet/intel/ice/ice_vf_vsi_vlan_ops.c
> > +++ b/drivers/net/ethernet/intel/ice/ice_vf_vsi_vlan_ops.c
> > @@ -26,24 +26,22 @@ static void ice_port_vlan_on(struct ice_vsi *vsi)
> >  	struct ice_vsi_vlan_ops *vlan_ops;
> >  	struct ice_pf *pf = vsi->back;
> >
> > -	if (ice_is_dvm_ena(&pf->hw)) {
> > -		vlan_ops = &vsi->outer_vlan_ops;
> > -
> > -		/* setup outer VLAN ops */
> > -		vlan_ops->set_port_vlan = ice_vsi_set_outer_port_vlan;
> > -		vlan_ops->clear_port_vlan = ice_vsi_clear_outer_port_vlan;
> > +	/* setup inner VLAN ops */
> > +	vlan_ops = &vsi->inner_vlan_ops;
> >
> > -		/* setup inner VLAN ops */
> > -		vlan_ops = &vsi->inner_vlan_ops;
> > +	if (ice_is_dvm_ena(&pf->hw)) {
> >  		vlan_ops->add_vlan = noop_vlan_arg;
> >  		vlan_ops->del_vlan = noop_vlan_arg;
> >  		vlan_ops->ena_stripping = ice_vsi_ena_inner_stripping;
> >  		vlan_ops->dis_stripping = ice_vsi_dis_inner_stripping;
> >  		vlan_ops->ena_insertion = ice_vsi_ena_inner_insertion;
> >  		vlan_ops->dis_insertion = ice_vsi_dis_inner_insertion;
> > -	} else {
> > -		vlan_ops = &vsi->inner_vlan_ops;
> >
> > +		/* setup outer VLAN ops */
> > +		vlan_ops = &vsi->outer_vlan_ops;
> > +		vlan_ops->set_port_vlan = ice_vsi_set_outer_port_vlan;
> > +		vlan_ops->clear_port_vlan = ice_vsi_clear_outer_port_vlan;
> > +	} else {
> >  		vlan_ops->set_port_vlan = ice_vsi_set_inner_port_vlan;
> >  		vlan_ops->clear_port_vlan = ice_vsi_clear_inner_port_vlan;
> >  	}
> 
> Thanks for fixing it.
> Reviewed-by: Michal Swiatkowski <michal.swiatkowski@...ux.intel.com>
> 
> > --
> > 2.43.2
> >


Tested-by: Rafal Romanowski <rafal.romanowski@...el.com>