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

Smb vecdeque 5753 v4 #8359

Closed
wants to merge 2 commits into from
Closed

Conversation

catenacyber
Copy link
Contributor

Link to redmine ticket:
https://redmine.openinfosecfoundation.org/issues/5753

Describe changes:

  • smb : use VecDeque
  • smb : have a max tx config parameter

So that transactions cleaning happens faster

Replaces #8357 with better commit message

Allows for more efficient removal from front of the list.

Ticket: OISF#5753
@suricata-qa
Copy link

WARNING:

field baseline test %
SURI_TLPW1_stats_chk
.tcp.rst 128237 104031 81.12%
SURI_TLPR1_stats_chk
.flow.memuse 565135168 824802048 145.95%
TREX_GENERIC_stats_chk
.capture.kernel_drops 0 499 0.00

Pipeline 11429

@victorjulien victorjulien mentioned this pull request Jan 10, 2023
@suricata-qa
Copy link

Information:

field baseline test %
TREX_GENERIC_stats_chk
.capture.kernel_drops 0 541125 0.09

Pipeline 11434

@victorjulien
Copy link
Member

Merged in #8360, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants