diff options
author | Xavi Hernandez <xhernandez@redhat.com> | 2019-01-31 08:13:58 +0100 |
---|---|---|
committer | Shyamsundar Ranganathan <srangana@redhat.com> | 2019-01-31 15:41:30 +0000 |
commit | 2fb445babdd621b71676e40804fe98e95fc9084d (patch) | |
tree | fce4ef156109c5ff816b01c75176e1402a28b2e4 /libglusterfs/src/syncop.c | |
parent | 8bc33a2d90df6439e7636aadbefdd9041f9698c3 (diff) | |
download | glusterfs-2fb445babdd621b71676e40804fe98e95fc9084d.tar.gz glusterfs-2fb445babdd621b71676e40804fe98e95fc9084d.tar.xz glusterfs-2fb445babdd621b71676e40804fe98e95fc9084d.zip |
syncop: remove unnecessary call to gf_backtrace_save()
A call to gf_backtrace_save() was done on each context switch of a
synctask. The backtrace is generated writing to the filesystem, so it
can have an important impact on latency.
The generated backtrace was not used anywhere, so it's been removed.
Change-Id: I399a93b932c5b6e981c696c72c3e1ef44710ba52
Updates: bz#1193929
Signed-off-by: Xavi Hernandez <xhernandez@redhat.com>
Diffstat (limited to 'libglusterfs/src/syncop.c')
-rw-r--r-- | libglusterfs/src/syncop.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/libglusterfs/src/syncop.c b/libglusterfs/src/syncop.c index bf70daf95c..6206d4cec7 100644 --- a/libglusterfs/src/syncop.c +++ b/libglusterfs/src/syncop.c @@ -327,7 +327,6 @@ synctask_yield(struct synctask *task) if (task->state != SYNCTASK_DONE) { task->state = SYNCTASK_SUSPEND; - (void)gf_backtrace_save(task->btbuf); } if (swapcontext(&task->ctx, &task->proc->sched) < 0) { gf_msg("syncop", GF_LOG_ERROR, errno, LG_MSG_SWAPCONTEXT_FAILED, |