* [PATCH net] ice: fix enabling RX VLAN filtering
@ 2024-03-25 20:19 Petr Oros
2024-03-25 21:19 ` Michal Swiatkowski
0 siblings, 1 reply; 3+ messages in thread
From: Petr Oros @ 2024-03-25 20:19 UTC (permalink / raw)
To: netdev
Cc: jesse.brandeburg, anthony.l.nguyen, davem, edumazet, kuba, pabeni,
horms, michal.swiatkowski, wojciech.drewek, intel-wired-lan,
linux-kernel
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@redhat.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;
}
--
2.43.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH net] ice: fix enabling RX VLAN filtering
2024-03-25 20:19 [PATCH net] ice: fix enabling RX VLAN filtering Petr Oros
@ 2024-03-25 21:19 ` Michal Swiatkowski
2024-04-03 9:43 ` [Intel-wired-lan] " Romanowski, Rafal
0 siblings, 1 reply; 3+ messages in thread
From: Michal Swiatkowski @ 2024-03-25 21:19 UTC (permalink / raw)
To: Petr Oros
Cc: netdev, jesse.brandeburg, anthony.l.nguyen, davem, edumazet, kuba,
pabeni, horms, wojciech.drewek, intel-wired-lan, linux-kernel
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@redhat.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@linux.intel.com>
> --
> 2.43.2
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* RE: [Intel-wired-lan] [PATCH net] ice: fix enabling RX VLAN filtering
2024-03-25 21:19 ` Michal Swiatkowski
@ 2024-04-03 9:43 ` Romanowski, Rafal
0 siblings, 0 replies; 3+ messages in thread
From: Romanowski, Rafal @ 2024-04-03 9:43 UTC (permalink / raw)
To: intel-wired-lan@lists.osuosl.org
Cc: Drewek, Wojciech, netdev@vger.kernel.org, kuba@kernel.org,
pabeni@redhat.com, davem@davemloft.net, poros, Michal Swiatkowski,
linux-kernel@vger.kernel.org, edumazet@google.com,
Nguyen, Anthony L, horms@kernel.org
> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces@osuosl.org> On Behalf Of
> Michal Swiatkowski
> Sent: Monday, March 25, 2024 10:20 PM
> To: poros <poros@redhat.com>
> Cc: Drewek, Wojciech <wojciech.drewek@intel.com>;
> netdev@vger.kernel.org; linux-kernel@vger.kernel.org;
> edumazet@google.com; Nguyen, Anthony L <anthony.l.nguyen@intel.com>;
> horms@kernel.org; intel-wired-lan@lists.osuosl.org; kuba@kernel.org;
> pabeni@redhat.com; davem@davemloft.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@redhat.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@linux.intel.com>
>
> > --
> > 2.43.2
> >
Tested-by: Rafal Romanowski <rafal.romanowski@intel.com>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-04-03 9:43 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-03-25 20:19 [PATCH net] ice: fix enabling RX VLAN filtering Petr Oros
2024-03-25 21:19 ` Michal Swiatkowski
2024-04-03 9:43 ` [Intel-wired-lan] " Romanowski, Rafal
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).