* [Intel-wired-lan] [PATCH net v2 0/2] ixgbe: fix promiscuous mode on VF
@ 2022-04-06 9:52 ` Olivier Matz
0 siblings, 0 replies; 19+ messages in thread
From: Olivier Matz @ 2022-04-06 9:52 UTC (permalink / raw)
To: intel-wired-lan
These 2 patches fix issues related to the promiscuous mode on VF.
Comments are welcome,
Olivier
Cc: stable at vger.kernel.org
Cc: Nicolas Dichtel <nicolas.dichtel@6wind.com>
Changes since v1:
- resend with CC intel-wired-lan
- remove CC Hiroshi Shimamoto (address does not exist anymore)
Olivier Matz (2):
ixgbe: fix bcast packets Rx on VF after promisc removal
ixgbe: fix unexpected VLAN Rx in promisc mode on VF
drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
--
2.30.2
^ permalink raw reply [flat|nested] 19+ messages in thread
* [PATCH net v2 1/2] ixgbe: fix bcast packets Rx on VF after promisc removal
2022-04-06 9:52 ` [Intel-wired-lan] " Olivier Matz
@ 2022-04-06 9:52 ` Olivier Matz
-1 siblings, 0 replies; 19+ messages in thread
From: Olivier Matz @ 2022-04-06 9:52 UTC (permalink / raw)
To: netdev
Cc: Jesse Brandeburg, Tony Nguyen, David S . Miller, Jakub Kicinski,
Paolo Abeni, intel-wired-lan, Paul Menzel, stable,
Nicolas Dichtel
After a VF requested to remove the promiscuous flag on an interface, the
broadcast packets are not received anymore. This breaks some protocols
like ARP.
In ixgbe_update_vf_xcast_mode(), we should keep the IXGBE_VMOLR_BAM
bit (Broadcast Accept) on promiscuous removal.
This flag is already set by default in ixgbe_set_vmolr() on VF reset.
Fixes: 8443c1a4b192 ("ixgbe, ixgbevf: Add new mbox API xcast mode")
Cc: stable@vger.kernel.org
Cc: Nicolas Dichtel <nicolas.dichtel@6wind.com>
Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
---
drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c b/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c
index 7f11c0a8e7a9..8d108a78941b 100644
--- a/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c
+++ b/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c
@@ -1184,9 +1184,9 @@ static int ixgbe_update_vf_xcast_mode(struct ixgbe_adapter *adapter,
switch (xcast_mode) {
case IXGBEVF_XCAST_MODE_NONE:
- disable = IXGBE_VMOLR_BAM | IXGBE_VMOLR_ROMPE |
+ disable = IXGBE_VMOLR_ROMPE |
IXGBE_VMOLR_MPE | IXGBE_VMOLR_UPE | IXGBE_VMOLR_VPE;
- enable = 0;
+ enable = IXGBE_VMOLR_BAM;
break;
case IXGBEVF_XCAST_MODE_MULTI:
disable = IXGBE_VMOLR_MPE | IXGBE_VMOLR_UPE | IXGBE_VMOLR_VPE;
--
2.30.2
^ permalink raw reply related [flat|nested] 19+ messages in thread* [Intel-wired-lan] [PATCH net v2 1/2] ixgbe: fix bcast packets Rx on VF after promisc removal
@ 2022-04-06 9:52 ` Olivier Matz
0 siblings, 0 replies; 19+ messages in thread
From: Olivier Matz @ 2022-04-06 9:52 UTC (permalink / raw)
To: intel-wired-lan
After a VF requested to remove the promiscuous flag on an interface, the
broadcast packets are not received anymore. This breaks some protocols
like ARP.
In ixgbe_update_vf_xcast_mode(), we should keep the IXGBE_VMOLR_BAM
bit (Broadcast Accept) on promiscuous removal.
This flag is already set by default in ixgbe_set_vmolr() on VF reset.
Fixes: 8443c1a4b192 ("ixgbe, ixgbevf: Add new mbox API xcast mode")
Cc: stable at vger.kernel.org
Cc: Nicolas Dichtel <nicolas.dichtel@6wind.com>
Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
---
drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c b/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c
index 7f11c0a8e7a9..8d108a78941b 100644
--- a/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c
+++ b/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c
@@ -1184,9 +1184,9 @@ static int ixgbe_update_vf_xcast_mode(struct ixgbe_adapter *adapter,
switch (xcast_mode) {
case IXGBEVF_XCAST_MODE_NONE:
- disable = IXGBE_VMOLR_BAM | IXGBE_VMOLR_ROMPE |
+ disable = IXGBE_VMOLR_ROMPE |
IXGBE_VMOLR_MPE | IXGBE_VMOLR_UPE | IXGBE_VMOLR_VPE;
- enable = 0;
+ enable = IXGBE_VMOLR_BAM;
break;
case IXGBEVF_XCAST_MODE_MULTI:
disable = IXGBE_VMOLR_MPE | IXGBE_VMOLR_UPE | IXGBE_VMOLR_VPE;
--
2.30.2
^ permalink raw reply related [flat|nested] 19+ messages in thread* RE: [Intel-wired-lan] [PATCH net v2 1/2] ixgbe: fix bcast packets Rx on VF after promisc removal
2022-04-06 9:52 ` [Intel-wired-lan] " Olivier Matz
@ 2022-06-06 11:43 ` Jankowski, Konrad0
-1 siblings, 0 replies; 19+ messages in thread
From: Jankowski, Konrad0 @ 2022-06-06 11:43 UTC (permalink / raw)
To: Matz, Olivier, netdev
Cc: Paul Menzel, intel-wired-lan, stable, Jakub Kicinski,
Nicolas Dichtel, Paolo Abeni, David S . Miller
> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces@osuosl.org> On Behalf Of
> Olivier Matz
> Sent: Wednesday, April 6, 2022 11:53 AM
> To: netdev@vger.kernel.org
> Cc: Paul Menzel <pmenzel@molgen.mpg.de>; intel-wired-lan@osuosl.org;
> stable@vger.kernel.org; Jakub Kicinski <kuba@kernel.org>; Nicolas Dichtel
> <nicolas.dichtel@6wind.com>; Paolo Abeni <pabeni@redhat.com>; David S .
> Miller <davem@davemloft.net>
> Subject: [Intel-wired-lan] [PATCH net v2 1/2] ixgbe: fix bcast packets Rx on VF
> after promisc removal
>
> After a VF requested to remove the promiscuous flag on an interface, the
> broadcast packets are not received anymore. This breaks some protocols like
> ARP.
>
> In ixgbe_update_vf_xcast_mode(), we should keep the IXGBE_VMOLR_BAM
> bit (Broadcast Accept) on promiscuous removal.
>
> This flag is already set by default in ixgbe_set_vmolr() on VF reset.
>
> Fixes: 8443c1a4b192 ("ixgbe, ixgbevf: Add new mbox API xcast mode")
> Cc: stable@vger.kernel.org
> Cc: Nicolas Dichtel <nicolas.dichtel@6wind.com>
> Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
> ---
> drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c
> b/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c
> index 7f11c0a8e7a9..8d108a78941b 100644
> --- a/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c
> +++ b/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c
> @@ -1184,9 +1184,9 @@ static int ixgbe_update_vf_xcast_mode(struct
Tested-by: Konrad Jankowski <konrad0.jankowski@intel.com>
^ permalink raw reply [flat|nested] 19+ messages in thread* Re: [Intel-wired-lan] [PATCH net v2 1/2] ixgbe: fix bcast packets Rx on VF after promisc removal
@ 2022-06-06 11:43 ` Jankowski, Konrad0
0 siblings, 0 replies; 19+ messages in thread
From: Jankowski, Konrad0 @ 2022-06-06 11:43 UTC (permalink / raw)
To: Matz, Olivier, netdev
Cc: Paul Menzel, intel-wired-lan, stable, Jakub Kicinski,
Nicolas Dichtel, Paolo Abeni, David S . Miller
> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces@osuosl.org> On Behalf Of
> Olivier Matz
> Sent: Wednesday, April 6, 2022 11:53 AM
> To: netdev@vger.kernel.org
> Cc: Paul Menzel <pmenzel@molgen.mpg.de>; intel-wired-lan@osuosl.org;
> stable@vger.kernel.org; Jakub Kicinski <kuba@kernel.org>; Nicolas Dichtel
> <nicolas.dichtel@6wind.com>; Paolo Abeni <pabeni@redhat.com>; David S .
> Miller <davem@davemloft.net>
> Subject: [Intel-wired-lan] [PATCH net v2 1/2] ixgbe: fix bcast packets Rx on VF
> after promisc removal
>
> After a VF requested to remove the promiscuous flag on an interface, the
> broadcast packets are not received anymore. This breaks some protocols like
> ARP.
>
> In ixgbe_update_vf_xcast_mode(), we should keep the IXGBE_VMOLR_BAM
> bit (Broadcast Accept) on promiscuous removal.
>
> This flag is already set by default in ixgbe_set_vmolr() on VF reset.
>
> Fixes: 8443c1a4b192 ("ixgbe, ixgbevf: Add new mbox API xcast mode")
> Cc: stable@vger.kernel.org
> Cc: Nicolas Dichtel <nicolas.dichtel@6wind.com>
> Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
> ---
> drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c
> b/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c
> index 7f11c0a8e7a9..8d108a78941b 100644
> --- a/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c
> +++ b/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c
> @@ -1184,9 +1184,9 @@ static int ixgbe_update_vf_xcast_mode(struct
Tested-by: Konrad Jankowski <konrad0.jankowski@intel.com>
_______________________________________________
Intel-wired-lan mailing list
Intel-wired-lan@osuosl.org
https://lists.osuosl.org/mailman/listinfo/intel-wired-lan
^ permalink raw reply [flat|nested] 19+ messages in thread
* [PATCH net v2 2/2] ixgbe: fix unexpected VLAN Rx in promisc mode on VF
2022-04-06 9:52 ` [Intel-wired-lan] " Olivier Matz
@ 2022-04-06 9:52 ` Olivier Matz
-1 siblings, 0 replies; 19+ messages in thread
From: Olivier Matz @ 2022-04-06 9:52 UTC (permalink / raw)
To: netdev
Cc: Jesse Brandeburg, Tony Nguyen, David S . Miller, Jakub Kicinski,
Paolo Abeni, intel-wired-lan, Paul Menzel, stable,
Nicolas Dichtel
When the promiscuous mode is enabled on a VF, the IXGBE_VMOLR_VPE
bit (VLAN Promiscuous Enable) is set. This means that the VF will
receive packets whose VLAN is not the same than the VLAN of the VF.
For instance, in this situation:
┌────────┐ ┌────────┐ ┌────────┐
│ │ │ │ │ │
│ │ │ │ │ │
│ VF0├────┤VF1 VF2├────┤VF3 │
│ │ │ │ │ │
└────────┘ └────────┘ └────────┘
VM1 VM2 VM3
vf 0: vlan 1000
vf 1: vlan 1000
vf 2: vlan 1001
vf 3: vlan 1001
If we tcpdump on VF3, we see all the packets, even those transmitted
on vlan 1000.
This behavior prevents to bridge VF1 and VF2 in VM2, because it will
create a loop: packets transmitted on VF1 will be received by VF2 and
vice-versa, and bridged again through the software bridge.
This patch remove the activation of VLAN Promiscuous when a VF enables
the promiscuous mode. However, the IXGBE_VMOLR_UPE bit (Unicast
Promiscuous) is kept, so that a VF receives all packets that has the
same VLAN, whatever the destination MAC address.
Fixes: 8443c1a4b192 ("ixgbe, ixgbevf: Add new mbox API xcast mode")
Cc: stable@vger.kernel.org
Cc: Nicolas Dichtel <nicolas.dichtel@6wind.com>
Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
---
drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c b/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c
index 8d108a78941b..d4e63f0644c3 100644
--- a/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c
+++ b/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c
@@ -1208,9 +1208,9 @@ static int ixgbe_update_vf_xcast_mode(struct ixgbe_adapter *adapter,
return -EPERM;
}
- disable = 0;
+ disable = IXGBE_VMOLR_VPE;
enable = IXGBE_VMOLR_BAM | IXGBE_VMOLR_ROMPE |
- IXGBE_VMOLR_MPE | IXGBE_VMOLR_UPE | IXGBE_VMOLR_VPE;
+ IXGBE_VMOLR_MPE | IXGBE_VMOLR_UPE;
break;
default:
return -EOPNOTSUPP;
--
2.30.2
^ permalink raw reply related [flat|nested] 19+ messages in thread* [Intel-wired-lan] [PATCH net v2 2/2] ixgbe: fix unexpected VLAN Rx in promisc mode on VF
@ 2022-04-06 9:52 ` Olivier Matz
0 siblings, 0 replies; 19+ messages in thread
From: Olivier Matz @ 2022-04-06 9:52 UTC (permalink / raw)
To: intel-wired-lan
When the promiscuous mode is enabled on a VF, the IXGBE_VMOLR_VPE
bit (VLAN Promiscuous Enable) is set. This means that the VF will
receive packets whose VLAN is not the same than the VLAN of the VF.
For instance, in this situation:
?????????? ?????????? ??????????
? ? ? ? ? ?
? ? ? ? ? ?
? VF0??????VF1 VF2??????VF3 ?
? ? ? ? ? ?
?????????? ?????????? ??????????
VM1 VM2 VM3
vf 0: vlan 1000
vf 1: vlan 1000
vf 2: vlan 1001
vf 3: vlan 1001
If we tcpdump on VF3, we see all the packets, even those transmitted
on vlan 1000.
This behavior prevents to bridge VF1 and VF2 in VM2, because it will
create a loop: packets transmitted on VF1 will be received by VF2 and
vice-versa, and bridged again through the software bridge.
This patch remove the activation of VLAN Promiscuous when a VF enables
the promiscuous mode. However, the IXGBE_VMOLR_UPE bit (Unicast
Promiscuous) is kept, so that a VF receives all packets that has the
same VLAN, whatever the destination MAC address.
Fixes: 8443c1a4b192 ("ixgbe, ixgbevf: Add new mbox API xcast mode")
Cc: stable at vger.kernel.org
Cc: Nicolas Dichtel <nicolas.dichtel@6wind.com>
Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
---
drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c b/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c
index 8d108a78941b..d4e63f0644c3 100644
--- a/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c
+++ b/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c
@@ -1208,9 +1208,9 @@ static int ixgbe_update_vf_xcast_mode(struct ixgbe_adapter *adapter,
return -EPERM;
}
- disable = 0;
+ disable = IXGBE_VMOLR_VPE;
enable = IXGBE_VMOLR_BAM | IXGBE_VMOLR_ROMPE |
- IXGBE_VMOLR_MPE | IXGBE_VMOLR_UPE | IXGBE_VMOLR_VPE;
+ IXGBE_VMOLR_MPE | IXGBE_VMOLR_UPE;
break;
default:
return -EOPNOTSUPP;
--
2.30.2
^ permalink raw reply related [flat|nested] 19+ messages in thread* RE: [Intel-wired-lan] [PATCH net v2 2/2] ixgbe: fix unexpected VLAN Rx in promisc mode on VF
2022-04-06 9:52 ` [Intel-wired-lan] " Olivier Matz
@ 2022-06-06 11:45 ` Jankowski, Konrad0
-1 siblings, 0 replies; 19+ messages in thread
From: Jankowski, Konrad0 @ 2022-06-06 11:45 UTC (permalink / raw)
To: Matz, Olivier, netdev
Cc: Paul Menzel, intel-wired-lan, stable, Jakub Kicinski,
Nicolas Dichtel, Paolo Abeni, David S . Miller
> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces@osuosl.org> On Behalf Of
> Olivier Matz
> Sent: Wednesday, April 6, 2022 11:53 AM
> To: netdev@vger.kernel.org
> Cc: Paul Menzel <pmenzel@molgen.mpg.de>; intel-wired-lan@osuosl.org;
> stable@vger.kernel.org; Jakub Kicinski <kuba@kernel.org>; Nicolas Dichtel
> <nicolas.dichtel@6wind.com>; Paolo Abeni <pabeni@redhat.com>; David S .
> Miller <davem@davemloft.net>
> Subject: [Intel-wired-lan] [PATCH net v2 2/2] ixgbe: fix unexpected VLAN Rx in
> promisc mode on VF
>
> When the promiscuous mode is enabled on a VF, the IXGBE_VMOLR_VPE bit
> (VLAN Promiscuous Enable) is set. This means that the VF will receive
> packets whose VLAN is not the same than the VLAN of the VF.
>
> For instance, in this situation:
>
> ┌────────┐ ┌────────┐ ┌────────┐
> │ │ │ │ │ │
> │ │ │ │ │ │
> │ VF0├────┤VF1 VF2├────┤VF3 │
> │ │ │ │ │ │
> └────────┘ └────────┘ └────────┘
> VM1 VM2 VM3
>
> vf 0: vlan 1000
> vf 1: vlan 1000
> vf 2: vlan 1001
> vf 3: vlan 1001
>
> If we tcpdump on VF3, we see all the packets, even those transmitted on
> vlan 1000.
>
> This behavior prevents to bridge VF1 and VF2 in VM2, because it will create a
> loop: packets transmitted on VF1 will be received by VF2 and vice-versa, and
> bridged again through the software bridge.
>
> This patch remove the activation of VLAN Promiscuous when a VF enables
> the promiscuous mode. However, the IXGBE_VMOLR_UPE bit (Unicast
> Promiscuous) is kept, so that a VF receives all packets that has the same
> VLAN, whatever the destination MAC address.
>
> Fixes: 8443c1a4b192 ("ixgbe, ixgbevf: Add new mbox API xcast mode")
> Cc: stable@vger.kernel.org
> Cc: Nicolas Dichtel <nicolas.dichtel@6wind.com>
> Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
> ---
> drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c
> b/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c
> index 8d108a78941b..d4e63f0644c3 100644
> --- a/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c
> +++ b/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c
> @@ -1208,9 +1208,9 @@ static int ixgbe_update_vf_xcast_mode(struct
Tested-by: Konrad Jankowski <konrad0.jankowski@intel.com>
^ permalink raw reply [flat|nested] 19+ messages in thread* Re: [Intel-wired-lan] [PATCH net v2 2/2] ixgbe: fix unexpected VLAN Rx in promisc mode on VF
@ 2022-06-06 11:45 ` Jankowski, Konrad0
0 siblings, 0 replies; 19+ messages in thread
From: Jankowski, Konrad0 @ 2022-06-06 11:45 UTC (permalink / raw)
To: Matz, Olivier, netdev
Cc: Paul Menzel, intel-wired-lan, stable, Jakub Kicinski,
Nicolas Dichtel, Paolo Abeni, David S . Miller
> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces@osuosl.org> On Behalf Of
> Olivier Matz
> Sent: Wednesday, April 6, 2022 11:53 AM
> To: netdev@vger.kernel.org
> Cc: Paul Menzel <pmenzel@molgen.mpg.de>; intel-wired-lan@osuosl.org;
> stable@vger.kernel.org; Jakub Kicinski <kuba@kernel.org>; Nicolas Dichtel
> <nicolas.dichtel@6wind.com>; Paolo Abeni <pabeni@redhat.com>; David S .
> Miller <davem@davemloft.net>
> Subject: [Intel-wired-lan] [PATCH net v2 2/2] ixgbe: fix unexpected VLAN Rx in
> promisc mode on VF
>
> When the promiscuous mode is enabled on a VF, the IXGBE_VMOLR_VPE bit
> (VLAN Promiscuous Enable) is set. This means that the VF will receive
> packets whose VLAN is not the same than the VLAN of the VF.
>
> For instance, in this situation:
>
> ┌────────┐ ┌────────┐ ┌────────┐
> │ │ │ │ │ │
> │ │ │ │ │ │
> │ VF0├────┤VF1 VF2├────┤VF3 │
> │ │ │ │ │ │
> └────────┘ └────────┘ └────────┘
> VM1 VM2 VM3
>
> vf 0: vlan 1000
> vf 1: vlan 1000
> vf 2: vlan 1001
> vf 3: vlan 1001
>
> If we tcpdump on VF3, we see all the packets, even those transmitted on
> vlan 1000.
>
> This behavior prevents to bridge VF1 and VF2 in VM2, because it will create a
> loop: packets transmitted on VF1 will be received by VF2 and vice-versa, and
> bridged again through the software bridge.
>
> This patch remove the activation of VLAN Promiscuous when a VF enables
> the promiscuous mode. However, the IXGBE_VMOLR_UPE bit (Unicast
> Promiscuous) is kept, so that a VF receives all packets that has the same
> VLAN, whatever the destination MAC address.
>
> Fixes: 8443c1a4b192 ("ixgbe, ixgbevf: Add new mbox API xcast mode")
> Cc: stable@vger.kernel.org
> Cc: Nicolas Dichtel <nicolas.dichtel@6wind.com>
> Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
> ---
> drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c
> b/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c
> index 8d108a78941b..d4e63f0644c3 100644
> --- a/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c
> +++ b/drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c
> @@ -1208,9 +1208,9 @@ static int ixgbe_update_vf_xcast_mode(struct
Tested-by: Konrad Jankowski <konrad0.jankowski@intel.com>
_______________________________________________
Intel-wired-lan mailing list
Intel-wired-lan@osuosl.org
https://lists.osuosl.org/mailman/listinfo/intel-wired-lan
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH net v2 0/2] ixgbe: fix promiscuous mode on VF
2022-04-06 9:52 ` [Intel-wired-lan] " Olivier Matz
@ 2022-04-25 11:51 ` Olivier Matz
-1 siblings, 0 replies; 19+ messages in thread
From: Olivier Matz @ 2022-04-25 11:51 UTC (permalink / raw)
To: netdev
Cc: Jesse Brandeburg, Tony Nguyen, David S . Miller, Jakub Kicinski,
Paolo Abeni, intel-wired-lan, Paul Menzel, stable,
Nicolas Dichtel
Hi,
On Wed, Apr 06, 2022 at 11:52:50AM +0200, Olivier Matz wrote:
> These 2 patches fix issues related to the promiscuous mode on VF.
>
> Comments are welcome,
> Olivier
>
> Cc: stable@vger.kernel.org
> Cc: Nicolas Dichtel <nicolas.dichtel@6wind.com>
>
> Changes since v1:
> - resend with CC intel-wired-lan
> - remove CC Hiroshi Shimamoto (address does not exist anymore)
>
> Olivier Matz (2):
> ixgbe: fix bcast packets Rx on VF after promisc removal
> ixgbe: fix unexpected VLAN Rx in promisc mode on VF
>
> drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
Any feedback about this patchset?
Comments are welcome.
Thanks,
Olivier
^ permalink raw reply [flat|nested] 19+ messages in thread
* [Intel-wired-lan] [PATCH net v2 0/2] ixgbe: fix promiscuous mode on VF
@ 2022-04-25 11:51 ` Olivier Matz
0 siblings, 0 replies; 19+ messages in thread
From: Olivier Matz @ 2022-04-25 11:51 UTC (permalink / raw)
To: intel-wired-lan
Hi,
On Wed, Apr 06, 2022 at 11:52:50AM +0200, Olivier Matz wrote:
> These 2 patches fix issues related to the promiscuous mode on VF.
>
> Comments are welcome,
> Olivier
>
> Cc: stable at vger.kernel.org
> Cc: Nicolas Dichtel <nicolas.dichtel@6wind.com>
>
> Changes since v1:
> - resend with CC intel-wired-lan
> - remove CC Hiroshi Shimamoto (address does not exist anymore)
>
> Olivier Matz (2):
> ixgbe: fix bcast packets Rx on VF after promisc removal
> ixgbe: fix unexpected VLAN Rx in promisc mode on VF
>
> drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
Any feedback about this patchset?
Comments are welcome.
Thanks,
Olivier
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH net v2 0/2] ixgbe: fix promiscuous mode on VF
2022-04-25 11:51 ` [Intel-wired-lan] " Olivier Matz
@ 2022-05-24 7:37 ` Olivier Matz
-1 siblings, 0 replies; 19+ messages in thread
From: Olivier Matz @ 2022-05-24 7:37 UTC (permalink / raw)
To: netdev
Cc: Jesse Brandeburg, Tony Nguyen, David S . Miller, Jakub Kicinski,
Paolo Abeni, intel-wired-lan, Paul Menzel, stable,
Nicolas Dichtel
Hi,
On Mon, Apr 25, 2022 at 01:51:53PM +0200, Olivier Matz wrote:
> Hi,
>
> On Wed, Apr 06, 2022 at 11:52:50AM +0200, Olivier Matz wrote:
> > These 2 patches fix issues related to the promiscuous mode on VF.
> >
> > Comments are welcome,
> > Olivier
> >
> > Cc: stable@vger.kernel.org
> > Cc: Nicolas Dichtel <nicolas.dichtel@6wind.com>
> >
> > Changes since v1:
> > - resend with CC intel-wired-lan
> > - remove CC Hiroshi Shimamoto (address does not exist anymore)
> >
> > Olivier Matz (2):
> > ixgbe: fix bcast packets Rx on VF after promisc removal
> > ixgbe: fix unexpected VLAN Rx in promisc mode on VF
> >
> > drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c | 8 ++++----
> > 1 file changed, 4 insertions(+), 4 deletions(-)
>
> Any feedback about this patchset?
> Comments are welcome.
I didn't get feedback for this patchset until now. Am I doing things
correctly? Am I targeting the appropriate mailing lists and people?
Please let me know if I missed something.
Thanks,
Olivier
^ permalink raw reply [flat|nested] 19+ messages in thread
* [Intel-wired-lan] [PATCH net v2 0/2] ixgbe: fix promiscuous mode on VF
@ 2022-05-24 7:37 ` Olivier Matz
0 siblings, 0 replies; 19+ messages in thread
From: Olivier Matz @ 2022-05-24 7:37 UTC (permalink / raw)
To: intel-wired-lan
Hi,
On Mon, Apr 25, 2022 at 01:51:53PM +0200, Olivier Matz wrote:
> Hi,
>
> On Wed, Apr 06, 2022 at 11:52:50AM +0200, Olivier Matz wrote:
> > These 2 patches fix issues related to the promiscuous mode on VF.
> >
> > Comments are welcome,
> > Olivier
> >
> > Cc: stable at vger.kernel.org
> > Cc: Nicolas Dichtel <nicolas.dichtel@6wind.com>
> >
> > Changes since v1:
> > - resend with CC intel-wired-lan
> > - remove CC Hiroshi Shimamoto (address does not exist anymore)
> >
> > Olivier Matz (2):
> > ixgbe: fix bcast packets Rx on VF after promisc removal
> > ixgbe: fix unexpected VLAN Rx in promisc mode on VF
> >
> > drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c | 8 ++++----
> > 1 file changed, 4 insertions(+), 4 deletions(-)
>
> Any feedback about this patchset?
> Comments are welcome.
I didn't get feedback for this patchset until now. Am I doing things
correctly? Am I targeting the appropriate mailing lists and people?
Please let me know if I missed something.
Thanks,
Olivier
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH net v2 0/2] ixgbe: fix promiscuous mode on VF
2022-05-24 7:37 ` [Intel-wired-lan] " Olivier Matz
@ 2022-05-24 17:57 ` Jakub Kicinski
-1 siblings, 0 replies; 19+ messages in thread
From: Jakub Kicinski @ 2022-05-24 17:57 UTC (permalink / raw)
To: Olivier Matz, Jesse Brandeburg, Tony Nguyen
Cc: netdev, David S . Miller, Paolo Abeni, intel-wired-lan,
Paul Menzel, stable, Nicolas Dichtel, Alexander Lobakin,
Maciej Fijalkowski
On Tue, 24 May 2022 09:37:52 +0200 Olivier Matz wrote:
> On Mon, Apr 25, 2022 at 01:51:53PM +0200, Olivier Matz wrote:
> > On Wed, Apr 06, 2022 at 11:52:50AM +0200, Olivier Matz wrote:
> > > These 2 patches fix issues related to the promiscuous mode on VF.
> > >
> > > Comments are welcome,
> > > Olivier
> > >
> > > Cc: stable@vger.kernel.org
> > > Cc: Nicolas Dichtel <nicolas.dichtel@6wind.com>
> > >
> > > Changes since v1:
> > > - resend with CC intel-wired-lan
> > > - remove CC Hiroshi Shimamoto (address does not exist anymore)
> > >
> > > Olivier Matz (2):
> > > ixgbe: fix bcast packets Rx on VF after promisc removal
> > > ixgbe: fix unexpected VLAN Rx in promisc mode on VF
> > >
> > > drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c | 8 ++++----
> > > 1 file changed, 4 insertions(+), 4 deletions(-)
> >
> > Any feedback about this patchset?
> > Comments are welcome.
>
> I didn't get feedback for this patchset until now. Am I doing things
> correctly? Am I targeting the appropriate mailing lists and people?
>
> Please let me know if I missed something.
You are doing this correctly.. adding a couple more Intel folks.
^ permalink raw reply [flat|nested] 19+ messages in thread
* [Intel-wired-lan] [PATCH net v2 0/2] ixgbe: fix promiscuous mode on VF
@ 2022-05-24 17:57 ` Jakub Kicinski
0 siblings, 0 replies; 19+ messages in thread
From: Jakub Kicinski @ 2022-05-24 17:57 UTC (permalink / raw)
To: intel-wired-lan
On Tue, 24 May 2022 09:37:52 +0200 Olivier Matz wrote:
> On Mon, Apr 25, 2022 at 01:51:53PM +0200, Olivier Matz wrote:
> > On Wed, Apr 06, 2022 at 11:52:50AM +0200, Olivier Matz wrote:
> > > These 2 patches fix issues related to the promiscuous mode on VF.
> > >
> > > Comments are welcome,
> > > Olivier
> > >
> > > Cc: stable at vger.kernel.org
> > > Cc: Nicolas Dichtel <nicolas.dichtel@6wind.com>
> > >
> > > Changes since v1:
> > > - resend with CC intel-wired-lan
> > > - remove CC Hiroshi Shimamoto (address does not exist anymore)
> > >
> > > Olivier Matz (2):
> > > ixgbe: fix bcast packets Rx on VF after promisc removal
> > > ixgbe: fix unexpected VLAN Rx in promisc mode on VF
> > >
> > > drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c | 8 ++++----
> > > 1 file changed, 4 insertions(+), 4 deletions(-)
> >
> > Any feedback about this patchset?
> > Comments are welcome.
>
> I didn't get feedback for this patchset until now. Am I doing things
> correctly? Am I targeting the appropriate mailing lists and people?
>
> Please let me know if I missed something.
You are doing this correctly.. adding a couple more Intel folks.
^ permalink raw reply [flat|nested] 19+ messages in thread
* [Intel-wired-lan] [PATCH net v2 0/2] ixgbe: fix promiscuous mode on VF
2022-04-06 9:52 ` [Intel-wired-lan] " Olivier Matz
` (3 preceding siblings ...)
(?)
@ 2022-05-26 14:10 ` Piotr Skajewski
2022-05-29 19:03 ` Olivier Matz
-1 siblings, 1 reply; 19+ messages in thread
From: Piotr Skajewski @ 2022-05-26 14:10 UTC (permalink / raw)
To: intel-wired-lan, olivier.matz
Cc: netdev, davem, pabeni, pmenzel, stable, nicolas.dichtel,
alexandr.lobakin, maciej.fijalkowski, jesse.brandeburg,
anthony.l.nguyen
> On Mon, Apr 25, 2022 at 01:51:53PM +0200, Olivier Matz wrote:
> > Hi,
> >
> > On Wed, Apr 06, 2022 at 11:52:50AM +0200, Olivier Matz wrote:
> > > These 2 patches fix issues related to the promiscuous mode on VF.
> > >
> > > Comments are welcome,
> > > Olivier
> > >
> > > Cc: stable@vger.kernel.org
> > > Cc: Nicolas Dichtel <nicolas.dichtel@6wind.com>
> > >
> > > Changes since v1:
> > > - resend with CC intel-wired-lan
> > > - remove CC Hiroshi Shimamoto (address does not exist anymore)
> > >
> > > Olivier Matz (2):
> > > ixgbe: fix bcast packets Rx on VF after promisc removal
> > > ixgbe: fix unexpected VLAN Rx in promisc mode on VF
> > >
> > > drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c | 8 ++++----
> > > 1 file changed, 4 insertions(+), 4 deletions(-)
> >
> > Any feedback about this patchset?
> > Comments are welcome.
>
> I didn't get feedback for this patchset until now. Am I doing things
> correctly? Am I targeting the appropriate mailing lists and people?
>
> Please let me know if I missed something.
Hi Olivier,
Sorry for the late reply,
We had to analyze it internally and it took us some time.
After reviewing, we decided that the proposed patches could be accepted.
ACK for series.
Thanks,
Piotr
^ permalink raw reply [flat|nested] 19+ messages in thread* Re: [Intel-wired-lan] [PATCH net v2 0/2] ixgbe: fix promiscuous mode on VF
2022-05-26 14:10 ` Piotr Skajewski
@ 2022-05-29 19:03 ` Olivier Matz
0 siblings, 0 replies; 19+ messages in thread
From: Olivier Matz @ 2022-05-29 19:03 UTC (permalink / raw)
To: Piotr Skajewski
Cc: intel-wired-lan, netdev, davem, pabeni, pmenzel, stable,
nicolas.dichtel, alexandr.lobakin, maciej.fijalkowski,
jesse.brandeburg, anthony.l.nguyen
Hi Piotr,
On Thu, May 26, 2022 at 04:10:15PM +0200, Piotr Skajewski wrote:
> > On Mon, Apr 25, 2022 at 01:51:53PM +0200, Olivier Matz wrote:
> > > Hi,
> > >
> > > On Wed, Apr 06, 2022 at 11:52:50AM +0200, Olivier Matz wrote:
> > > > These 2 patches fix issues related to the promiscuous mode on VF.
> > > >
> > > > Comments are welcome,
> > > > Olivier
> > > >
> > > > Cc: stable@vger.kernel.org
> > > > Cc: Nicolas Dichtel <nicolas.dichtel@6wind.com>
> > > >
> > > > Changes since v1:
> > > > - resend with CC intel-wired-lan
> > > > - remove CC Hiroshi Shimamoto (address does not exist anymore)
> > > >
> > > > Olivier Matz (2):
> > > > ixgbe: fix bcast packets Rx on VF after promisc removal
> > > > ixgbe: fix unexpected VLAN Rx in promisc mode on VF
> > > >
> > > > drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c | 8 ++++----
> > > > 1 file changed, 4 insertions(+), 4 deletions(-)
> > >
> > > Any feedback about this patchset?
> > > Comments are welcome.
> >
> > I didn't get feedback for this patchset until now. Am I doing things
> > correctly? Am I targeting the appropriate mailing lists and people?
> >
> > Please let me know if I missed something.
>
> Hi Olivier,
>
> Sorry for the late reply,
> We had to analyze it internally and it took us some time.
> After reviewing, we decided that the proposed patches could be accepted.
>
> ACK for series.
No problem for the delay.
Thank you!
Olivier
^ permalink raw reply [flat|nested] 19+ messages in thread
* [Intel-wired-lan] [PATCH net v2 0/2] ixgbe: fix promiscuous mode on VF
@ 2022-05-29 19:03 ` Olivier Matz
0 siblings, 0 replies; 19+ messages in thread
From: Olivier Matz @ 2022-05-29 19:03 UTC (permalink / raw)
To: intel-wired-lan
Hi Piotr,
On Thu, May 26, 2022 at 04:10:15PM +0200, Piotr Skajewski wrote:
> > On Mon, Apr 25, 2022 at 01:51:53PM +0200, Olivier Matz wrote:
> > > Hi,
> > >
> > > On Wed, Apr 06, 2022 at 11:52:50AM +0200, Olivier Matz wrote:
> > > > These 2 patches fix issues related to the promiscuous mode on VF.
> > > >
> > > > Comments are welcome,
> > > > Olivier
> > > >
> > > > Cc: stable at vger.kernel.org
> > > > Cc: Nicolas Dichtel <nicolas.dichtel@6wind.com>
> > > >
> > > > Changes since v1:
> > > > - resend with CC intel-wired-lan
> > > > - remove CC Hiroshi Shimamoto (address does not exist anymore)
> > > >
> > > > Olivier Matz (2):
> > > > ixgbe: fix bcast packets Rx on VF after promisc removal
> > > > ixgbe: fix unexpected VLAN Rx in promisc mode on VF
> > > >
> > > > drivers/net/ethernet/intel/ixgbe/ixgbe_sriov.c | 8 ++++----
> > > > 1 file changed, 4 insertions(+), 4 deletions(-)
> > >
> > > Any feedback about this patchset?
> > > Comments are welcome.
> >
> > I didn't get feedback for this patchset until now. Am I doing things
> > correctly? Am I targeting the appropriate mailing lists and people?
> >
> > Please let me know if I missed something.
>
> Hi Olivier,
>
> Sorry for the late reply,
> We had to analyze it internally and it took us some time.
> After reviewing, we decided that the proposed patches could be accepted.
>
> ACK for series.
No problem for the delay.
Thank you!
Olivier
^ permalink raw reply [flat|nested] 19+ messages in thread