Skip to content

Stats/v5#8936

Closed
victorjulien wants to merge 8 commits into
OISF:masterfrom
victorjulien:stats/v5
Closed

Stats/v5#8936
victorjulien wants to merge 8 commits into
OISF:masterfrom
victorjulien:stats/v5

Conversation

@victorjulien
Copy link
Copy Markdown
Member

Various optimizations and cleanups.

Don't update stat from loop as it's not read until after the loop.
To reduce locking overhead in the spare pool, batch returns per
100 (spare pool block size).
@codecov
Copy link
Copy Markdown

codecov Bot commented May 28, 2023

Codecov Report

Merging #8936 (8988f35) into master (afef35b) will increase coverage by 0.03%.
The diff coverage is 85.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #8936      +/-   ##
==========================================
+ Coverage   82.34%   82.38%   +0.03%     
==========================================
  Files         969      969              
  Lines      273336   273386      +50     
==========================================
+ Hits       225090   225226     +136     
+ Misses      48246    48160      -86     
Flag Coverage Δ
fuzzcorpus 64.84% <13.33%> (+0.08%) ⬆️
suricata-verify 60.46% <85.00%> (-0.02%) ⬇️
unittests 62.94% <0.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

@suricata-qa
Copy link
Copy Markdown

ERROR:

ERROR: QA failed on ASAN_TLPR1_suri.

Pipeline 14104

@suricata-qa
Copy link
Copy Markdown

Information: QA ran without warnings.

Pipeline 14107

Since the queue isn't fully processed every run, double counting
could happen.

Fix by only counting actually processed flows from the queue.
@suricata-qa
Copy link
Copy Markdown

Information: QA ran without warnings.

Pipeline 14108

@suricata-qa
Copy link
Copy Markdown

WARNING:

field baseline test %
SURI_TLPR1_stats_chk
.flow.memuse 526058088 624566000 118.73%

Pipeline 14110

In case small blocks of flows are returned, try to merge
them with existing small list head. Add full block as second
in the list as with the rest of the code.
@suricata-qa
Copy link
Copy Markdown

WARNING:

field baseline test %
SURI_TLPR1_stats_chk
.flow.memuse 526058088 596973176 113.48%

Pipeline 14113

@suricata-qa
Copy link
Copy Markdown

Information: QA ran without warnings.

Pipeline 14115

This was referenced May 30, 2023
@victorjulien
Copy link
Copy Markdown
Member Author

Merged in #8945

@victorjulien victorjulien deleted the stats/v5 branch July 17, 2023 11:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants