diff options
| author | Krishnan Parthasarathi <kparthas@redhat.com> | 2013-05-11 14:29:29 +0530 |
|---|---|---|
| committer | Anand Avati <avati@redhat.com> | 2013-05-20 11:42:22 -0700 |
| commit | c377febe7ac559a6940143db09a092b511647958 (patch) | |
| tree | da7e50623244448e0bd4e8a769869f12d886cc15 /libglusterfs/src/syncop.h | |
| parent | 10fbae681e072dd6359bf7a8c56e93c541c83b0b (diff) | |
syncop: Update synctask state appropriately
* Earlier, SYNCOP macro, the only consumer of synctask_yield, would set
the task->state to SYNCTASK_SUSPEND. Today, we have glusterd having its
own wrapper macros which don't set task's state. There is also the
syncbarrier and synclock framework, which also participate in a
synctask's scheduling (and need to keep a task's state up to date). It
only makes more sense to leave a synctask's state to the synctask
library, since its an internal affair.
* Need to 'yawn' before 'yield' to avoid re-running tasks to set
task->woken appropriately.
Change-Id: Ic7a59e6ebcc46f03e53223ca237668d45a3cba40
BUG: 948686
Signed-off-by: Krishnan Parthasarathi <kparthas@redhat.com>
Reviewed-on: http://review.gluster.org/4985
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Anand Avati <avati@redhat.com>
Diffstat (limited to 'libglusterfs/src/syncop.h')
| -rw-r--r-- | libglusterfs/src/syncop.h | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/libglusterfs/src/syncop.h b/libglusterfs/src/syncop.h index 7ede9663819..5c439f85d3d 100644 --- a/libglusterfs/src/syncop.h +++ b/libglusterfs/src/syncop.h @@ -210,8 +210,6 @@ struct syncargs { \ STACK_WIND_COOKIE (frame, cbk, (void *)stb, subvol, \ op, params); \ - if (task) \ - task->state = SYNCTASK_SUSPEND; \ \ __yield (stb); \ if (task) \ |