Commit 18667806 authored by Tomasz Zawadzki's avatar Tomasz Zawadzki
Browse files

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: default avatarTomasz Zawadzki <tomasz.zawadzki@intel.com>
Change-Id: I0477830902f68102e4e4f0ffc9359bd004a8ad42
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/7961


Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Reviewed-by: default avatarPaul Luse <paul.e.luse@intel.com>
Reviewed-by: default avatarShuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-by: default avatarKonrad Sztyber <konrad.sztyber@intel.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Community-CI: Mellanox Build Bot
parent d9f5da13

lib/event/reactor.c

+2 −2
Original line number Diff line number Diff line
@@ -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++;