Commit 72eed604 authored by Tomasz Zawadzki's avatar Tomasz Zawadzki Committed by Jim Harris
Browse files

lib/event: do not spawn a thread for each reactor 



Those spdk threads were neither used or tracked in event framework.

Needed to edit hardcoded cpumask value in spdkcli test to accept any string.

Signed-off-by: default avatarTomasz Zawadzki <tomasz.zawadzki@intel.com>
Change-Id: Ieffea1bdf467f50ec0bd3af4b17a1d259fd876b4
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/5817


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Reviewed-by: default avatarChangpeng Liu <changpeng.liu@intel.com>
Reviewed-by: default avatarShuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
parent 229a67ad

lib/event/reactor.c

+0 −10
Original line number Diff line number Diff line
@@ -811,10 +811,8 @@ void 
spdk_reactors_start(void)

{

	struct spdk_reactor *reactor;

	struct spdk_cpuset tmp_cpumask = {};

	uint32_t i, current_core;

	int rc;

	char thread_name[32];



	g_rusage_period = (CONTEXT_SWITCH_MONITOR_PERIOD * spdk_get_ticks_hz()) / SPDK_SEC_TO_USEC;

	g_reactor_state = SPDK_REACTOR_STATE_RUNNING;
@@ -833,14 +831,6 @@ spdk_reactors_start(void) 
				assert(false);

				return;

			}



			/* For now, for each reactor spawn one thread. */

			snprintf(thread_name, sizeof(thread_name), "reactor_%u", reactor->lcore);



			spdk_cpuset_zero(&tmp_cpumask);

			spdk_cpuset_set_cpu(&tmp_cpumask, i, true);



			spdk_thread_create(thread_name, &tmp_cpumask);

		}

		spdk_cpuset_set_cpu(&g_reactor_core_mask, i, true);

	}

test/spdkcli/match_files/spdkcli_details_vhost_ctrl.test.match

+1 −1
Original line number Diff line number Diff line
@@ -14,7 +14,7 @@ 
      }

    ]

  },

  "cpumask": "0x3",

  "cpumask": "$(S)",

  "ctrlr": "vhost_scsi1",

  "delay_base_us": 20,

  "iops_threshold": 1000000,