From 0a5bda8d34e0a5fe7b953c36ca12ece8d7701822 Mon Sep 17 00:00:00 2001 From: Eric Leblond Date: Sun, 9 Sep 2012 22:48:46 +0200 Subject: [PATCH] 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. --- src/decode.c | 3 +++ src/decode.h | 1 + src/defrag.c | 4 ++-- 3 files changed, 6 insertions(+), 2 deletions(-) diff --git a/src/decode.c b/src/decode.c index 4bdd287587..a6b5548ff9 100644 --- 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 eb62bcd341..035f25c474 100644 --- 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 99a320a848..729f884407 100644 --- 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); -- 2.47.3