fixed param orders in perf_module.h. moved perf file macros to respective class defin...

diff --git a/src/network_inspectors/perf_monitor/base_tracker.cc b/src/network_inspectors/perf_monitor/base_tracker.cc
index fba2ebb574129eb25936c6b6bbc0062f6909e646..60ed0b0ddf49eff4e0668948b79ae84241b0c74c 100644 (file)
--- a/src/network_inspectors/perf_monitor/base_tracker.cc
+++ b/src/network_inspectors/perf_monitor/base_tracker.cc
@@ -27,6 +27,8 @@
 #include "managers/plugin_manager.h"
 #include "utils/stats.h"
 
+#define BASE_FILE (PERF_NAME ".csv")
+
 using namespace std;
 
 BaseTracker::BaseTracker(PerfConfig* perf) : PerfTracker(perf,

diff --git a/src/network_inspectors/perf_monitor/cpu_tracker.cc b/src/network_inspectors/perf_monitor/cpu_tracker.cc
index 41cb8ce6fa063c58b6c226ffe6be57eb2ee1c946..6d3083f28b38ffa1ccefa6ea3a067a3c57ffc57b 100644 (file)
--- a/src/network_inspectors/perf_monitor/cpu_tracker.cc
+++ b/src/network_inspectors/perf_monitor/cpu_tracker.cc
@@ -25,6 +25,8 @@
 
 #include <sys/resource.h>
 
+#define CPU_FILE (PERF_NAME "_cpu.csv")
+
 #ifdef UNIT_TEST
 #include "catch/catch.hpp"
 #endif
@@ -37,6 +39,10 @@ static inline uint64_t get_microseconds(struct timeval t)
     return (uint64_t)t.tv_sec * 1000000 + t.tv_usec;
 }
 
+
+CPUTracker::CPUTracker(PerfConfig *perf) :
+    PerfTracker(perf, perf->output == PERF_FILE ? CPU_FILE : nullptr){}
+
 void CPUTracker::get_clocks(struct rusage& usage, struct timeval& wall_time)
 {
     getrusage(RUSAGE_THREAD, &usage);

diff --git a/src/network_inspectors/perf_monitor/cpu_tracker.h b/src/network_inspectors/perf_monitor/cpu_tracker.h
index 6d416b71400ff2dd27ae21b875757eb8939653d9..336f68cbe5fd13bca7ed518873add7096a01c0b8 100644 (file)
--- a/src/network_inspectors/perf_monitor/cpu_tracker.h
+++ b/src/network_inspectors/perf_monitor/cpu_tracker.h
@@ -29,9 +29,7 @@
 class CPUTracker : public PerfTracker
 {
 public:
-    CPUTracker(PerfConfig *perf):
-        PerfTracker(perf, perf->output == PERF_FILE ? CPU_FILE : nullptr)
-    {}
+    CPUTracker(PerfConfig*);
     void reset() override;
     void process(bool) override;
 

diff --git a/src/network_inspectors/perf_monitor/event_tracker.cc b/src/network_inspectors/perf_monitor/event_tracker.cc
index 126289b625368975f60602a514ab77baaaa35f52..6d8eac981edb7d659139d7a3d13e9dd5ec7667ed 100644 (file)
--- a/src/network_inspectors/perf_monitor/event_tracker.cc
+++ b/src/network_inspectors/perf_monitor/event_tracker.cc
@@ -23,11 +23,16 @@
 #include "utils/stats.h"
 #include "utils/util.h"
 
+#define EVENT_FILE (PERF_NAME "_event.csv")
+
 THREAD_LOCAL EventTracker* perf_event;
 
 static std::string csv_header =
     "#timestamp,qualified_events,non_qualified_events\n";
 
+EventTracker::EventTracker(PerfConfig *perf) :
+    PerfTracker(perf, perf->output == PERF_FILE ? EVENT_FILE : nullptr) {}
+
 void EventTracker::reset()
 {
     event_counts.NQEvents = 0;

diff --git a/src/network_inspectors/perf_monitor/event_tracker.h b/src/network_inspectors/perf_monitor/event_tracker.h
index a701debca23f448697972756e96b0d66c1cb833c..855526cd0cbee2ccda6572ce56a33ec3da36bd40 100644 (file)
--- a/src/network_inspectors/perf_monitor/event_tracker.h
+++ b/src/network_inspectors/perf_monitor/event_tracker.h
@@ -36,8 +36,7 @@ struct PerfEventCounts
 class EventTracker : public PerfTracker
 {
 public:
-    EventTracker(PerfConfig* perf) :
-        PerfTracker(perf, perf->output == PERF_FILE ? EVENT_FILE : nullptr) { }
+    EventTracker(PerfConfig*);
     void reset() override;
     void process(bool) override;
 

diff --git a/src/network_inspectors/perf_monitor/flow_ip_tracker.cc b/src/network_inspectors/perf_monitor/flow_ip_tracker.cc
index f5051547453343ae4f5336a1ee2845698e6e907f..89bdd7a5945c745c01a83876796880c10cc766a0 100644 (file)
--- a/src/network_inspectors/perf_monitor/flow_ip_tracker.cc
+++ b/src/network_inspectors/perf_monitor/flow_ip_tracker.cc
@@ -25,6 +25,8 @@
 #include "sfip/sf_ip.h"
 #include "utils/util.h"
 
+#define FLIP_FILE (PERF_NAME "_flow_ip.csv")
+
 struct FlowStateKey
 {
     sfip_t ipA;

diff --git a/src/network_inspectors/perf_monitor/flow_tracker.cc b/src/network_inspectors/perf_monitor/flow_tracker.cc
index f05f6c5b798d4021411a7a97ef9054b298f5dcce..29a8ca670bf091b2c187afe5dbfe864469332e58 100644 (file)
--- a/src/network_inspectors/perf_monitor/flow_tracker.cc
+++ b/src/network_inspectors/perf_monitor/flow_tracker.cc
@@ -23,6 +23,8 @@
 
 #include "utils/util.h"
 
+#define FLOW_FILE (PERF_NAME "_flow.csv")
+
 THREAD_LOCAL FlowTracker* perf_flow;
 
 FlowTracker::FlowTracker(PerfConfig* perf) : PerfTracker(perf,

diff --git a/src/network_inspectors/perf_monitor/perf_module.cc b/src/network_inspectors/perf_monitor/perf_module.cc
index 1beceb9bb6f5ddb998ec34d3450d8bfd2f136dbe..5994c8a7c5430fe71d7903199a220c14cf608c06 100644 (file)
--- a/src/network_inspectors/perf_monitor/perf_module.cc
+++ b/src/network_inspectors/perf_monitor/perf_module.cc
@@ -112,6 +112,11 @@ bool PerfMonModule::set(const char*, Value& v, SnortConfig*)
         if ( v.get_bool() )
             config.perf_flags |= PERF_CPU;
     }
+    else if ( v.is("events") )
+    {
+        if ( v.get_bool() )
+            config.perf_flags |= PERF_EVENT;
+    }
     else if ( v.is("flow") )
     {
         if ( v.get_bool() )
@@ -122,11 +127,6 @@ bool PerfMonModule::set(const char*, Value& v, SnortConfig*)
         if ( v.get_bool() )
             config.perf_flags |= PERF_FLOWIP;
     }
-    else if ( v.is("events") )
-    {
-        if ( v.get_bool() )
-            config.perf_flags |= PERF_EVENT;
-    }
     else if ( v.is("packets") )
     {
         config.pkt_cnt = v.get_long();

diff --git a/src/network_inspectors/perf_monitor/perf_module.h b/src/network_inspectors/perf_monitor/perf_module.h
index f6db8f4de03823e2803f91c1a2ac33f7f6c00ed6..b066a2d0dc51995ffd4af2a35beb3a8877f2f639 100644 (file)
--- a/src/network_inspectors/perf_monitor/perf_module.h
+++ b/src/network_inspectors/perf_monitor/perf_module.h
@@ -35,12 +35,6 @@
 #define PERF_FLOWIP     0x00000020
 #define PERF_SUMMARY    0x00000040
 
-#define BASE_FILE "perf_monitor.csv"
-#define FLOW_FILE "perf_monitor_flow.csv"
-#define FLIP_FILE "perf_monitor_flow_ip.csv"
-#define EVENT_FILE "perf_monitor_event.csv"
-#define CPU_FILE "perf_monitor_cpu.csv"
-
 #define ROLLOVER_THRESH     512
 #define MAX_PERF_FILE_SIZE  UINT64_MAX
 #define MIN_PERF_FILE_SIZE  4096