Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pcap perf/v6 #11464

Closed
wants to merge 4 commits into from
Closed

Pcap perf/v6 #11464

wants to merge 4 commits into from

Conversation

victorjulien
Copy link
Member

victorjulien and others added 4 commits July 10, 2024 10:36
Original limit was due to a specific data structure.
Inspired by a recent Zeek blog post, this could speed up PCAP
processing by a few percent.

Ticket: OISF#7155.
Make optional through `pcap-file.buffer-size` config option.

Make sure to check through configure.

Ticket: OISF#7155.
Allows easy specification of buffer size on the commandline.

Ticket: OISF#7155.
@victorjulien victorjulien requested a review from a team as a code owner July 10, 2024 08:41
@victorjulien victorjulien mentioned this pull request Jul 10, 2024
Copy link

codecov bot commented Jul 10, 2024

Codecov Report

Attention: Patch coverage is 62.22222% with 17 lines in your changes missing coverage. Please review.

Project coverage is 82.54%. Comparing base (090079c) to head (8cf3128).

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #11464      +/-   ##
==========================================
+ Coverage   82.52%   82.54%   +0.01%     
==========================================
  Files         938      938              
  Lines      248297   248328      +31     
==========================================
+ Hits       204917   204985      +68     
+ Misses      43380    43343      -37     
Flag Coverage Δ
fuzzcorpus 60.67% <40.90%> (+0.22%) ⬆️
livemode 18.69% <15.90%> (-0.01%) ⬇️
pcap 43.81% <47.72%> (+0.03%) ⬆️
suricata-verify 61.55% <61.36%> (+0.01%) ⬆️
unittests 59.41% <6.66%> (-0.01%) ⬇️

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

@victorjulien victorjulien added this to the 8.0 milestone Jul 11, 2024
@suricata-qa
Copy link

Information: QA ran without warnings.

Pipeline 21486

@inashivb inashivb requested review from inashivb and a team and removed request for inashivb July 12, 2024 05:42
@victorjulien victorjulien mentioned this pull request Jul 12, 2024
@victorjulien
Copy link
Member Author

replaced by #11486

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.

3 participants