msquic icon indicating copy to clipboard operation
msquic copied to clipboard

lock tx/rx/cq/fq for multi thread (multi Connection)

Open ami-GS opened this issue 1 year ago • 1 comments

Description

secnetperf 100 connections test caused system to completely hang. It requires to reboot the system. Each ring need to be locked when accessed by multi thread as it is single producer and single consumer. https://www.kernel.org/doc/html/v5.18/networking/af_xdp.html#rings

Testing

see automation. secnetperf 100 connections test can avoid the crash. locally tested several times. (There is segfault yet sometimes.)

Documentation

Is there any documentation impact for this change?

ami-GS avatar May 08 '24 23:05 ami-GS

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.30%. Comparing base (0576776) to head (e1dd136). Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4291      +/-   ##
==========================================
- Coverage   84.31%   84.30%   -0.02%     
==========================================
  Files          56       56              
  Lines       15382    15382              
==========================================
- Hits        12970    12968       -2     
- Misses       2412     2414       +2     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar May 08 '24 23:05 codecov[bot]