]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
threading: remove 'id' field from TmSlot
authorVictor Julien <victor@inliniac.net>
Tue, 12 Nov 2019 12:28:09 +0000 (13:28 +0100)
committerVictor Julien <victor@inliniac.net>
Fri, 7 Feb 2020 14:43:10 +0000 (15:43 +0100)
Field was now unused.

src/tm-threads.c
src/tm-threads.h

index 4b4080fb873f40930d7806444b48e1743b57540b..d1654f0dbef1e0cc0cf71213f4a8776fee179df6 100644 (file)
@@ -155,8 +155,8 @@ static void CheckSlot(const TmSlot *slot)
 {
     if (slot->slot_pre_pq.len) {
         for (Packet *xp = slot->slot_pre_pq.top; xp != NULL; xp = xp->next) {
-            SCLogNotice("pre_pq: slot id %u slot tm_id %u pre_pq.len %u packet src %s",
-                    slot->id, slot->tm_id, slot->slot_pre_pq.len, PktSrcToString(xp->pkt_src));
+            SCLogNotice("pre_pq: slot tm_id %u pre_pq.len %u packet src %s",
+                    slot->tm_id, slot->slot_pre_pq.len, PktSrcToString(xp->pkt_src));
         }
         TmThreadDumpThreads();
         abort();
@@ -798,7 +798,6 @@ void TmSlotSetFuncAppend(ThreadVars *tv, TmModule *tm, const void *data)
 
     if (tv->tm_slots == NULL) {
         tv->tm_slots = slot;
-        slot->id = 0;
     } else {
         TmSlot *a = (TmSlot *)tv->tm_slots, *b = NULL;
 
@@ -809,7 +808,6 @@ void TmSlotSetFuncAppend(ThreadVars *tv, TmModule *tm, const void *data)
         /* append the new slot */
         if (b != NULL) {
             b->slot_next = slot;
-            slot->id = b->id + 1;
         }
     }
     return;
@@ -2092,11 +2090,11 @@ static void TmThreadDoDumpSlots(const ThreadVars *tv)
 {
     for (TmSlot *s = tv->tm_slots; s != NULL; s = s->slot_next) {
         TmModule *m = TmModuleGetById(s->tm_id);
-        SCLogNotice("tv %p: -> slot %p id %d tm_id %d name %s",
-            tv, s, s->id, s->tm_id, m->name);
+        SCLogNotice("tv %p: -> slot %p tm_id %d name %s",
+            tv, s, s->tm_id, m->name);
         for (Packet *xp = s->slot_pre_pq.top; xp != NULL; xp = xp->next) {
-            SCLogNotice("tv %p: ==> pre_pq: slot id %u slot tm_id %u pre_pq.len %u packet src %s",
-                    tv, s->id, s->tm_id, s->slot_pre_pq.len, PktSrcToString(xp->pkt_src));
+            SCLogNotice("tv %p: ==> pre_pq: slot tm_id %u pre_pq.len %u packet src %s",
+                    tv, s->tm_id, s->slot_pre_pq.len, PktSrcToString(xp->pkt_src));
         }
     }
 }
index 57f3904e0babda9921534e96f342f15a621bc61b..5c4e9d64b2ef2198025aa47d9b77a04abeff434d 100644 (file)
@@ -76,9 +76,6 @@ typedef struct TmSlot_ {
     /* store the thread module id */
     int tm_id;
 
-    /* slot id, only used my TmVarSlot to know what the first slot is */
-    int id;
-
 } TmSlot;
 
 extern ThreadVars *tv_root[TVT_MAX];