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

[Tim Gardner]

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 at canonical.com>
> *Date: *Wednesday, May 24, 2023 at 5:39 AM
> *To: *William Tu <witu at nvidia.com>, kernel-team at lists.ubuntu.com 
> <kernel-team at lists.ubuntu.com>
> *Cc: *dann.frazier at canonical.com <dann.frazier at canonical.com>, Paul 
> Blakey <paulb at nvidia.com>, Vladimir Sokolovsky <vlad at nvidia.com>, Bodong 
> Wang <bodong at nvidia.com>, Chris Mi <cmi at 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 at canonical.com>
> --
> -----------
> Tim Gardner
> Canonical, Inc
> 

Tying off this thread.
-- 
-----------
Tim Gardner
Canonical, Inc