From c6de1e9de73e5ce08bf9099f14da74c2c1946132 Mon Sep 17 00:00:00 2001 From: vmallika Date: Thu, 9 Jul 2015 15:34:21 +0530 Subject: quota/marker: use smaller stacksize in synctask for marker updation This is a backport of http://review.gluster.org/#/c/11499/ Default stacksize that synctask uses is 2M. For marker we set it to 16k Also move market xlator close to io-threads to have smaller stack > Change-Id: I8730132a6365cc9e242a3564a1e615d94ef2c651 > BUG: 1207735 > Signed-off-by: vmallika Change-Id: Id1cb6288a38d370956cc47aed5253ff95f04c966 BUG: 1229282 Signed-off-by: vmallika Reviewed-on: http://review.gluster.org/11595 Tested-by: Gluster Build System Reviewed-by: Raghavendra G Tested-by: Raghavendra G --- xlators/features/marker/src/marker-quota.c | 4 ++-- xlators/features/qemu-block/src/coroutine-synctask.c | 2 +- xlators/mgmt/glusterd/src/glusterd-volgen.c | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) (limited to 'xlators') diff --git a/xlators/features/marker/src/marker-quota.c b/xlators/features/marker/src/marker-quota.c index 515813873d6..b7a6efffd55 100644 --- a/xlators/features/marker/src/marker-quota.c +++ b/xlators/features/marker/src/marker-quota.c @@ -2823,8 +2823,8 @@ mq_synctask (xlator_t *this, synctask_fn_t task, gf_boolean_t spawn, loc_t *loc, args->contri = contri; if (spawn) { - ret = synctask_new (this->ctx->env, task, mq_synctask_cleanup, - NULL, args); + ret = synctask_new1 (this->ctx->env, 1024 * 16, task, + mq_synctask_cleanup, NULL, args); if (ret) { gf_log (this->name, GF_LOG_ERROR, "Failed to spawn " "new synctask"); diff --git a/xlators/features/qemu-block/src/coroutine-synctask.c b/xlators/features/qemu-block/src/coroutine-synctask.c index e43988a953f..5b7ca0ceedc 100644 --- a/xlators/features/qemu-block/src/coroutine-synctask.c +++ b/xlators/features/qemu-block/src/coroutine-synctask.c @@ -105,7 +105,7 @@ qb_coroutine (call_frame_t *frame, synctask_fn_t fn) LOCK(&qb_co.lock); if (!qb_co.task) - qb_co.task = synctask_create(qb_conf->env, qb_synctask_wrap, + qb_co.task = synctask_create(qb_conf->env, 0, qb_synctask_wrap, synctask_nop_cbk, frame, NULL); list_add_tail(&qb_local->list, &qb_co.queue); diff --git a/xlators/mgmt/glusterd/src/glusterd-volgen.c b/xlators/mgmt/glusterd/src/glusterd-volgen.c index b7d850ccdaa..79de941bda2 100644 --- a/xlators/mgmt/glusterd/src/glusterd-volgen.c +++ b/xlators/mgmt/glusterd/src/glusterd-volgen.c @@ -2234,9 +2234,9 @@ static volgen_brick_xlator_t server_graph_table[] = { {brick_graph_add_ro, NULL}, {brick_graph_add_worm, NULL}, {brick_graph_add_quota, "quota"}, - {brick_graph_add_marker, "marker"}, {brick_graph_add_index, "index"}, {brick_graph_add_barrier, NULL}, + {brick_graph_add_marker, "marker"}, {brick_graph_add_iot, "io-threads"}, {brick_graph_add_upcall, "upcall"}, {brick_graph_add_pump, NULL}, -- cgit