mbox series

[SRU,F,J:linux-bluefield,0/1] net: openvswitch: fix race on port output

Message ID 20230524024803.14253-1-witu@nvidia.com
Headers show
Series net: openvswitch: fix race on port output | expand

Message

William Tu May 24, 2023, 2:48 a.m. UTC
BugLink: https://bugs.launchpad.net/bugs/2020606

fix race on port output

* Explain the bug(s)

there is a race condition on port output

* brief explanation of fixes

We found that upstream linux has the same issue and fix
commit 066b86787fa3d97b7aefb5ac0a99a22dad2d15f8
    net: openvswitch: fix race on port output

* How to test
After configuring QoS on BF-3:
Disable all priorities
Set priorities [0, 0, 0, 0, 0, 0, 0, 0]
Remove all policies
Remove all traffic classes
Add policy (policy_name=ND_traffic, traffic_type=RDMA, priority=5, port=20000, dscp=None)
Add policy (policy_name=TCP_traffic, traffic_type=TCP, priority=4, port=26000, dscp=None)
Add traffic class two prio TC with prio_list=[5, 4], min_bw=76, algorithm=ETS
Enable QOS on adapter SLOT 5 Port 1
Restart driver (detach VFs -> restart with devcon -> attach VFs)

BF-3 sometimes stops responding. need to run it multiple times!


Felix Huettner (1):
  net: openvswitch: fix race on port output

 net/core/dev.c            | 1 +
 net/openvswitch/actions.c | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

Comments

Andrei Gherzan May 24, 2023, 11:04 a.m. UTC | #1
On 23/05/24 05:48AM, William Tu wrote:
> BugLink: https://bugs.launchpad.net/bugs/2020606
> 
> fix race on port output
> 
> * Explain the bug(s)
> 
> there is a race condition on port output
> 
> * brief explanation of fixes
> 
> We found that upstream linux has the same issue and fix
> commit 066b86787fa3d97b7aefb5ac0a99a22dad2d15f8
>     net: openvswitch: fix race on port output
> 
> * How to test
> After configuring QoS on BF-3:
> Disable all priorities
> Set priorities [0, 0, 0, 0, 0, 0, 0, 0]
> Remove all policies
> Remove all traffic classes
> Add policy (policy_name=ND_traffic, traffic_type=RDMA, priority=5, port=20000, dscp=None)
> Add policy (policy_name=TCP_traffic, traffic_type=TCP, priority=4, port=26000, dscp=None)
> Add traffic class two prio TC with prio_list=[5, 4], min_bw=76, algorithm=ETS
> Enable QOS on adapter SLOT 5 Port 1
> Restart driver (detach VFs -> restart with devcon -> attach VFs)
> 
> BF-3 sometimes stops responding. need to run it multiple times!
> 
> 
> Felix Huettner (1):
>   net: openvswitch: fix race on port output
> 
>  net/core/dev.c            | 1 +
>  net/openvswitch/actions.c | 2 +-
>  2 files changed, 2 insertions(+), 1 deletion(-)
> 
> -- 
> 2.34.1

Acked-by: Andrei Gherzan <andrei.gherzan@canonical.com>
Tim Gardner May 24, 2023, 12:39 p.m. UTC | #2
On 5/23/23 8:48 PM, William Tu wrote:
> BugLink: https://bugs.launchpad.net/bugs/2020606
> 
> fix race on port output
> 
> * Explain the bug(s)
> 
> there is a race condition on port output
> 
> * brief explanation of fixes
> 
> We found that upstream linux has the same issue and fix
> commit 066b86787fa3d97b7aefb5ac0a99a22dad2d15f8
>      net: openvswitch: fix race on port output
> 
> * How to test
> After configuring QoS on BF-3:
> Disable all priorities
> Set priorities [0, 0, 0, 0, 0, 0, 0, 0]
> Remove all policies
> Remove all traffic classes
> Add policy (policy_name=ND_traffic, traffic_type=RDMA, priority=5, port=20000, dscp=None)
> Add policy (policy_name=TCP_traffic, traffic_type=TCP, priority=4, port=26000, dscp=None)
> Add traffic class two prio TC with prio_list=[5, 4], min_bw=76, algorithm=ETS
> Enable QOS on adapter SLOT 5 Port 1
> Restart driver (detach VFs -> restart with devcon -> attach VFs)
> 
> BF-3 sometimes stops responding. need to run it multiple times!
> 
> 
> Felix Huettner (1):
>    net: openvswitch: fix race on port output
> 
>   net/core/dev.c            | 1 +
>   net/openvswitch/actions.c | 2 +-
>   2 files changed, 2 insertions(+), 1 deletion(-)
> 
Acked-by: Tim Gardner <tim.gardner@canonical.com>
William Tu May 24, 2023, 3:06 p.m. UTC | #3
Please ignore this version. Focal and Jammy have different DEBUG_NET_WARN_ON_ONCE.
I will send v2.

From: Tim Gardner <tim.gardner@canonical.com>
Date: Wednesday, May 24, 2023 at 5:39 AM
To: William Tu <witu@nvidia.com>, kernel-team@lists.ubuntu.com <kernel-team@lists.ubuntu.com>
Cc: dann.frazier@canonical.com <dann.frazier@canonical.com>, Paul Blakey <paulb@nvidia.com>, Vladimir Sokolovsky <vlad@nvidia.com>, Bodong Wang <bodong@nvidia.com>, Chris Mi <cmi@nvidia.com>
Subject: ACK: [SRU][F, J:linux-bluefield][PATCH 0/1] net: openvswitch: fix race on port output
External email: Use caution opening links or attachments


On 5/23/23 8:48 PM, William Tu wrote:
> BugLink: https://bugs.launchpad.net/bugs/2020606
>
> fix race on port output
>
> * Explain the bug(s)
>
> there is a race condition on port output
>
> * brief explanation of fixes
>
> We found that upstream linux has the same issue and fix
> commit 066b86787fa3d97b7aefb5ac0a99a22dad2d15f8
>      net: openvswitch: fix race on port output
>
> * How to test
> After configuring QoS on BF-3:
> Disable all priorities
> Set priorities [0, 0, 0, 0, 0, 0, 0, 0]
> Remove all policies
> Remove all traffic classes
> Add policy (policy_name=ND_traffic, traffic_type=RDMA, priority=5, port=20000, dscp=None)
> Add policy (policy_name=TCP_traffic, traffic_type=TCP, priority=4, port=26000, dscp=None)
> Add traffic class two prio TC with prio_list=[5, 4], min_bw=76, algorithm=ETS
> Enable QOS on adapter SLOT 5 Port 1
> Restart driver (detach VFs -> restart with devcon -> attach VFs)
>
> BF-3 sometimes stops responding. need to run it multiple times!
>
>
> Felix Huettner (1):
>    net: openvswitch: fix race on port output
>
>   net/core/dev.c            | 1 +
>   net/openvswitch/actions.c | 2 +-
>   2 files changed, 2 insertions(+), 1 deletion(-)
>
Acked-by: Tim Gardner <tim.gardner@canonical.com>
--
-----------
Tim Gardner
Canonical, Inc
Tim Gardner May 24, 2023, 3:10 p.m. UTC | #4
On 5/24/23 9:06 AM, William Tu wrote:
> Please ignore this version. Focal and Jammy have different 
> DEBUG_NET_WARN_ON_ONCE.
> 
> I will send v2.
> 
> *From: *Tim Gardner <tim.gardner@canonical.com>
> *Date: *Wednesday, May 24, 2023 at 5:39 AM
> *To: *William Tu <witu@nvidia.com>, kernel-team@lists.ubuntu.com 
> <kernel-team@lists.ubuntu.com>
> *Cc: *dann.frazier@canonical.com <dann.frazier@canonical.com>, Paul 
> Blakey <paulb@nvidia.com>, Vladimir Sokolovsky <vlad@nvidia.com>, Bodong 
> Wang <bodong@nvidia.com>, Chris Mi <cmi@nvidia.com>
> *Subject: *ACK: [SRU][F, J:linux-bluefield][PATCH 0/1] net: openvswitch: 
> fix race on port output
> 
> External email: Use caution opening links or attachments
> 
> 
> On 5/23/23 8:48 PM, William Tu wrote:
>> BugLink: https://bugs.launchpad.net/bugs/2020606 
> <https://bugs.launchpad.net/bugs/2020606>
>>
>> fix race on port output
>>
>> * Explain the bug(s)
>>
>> there is a race condition on port output
>>
>> * brief explanation of fixes
>>
>> We found that upstream linux has the same issue and fix
>> commit 066b86787fa3d97b7aefb5ac0a99a22dad2d15f8
>>      net: openvswitch: fix race on port output
>>
>> * How to test
>> After configuring QoS on BF-3:
>> Disable all priorities
>> Set priorities [0, 0, 0, 0, 0, 0, 0, 0]
>> Remove all policies
>> Remove all traffic classes
>> Add policy (policy_name=ND_traffic, traffic_type=RDMA, priority=5, port=20000, dscp=None)
>> Add policy (policy_name=TCP_traffic, traffic_type=TCP, priority=4, port=26000, dscp=None)
>> Add traffic class two prio TC with prio_list=[5, 4], min_bw=76, algorithm=ETS
>> Enable QOS on adapter SLOT 5 Port 1
>> Restart driver (detach VFs -> restart with devcon -> attach VFs)
>>
>> BF-3 sometimes stops responding. need to run it multiple times!
>>
>>
>> Felix Huettner (1):
>>    net: openvswitch: fix race on port output
>>
>>   net/core/dev.c            | 1 +
>>   net/openvswitch/actions.c | 2 +-
>>   2 files changed, 2 insertions(+), 1 deletion(-)
>>
> Acked-by: Tim Gardner <tim.gardner@canonical.com>
> --
> -----------
> Tim Gardner
> Canonical, Inc
> 

Tying off this thread.