invert switch logic for hyperthreading/efficiency cores

Author: dbsanfte

common/arg.cpp

@@ -1387,17 +1387,17 @@ common_params_context common_params_parser_init(common_params & params, llama_ex
         }
     ));
     add_opt(common_arg(
-        {"--no-hyperthreading"},
+        {"--cpu-no-hyperthreading"},
         "disable hyperthreading/SMT for math operations (use only physical cores)",
         [](common_params & params) {
             params.cpuparams.use_hyperthreading = false;
         }
     ));
     add_opt(common_arg(
-        {"--use-efficiency-cores"},
-        "use efficiency cores (E-cores) for math operations (may degrade performance)",
+        {"--cpu-no-efficiency-cores"},
+        "disable efficiency cores (E-cores) for math operations (use only performance cores)",
         [](common_params & params) {
-            params.cpuparams.use_efficiency_cores = true;
+            params.cpuparams.use_efficiency_cores = false;
         }
     ));
     add_opt(common_arg(

common/common.cpp

@@ -281,10 +281,10 @@ int32_t cpu_get_num_math() {
     if (is_hybrid_cpu()) {
         cpu_set_t affinity;
         if (!pthread_getaffinity_np(pthread_self(), sizeof(affinity), &affinity)) {
-            // Default behavior: use hyperthreading but not efficiency cores for math
+            // Default behavior: use hyperthreading and efficiency cores for math
             // This can be overridden by environment variables or command-line options
-            bool use_hyperthreading = std::getenv("LLAMA_NO_HYPERTHREADING") == nullptr;
-            bool use_efficiency_cores = std::getenv("LLAMA_USE_EFFICIENCY_CORES") != nullptr;
+            bool use_hyperthreading = std::getenv("LLAMA_CPU_NO_HYPERTHREADING") == nullptr;
+            bool use_efficiency_cores = std::getenv("LLAMA_CPU_NO_EFFICIENCY_CORES") == nullptr;
 
             int result = cpu_count_math_cpus(n_cpu, use_hyperthreading, use_efficiency_cores);
             pthread_setaffinity_np(pthread_self(), sizeof(affinity), &affinity);

common/common.h

@@ -56,7 +56,7 @@ struct cpu_params {
     bool     strict_cpu                  = false;   // Use strict CPU placement
     uint32_t poll                        = 50;      // Polling (busywait) level (0 - no polling, 100 - mostly polling)
     bool     use_hyperthreading          = true;    // Use hyperthreading/SMT for math operations (enabled by default)
-    bool     use_efficiency_cores        = false;   // Use efficiency cores (E-cores) for math operations
+    bool     use_efficiency_cores        = true;    // Use efficiency cores (E-cores) for math operations (enabled by default)
 };
 
 int32_t cpu_get_num_physical_cores();