From a1133a9b675fb202414ecd16f279dfa1f8e97d94 Mon Sep 17 00:00:00 2001 From: Ben Walker Date: Fri, 10 Nov 2017 14:58:44 -0700 Subject: [PATCH] event: Remove extra event from subsystem shutdown path Change-Id: I1b4a64fc5efdb5abd8bdcd9e91a7e8fb06fbfe3a Reviewed-on: https://review.gerrithub.io/386714 Reviewed-by: Dariusz Stojaczyk Reviewed-by: Daniel Verkamp Reviewed-by: Jim Harris Tested-by: SPDK Automated Test System --- lib/event/subsystem.c | 16 ++-------------- 1 file changed, 2 insertions(+), 14 deletions(-) diff --git a/lib/event/subsystem.c b/lib/event/subsystem.c index 4adec0ea1..a4248f724 100644 --- a/lib/event/subsystem.c +++ b/lib/event/subsystem.c @@ -204,27 +204,15 @@ spdk_subsystem_fini_next(void) } } -static void -spdk_subsystem_fini_schedule(void *arg1, void *arg2) -{ - spdk_subsystem_fini_next(); -} - void spdk_subsystem_fini(struct spdk_event *app_stop_event) { - struct spdk_event *fini_event; - assert(g_next_subsystem == NULL); g_app_stop_event = app_stop_event; - - /* There is assumption that whole fini path is done on one core. */ - assert(g_app_stop_event->lcore == spdk_env_get_current_core()); - g_fini_core = spdk_env_get_current_core(); - fini_event = spdk_event_allocate(g_fini_core, spdk_subsystem_fini_schedule, NULL, NULL); - spdk_event_call(fini_event); + + spdk_subsystem_fini_next(); } void