From 723e90a174fd6f4df6677eff348fa46c25e36f65 Mon Sep 17 00:00:00 2001
From: Victor Julien <victor@inliniac.net>
Date: Fri, 3 Jun 2016 10:32:43 +0200
Subject: [PATCH] affinity: rename detect-cpu-set to worker-cpu-set

Add fallback for existing configs.
---
 src/util-affinity.c | 20 ++++++++++++--------
 suricata.yaml.in    |  6 +++---
 2 files changed, 15 insertions(+), 11 deletions(-)

diff --git a/src/util-affinity.c b/src/util-affinity.c
index 08b85f8e15..9356f67c2c 100644
--- a/src/util-affinity.c
+++ b/src/util-affinity.c
@@ -39,7 +39,7 @@ ThreadsAffinityType thread_affinity[MAX_CPU_SET] = {
         .lcpu = 0,
     },
     {
-        .name = "detect-cpu-set",
+        .name = "worker-cpu-set",
         .mode_flag = EXCLUSIVE_AFFINITY,
         .prio = PRIO_MEDIUM,
         .lcpu = 0,
@@ -95,7 +95,7 @@ static void AffinitySetupInit()
     return;
 }
 
-static void build_cpuset(char *name, ConfNode *node, cpu_set_t *cpu)
+static void build_cpuset(const char *name, ConfNode *node, cpu_set_t *cpu)
 {
     ConfNode *lnode;
     TAILQ_FOREACH(lnode, &node->head, next) {
@@ -189,7 +189,11 @@ void AffinitySetupLoadFromConfig()
             continue;
         }
 
-        ThreadsAffinityType *taf = GetAffinityTypeFromName(affinity->val);
+        const char *setname = affinity->val;
+        if (strcmp(affinity->val, "detect-cpu-set") == 0)
+            setname = "worker-cpu-set";
+
+        ThreadsAffinityType *taf = GetAffinityTypeFromName(setname);
         ConfNode *node = NULL;
         ConfNode *nprio = NULL;
 
@@ -198,7 +202,7 @@ void AffinitySetupLoadFromConfig()
             exit(EXIT_FAILURE);
         } else {
             SCLogInfo("Found affinity definition for \"%s\"",
-                      affinity->val);
+                      setname);
         }
 
         CPU_ZERO(&taf->cpu_set);
@@ -206,7 +210,7 @@ void AffinitySetupLoadFromConfig()
         if (node == NULL) {
             SCLogInfo("unable to find 'cpu'");
         } else {
-            build_cpuset(affinity->val, node, &taf->cpu_set);
+            build_cpuset(setname, node, &taf->cpu_set);
         }
 
         CPU_ZERO(&taf->lowprio_cpu);
@@ -218,21 +222,21 @@ void AffinitySetupLoadFromConfig()
             if (node == NULL) {
                 SCLogDebug("unable to find 'low' prio using default value");
             } else {
-                build_cpuset(affinity->val, node, &taf->lowprio_cpu);
+                build_cpuset(setname, node, &taf->lowprio_cpu);
             }
 
             node = ConfNodeLookupChild(nprio, "medium");
             if (node == NULL) {
                 SCLogDebug("unable to find 'medium' prio using default value");
             } else {
-                build_cpuset(affinity->val, node, &taf->medprio_cpu);
+                build_cpuset(setname, node, &taf->medprio_cpu);
             }
 
             node = ConfNodeLookupChild(nprio, "high");
             if (node == NULL) {
                 SCLogDebug("unable to find 'high' prio using default value");
             } else {
-                build_cpuset(affinity->val, node, &taf->hiprio_cpu);
+                build_cpuset(setname, node, &taf->hiprio_cpu);
             }
             node = ConfNodeLookupChild(nprio, "default");
             if (node != NULL) {
diff --git a/suricata.yaml.in b/suricata.yaml.in
index 55da24cbe0..6caadc84b3 100644
--- a/suricata.yaml.in
+++ b/suricata.yaml.in
@@ -1270,7 +1270,7 @@ threading:
   #
   # These 2 apply to the all runmodes:
   # management-cpu-set is used for flow timeout handling, counters
-  # detect-cpu-set is used for 'worker' threads
+  # worker-cpu-set is used for 'worker' threads
   #
   # Additionally, for autofp these apply:
   # receive-cpu-set is used for capture threads
@@ -1281,9 +1281,9 @@ threading:
         cpu: [ 0 ]  # include only these cpus in affinity settings
     - receive-cpu-set:
         cpu: [ 0 ]  # include only these cpus in affinity settings
-    - detect-cpu-set:
+    - worker-cpu-set:
         cpu: [ "all" ]
-        mode: "exclusive" # run detect threads in these cpus
+        mode: "exclusive"
         # Use explicitely 3 threads and don't compute number by using
         # detect-thread-ratio variable:
         # threads: 3