Commit 50c48ee9 authored by Ben Walker's avatar Ben Walker Committed by Changpeng Liu
Browse files

event: Eliminate extra event in spdk_subsystem_init 



This isn't required anymore because spdk_subsystem_init
is now always called after an event has been executed, so it
always runs after the framework has initialized.

Change-Id: I05e1a4dd605f27247b6d43f5234173e07d9b9dd2
Signed-off-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/446996


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

lib/event/subsystem.c

+5 −14
Original line number Diff line number Diff line
@@ -150,11 +150,14 @@ spdk_subsystem_init_next(int rc) 
	}

}



static void

spdk_subsystem_verify(void *arg1, void *arg2)

void

spdk_subsystem_init(spdk_msg_fn cb_fn, void *cb_arg)

{

	struct spdk_subsystem_depend *dep;



	g_app_start_fn = cb_fn;

	g_app_start_arg = cb_arg;



	/* Verify that all dependency name and depends_on subsystems are registered */

	TAILQ_FOREACH(dep, &g_subsystems_deps, tailq) {

		if (!spdk_subsystem_find(&g_subsystems, dep->name)) {
@@ -175,18 +178,6 @@ spdk_subsystem_verify(void *arg1, void *arg2) 
	spdk_subsystem_init_next(0);

}



void

spdk_subsystem_init(spdk_msg_fn cb_fn, void *cb_arg)

{

	struct spdk_event *verify_event;



	g_app_start_fn = cb_fn;

	g_app_start_arg = cb_arg;



	verify_event = spdk_event_allocate(spdk_env_get_current_core(), spdk_subsystem_verify, NULL, NULL);

	spdk_event_call(verify_event);

}



static void

_spdk_subsystem_fini_next(void *arg1)

{