127fc0d0c3
Previously core load was only considered for main lcore. Other cores were used based on cpumask only. Once an active thread was placed on core it remained there until idle. If _get_next_target_core() looped around, the core might receive another active thread. This patch makes the core load matter for placement of any thread. As of this patch if no core can fit a thread it will remain there. Later in the series least busy core will be used to balance threads when every core is already busy. Modified the functional test that depended on always selecting consecutive core, even if 'current' one fit the bill. Later in the series the round robin logic for core selection is removed all together. Fixed typo in test while here. Note: _can_core_fit_thread() intentionally does not check core->interrupt_mode and uses tsc. That flag is only updated at the end of balancing right now. Meanwhile tsc is updated one first thread moved to the core, so it is no longer considered in interrupt mode. Signed-off-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Change-Id: I95f58c94e3f5ae8a468723d1dd6e53b0e417dcc3 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/8069 Reviewed-by: Maciej Szwed <maciej.szwed@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com> Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Mellanox Build Bot |
||
|---|---|---|
| .. | ||
| app_rpc.c | ||
| app.c | ||
| dpdk_governor.c | ||
| gscheduler.c | ||
| log_rpc.c | ||
| Makefile | ||
| reactor.c | ||
| scheduler_dynamic.c | ||
| scheduler_static.c | ||
| spdk_event.map | ||