defrag: don't use message for repetitive error

When nothing can be fetch from the pool, this can repeat frequently.
Thus displaying a message in the log will not help. This patch
uses a counter instead of a log message. As this is a sort of memcap
this is conformed to what is done for other issues of the same type.

diff --git a/src/decode.c b/src/decode.c
index 4bdd287587891b0defa014f910b8601968650e13..a6b5548ff92bcf3348b21d2e99e1a5aa0e864ebc 100644 (file)
--- a/src/decode.c
+++ b/src/decode.c
@@ -357,6 +357,9 @@ void DecodeRegisterPerfCounters(DecodeThreadVars *dtv, ThreadVars *tv)
     dtv->counter_defrag_ipv6_timeouts =
         SCPerfTVRegisterCounter("defrag.ipv6.timeouts", tv,
             SC_PERF_TYPE_UINT64, "NULL");
+    dtv->counter_defrag_max_hit =
+        SCPerfTVRegisterCounter("defrag.max_frag_hits", tv,
+            SC_PERF_TYPE_UINT64, "NULL");
 
     tv->sc_perf_pca = SCPerfGetAllCountersArray(&tv->sc_perf_pctx);
     SCPerfAddToClubbedTMTable(tv->name, &tv->sc_perf_pctx);

diff --git a/src/decode.h b/src/decode.h
index eb62bcd341def08db6c87be1e1823122eb909ad7..035f25c474f7fa859a1f466dd391d55e86544c71 100644 (file)
--- a/src/decode.h
+++ b/src/decode.h
@@ -564,6 +564,7 @@ typedef struct DecodeThreadVars_
     uint16_t counter_defrag_ipv6_fragments;
     uint16_t counter_defrag_ipv6_reassembled;
     uint16_t counter_defrag_ipv6_timeouts;
+    uint16_t counter_defrag_max_hit;
 } DecodeThreadVars;
 
 /**

diff --git a/src/defrag.c b/src/defrag.c
index 99a320a8489ae2211eb2020a933cbdadd2d2d393..729f884407b893f668293e3df0daf14bb4dfd080 100644 (file)
--- a/src/defrag.c
+++ b/src/defrag.c
@@ -1133,8 +1133,8 @@ DefragGetTracker(ThreadVars *tv, DecodeThreadVars *dtv, DefragContext *dc,
         }
         SCMutexUnlock(&dc->tracker_pool_lock);
         if (tracker == NULL) {
-            /* Report memory error - actually a pool allocation error. */
-            SCLogError(SC_ERR_MEM_ALLOC, "Defrag: Failed to allocate tracker.");
+            SCPerfCounterIncr(dtv->counter_defrag_max_hit,
+                tv->sc_perf_pca);
             goto done;
         }
         DefragTrackerReset(tracker);