Project

General

Profile

Actions

Bug #4578

closed

perf shows excessive time in IPOnlyMatchPacket

Added by Jeff Lucovsky over 3 years ago. Updated over 1 year ago.

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

Description

With Suricata 5.0.x, perf shows excessive time spent in IPOnlyMatchPacket when there are rules containing IP addresses as filters. With roughly 400 rules and 15-20K IP addresses, perf showed a majority of time spent in IPOnlyMatchPacket.

Granted, a dataset is a better way to handle large IP address counts used within a rule but 5.0.x support didn't contain official dataset support.

Perhaps a warning or other indicator could be displayed to help users understand the effects of rules with large IP addr counts?


Files

iponlymatchpacketdiff.png (83.4 KB) iponlymatchpacketdiff.png Andreas Herz, 05/10/2023 01:13 PM

Subtasks 2 (0 open2 closed)

Bug #5997: perf shows excessive time in IPOnlyMatchPacket (6.0.x backport)ClosedJeff LucovskyActions
Bug #6005: perf shows excessive time in IPOnlyMatchPacket (6.0.x backport)RejectedJustin AzoffActions
Actions #1

Updated by Jeff Lucovsky almost 2 years ago

  • Status changed from New to In Review
Actions #2

Updated by Victor Julien almost 2 years ago

  • Assignee set to Justin Azoff
  • Target version set to 7.0.0-rc2
Actions #3

Updated by Victor Julien over 1 year ago

  • Status changed from In Review to Closed
  • Label Needs backport to 6.0 added
Actions #4

Updated by Victor Julien over 1 year ago

  • Status changed from Closed to Resolved
Actions #5

Updated by OISF Ticketbot over 1 year ago

  • Subtask #5997 added
Actions #6

Updated by OISF Ticketbot over 1 year ago

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

Updated by Jeff Lucovsky over 1 year ago

  • Label Needs backport to 6.0 added
Actions #8

Updated by OISF Ticketbot over 1 year ago

  • Subtask #6005 added
Actions #9

Updated by OISF Ticketbot over 1 year ago

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

Updated by Victor Julien over 1 year ago

  • Status changed from Resolved to Closed
Actions #11

Updated by Andreas Herz over 1 year ago


Added a perf top output to see the impact of the patch

Actions

Also available in: Atom PDF