ivampiresp/Spdk
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

thread: remove redundant set_thread in intr

Browse Source 
thread_interrupt_msg_process is registered to thread's
fd_group, so it will be called inside spdk_thread_poll.
Since spdk_thread_poll will set/restore tls_thread,
there is no need to set or restore it again here.

Change-Id: Ida10c736ef904ff975eeb42fd0cccad9fd8317cf
Signed-off-by: Liu Xiaodong <xiaodong.liu@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/6720
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Community-CI: Mellanox Build Bot
Reviewed-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
This commit is contained in:
Liu Xiaodong 2021-03-05 07:41:59 -05:00 committed by Tomasz Zawadzki
parent ba7aac83ed
commit 3690dbbd2e
1 changed files with 0 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
lib/thread/thread.c
View File

@ -1861,15 +1861,11 @@ static int
thread_interrupt_msg_process(void *arg) thread_interrupt_msg_process(void *arg)
{ {
struct spdk_thread *thread = arg; struct spdk_thread *thread = arg;
struct spdk_thread *orig_thread;
uint32_t msg_count; uint32_t msg_count;
spdk_msg_fn critical_msg; spdk_msg_fn critical_msg;
int rc = 0; int rc = 0;
uint64_t now = spdk_get_ticks(); uint64_t now = spdk_get_ticks();
orig_thread = _get_thread();
tls_thread = thread;
critical_msg = thread->critical_msg; critical_msg = thread->critical_msg;
if (spdk_unlikely(critical_msg != NULL)) { if (spdk_unlikely(critical_msg != NULL)) {
critical_msg(NULL); critical_msg(NULL);
@ -1883,8 +1879,6 @@ thread_interrupt_msg_process(void *arg)
thread_update_stats(thread, spdk_get_ticks(), now, rc); thread_update_stats(thread, spdk_get_ticks(), now, rc);
tls_thread = orig_thread;
return rc; return rc;
} }