Project

General

Profile

Actions

Bug #6414

closed

detect-engine/port: recursive DetectPortInsert calls are expensive

Added by Shivani Bhardwaj about 1 year ago. Updated 7 months ago.

Status:
Closed
Priority:
Normal
Target version:
Affected Versions:
Effort:
Difficulty:
Label:

Description

Problem
It seems that for certain kinds of rules, the recursive calls to DetectPortInsert can be very expensive.
There has been a todo to get rid of the recursive calls since a long time that needs to be addressed now.
The issue can be observed for large rulesets especially containing a mix of drop tls rules and others.
One noteworthy thing is that these rules loaded separately end up consuming much lesser time.

Useful info
Attached is one scenario where the flamegraph shows heavy frequenting of this fn.


Files

perf.svg (679 KB) perf.svg Shivani Bhardwaj, 10/19/2023 03:57 PM

Subtasks 3 (0 open3 closed)

Bug #6431: detect-engine/port: recursive DetectPortInsert calls are expensive (6.0.x backport)RejectedShivani BhardwajActions
Bug #6520: detect-engine/port: recursive DetectPortInsert calls are expensive (7.0.x backport)ClosedShivani BhardwajActions
Bug #6639: detect-engine/port: recursive DetectPortInsert calls are expensive (7.0.x backport)RejectedShivani BhardwajActions

Related issues 2 (0 open2 closed)

Related to Suricata - Optimization #6795: detect/port: PortGroupWhitelist fn takes a lot of processing timeClosedVictor JulienActions
Related to Suricata - Optimization #6792: detect/port: port grouping is quite slow in worst casesClosedShivani BhardwajActions
Actions #1

Updated by Shivani Bhardwaj about 1 year ago

  • Status changed from New to Assigned
  • Assignee changed from OISF Dev to Shivani Bhardwaj
  • Target version changed from TBD to 7.0.3
Actions #2

Updated by OISF Ticketbot about 1 year ago

  • Subtask #6431 added
Actions #3

Updated by OISF Ticketbot about 1 year ago

  • Label deleted (Needs backport to 6.0)
Actions #4

Updated by Victor Julien about 1 year ago

  • Target version changed from 7.0.3 to 8.0.0-beta1
  • Label Needs backport to 7.0 added
Actions #5

Updated by OISF Ticketbot about 1 year ago

  • Subtask #6520 added
Actions #6

Updated by OISF Ticketbot about 1 year ago

  • Label deleted (Needs backport to 7.0)
Actions #7

Updated by Shivani Bhardwaj about 1 year ago

  • Label Needs backport to 7.0 added
Actions #8

Updated by OISF Ticketbot about 1 year ago

  • Subtask #6639 added
Actions #9

Updated by OISF Ticketbot about 1 year ago

  • Label deleted (Needs backport to 7.0)
Actions #10

Updated by Shivani Bhardwaj about 1 year ago

  • Label Needs backport to 7.0 added
Actions #11

Updated by OISF Ticketbot about 1 year ago

  • Label deleted (Needs backport to 7.0)
Actions #12

Updated by Shivani Bhardwaj 10 months ago

  • Related to Optimization #6795: detect/port: PortGroupWhitelist fn takes a lot of processing time added
  • Related to Optimization #6792: detect/port: port grouping is quite slow in worst cases added
Actions #13

Updated by Shivani Bhardwaj 10 months ago

  • Status changed from Assigned to In Review
Actions #14

Updated by Shivani Bhardwaj 10 months ago

  • Status changed from In Review to Resolved
Actions #15

Updated by Shivani Bhardwaj 7 months ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF