Commit f619c3b1 authored by Wojciech Panfil's avatar Wojciech Panfil Committed by Tomasz Zawadzki
Browse files

scheduler/dynamic: Add assert to check for operation success 



spdk_reactor_get() may return NULL if it's unable to get the reactor. It is a critical error, so we can safely add assert there.

Change-Id: Iba10178b45891ece2e71626b16c75d956902a78d
Signed-off-by: default avatarWojciech Panfil <wojciech.panfil@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/18926


Reviewed-by: default avatarKamuda Szymon <szymon.kamuda@intel.com>
Community-CI: Mellanox Build Bot
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarTomasz Zawadzki <tomasz.zawadzki@intel.com>
Reviewed-by: default avatarKonrad Sztyber <konrad.sztyber@intel.com>
Reviewed-by: default avatarJacek Kalwas <jacek.kalwas@intel.com>
parent 77a523e4

module/scheduler/dynamic/scheduler_dynamic.c

+2 −0
Original line number Diff line number Diff line
@@ -315,6 +315,8 @@ balance(struct spdk_scheduler_core_info *cores_info, uint32_t cores_count) 
	 * if they will be used after rebalancing */

	SPDK_ENV_FOREACH_CORE(i) {

		reactor = spdk_reactor_get(i);

		assert(reactor != NULL);



		core = &cores_info[i];

		/* We can switch mode only if reactor already does not have any threads */

		if (g_cores[i].thread_count == 0 && TAILQ_EMPTY(&reactor->threads)) {