Project

General

Profile

Actions

Feature #789

closed
EL EL

Clean-up start and stop code

Feature #789: Clean-up start and stop code

Added by Eric Leblond about 13 years ago. Updated over 12 years ago.

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

Description

As seen during the work on unix socket, the start and stop code of suricata has become messy and needs to be improved. In particular, suricata.c needs to be clean up.


Subtasks 1 (0 open1 closed)

Feature #894: clean up outputClosedEric LeblondActions

VJ Updated by Victor Julien about 13 years ago Actions #1

I think one of the first things we should do is determine the runmode and determine how many threads of each type we are going to be using. For example, in oisf-devel we're talking about per-worker flow engines. This implies that at initialization of the flow engine(s), we need to know the runmode and number of threads. Unless we move the flow engine init to the threads init. Anyhow, something to consider.

EL Updated by Eric Leblond about 13 years ago Actions #2

I've started to look into it. Regarding two separate aspects: unix socket running mode and the flow issue you mentioned, I think we should migrate a lot of init and cleaning code to the running mode code. This will allow to simplify the unix socket related code and have full flexibility regarding the Flow manager.

If no objection, I continue to look into this.

EL Updated by Eric Leblond almost 13 years ago Actions #3

  • % Done changed from 0 to 50

EL Updated by Eric Leblond almost 13 years ago Actions #4

The PR https://github.com/inliniac/suricata/pull/415 is implementing some factorization...

PM Updated by Peter Manev almost 13 years ago Actions #5

Would that address somehow Bug #787 ?

EL Updated by Eric Leblond almost 13 years ago Actions #6

I don't think, packet acquisition is not changed.

VJ Updated by Victor Julien almost 13 years ago Actions #7

  • Target version changed from 2.0beta1 to 2.0beta2

EL Updated by Eric Leblond over 12 years ago Actions #8

  • Status changed from New to Closed

Closing it for now. Work has been merged.

Actions

Also available in: PDF Atom