From 18667806ca53b16be05008d01537f739ab838cb5 Mon Sep 17 00:00:00 2001 From: Tomasz Zawadzki Date: Wed, 19 May 2021 08:51:24 -0400 Subject: [PATCH] lib/event: update lw_thread->lcore after thread move lw_thread->lcore was set during gather_metrics, rather than just after the thread reschedule. This patch just moves it to the right place. Signed-off-by: Tomasz Zawadzki Change-Id: I0477830902f68102e4e4f0ffc9359bd004a8ad42 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/7961 Reviewed-by: Jim Harris Reviewed-by: Paul Luse Reviewed-by: Shuhei Matsumoto Reviewed-by: Ben Walker Reviewed-by: Konrad Sztyber Tested-by: SPDK CI Jenkins Community-CI: Mellanox Build Bot --- lib/event/reactor.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/event/reactor.c b/lib/event/reactor.c index ce575deaf..e6d817993 100644 --- a/lib/event/reactor.c +++ b/lib/event/reactor.c @@ -679,8 +679,6 @@ _init_thread_stats(struct spdk_reactor *reactor, struct spdk_lw_thread *lw_threa /* Read total_stats before updating it to calculate stats during the last scheduling period. */ prev_total_stats = lw_thread->total_stats; - lw_thread->lcore = reactor->lcore; - spdk_set_thread(thread); spdk_thread_get_stats(&lw_thread->total_stats); spdk_set_thread(NULL); @@ -1155,6 +1153,8 @@ _schedule_thread(void *arg1, void *arg2) spdk_thread_get_stats(&lw_thread->total_stats); spdk_set_thread(NULL); + lw_thread->lcore = current_core; + TAILQ_INSERT_TAIL(&reactor->threads, lw_thread, link); reactor->thread_count++;