summaryrefslogtreecommitdiffstats
path: root/xlators/protocol/server/src/server.c
diff options
context:
space:
mode:
authorSachin Pandit <spandit@redhat.com>2014-04-28 05:58:41 +0530
committerVijay Bellur <vbellur@redhat.com>2014-04-30 02:13:26 -0700
commitd09b327a2796152eb80074169e17359394ae7cf8 (patch)
tree3578f654c5a2a97aab689fd5e8e5518402c3eda9 /xlators/protocol/server/src/server.c
parente2034a84d284c37ade9f49be1589e3f53321bb23 (diff)
glusterd/snapshot : Clean up of old barrier code.
As a new barrier translator is introduced, we dont require the old barrier code. Hence cleaning thar up. Change-Id: Ieedca6f33a746898f0d2332fda1f1d4c86fff98f BUG: 1061685 Signed-off-by: Sachin Pandit <spandit@redhat.com> Reviewed-on: http://review.gluster.org/7577 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Kaushal M <kaushal@redhat.com> Reviewed-by: Vijaikumar Mallikarjuna <vmallika@redhat.com> Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'xlators/protocol/server/src/server.c')
-rw-r--r--xlators/protocol/server/src/server.c109
1 files changed, 0 insertions, 109 deletions
diff --git a/xlators/protocol/server/src/server.c b/xlators/protocol/server/src/server.c
index 6bd00cac047..5bd702c537c 100644
--- a/xlators/protocol/server/src/server.c
+++ b/xlators/protocol/server/src/server.c
@@ -142,9 +142,6 @@ server_submit_reply (call_frame_t *frame, rpcsvc_request_t *req, void *arg,
char new_iobref = 0;
client_t *client = NULL;
gf_boolean_t lk_heal = _gf_false;
- server_conf_t *conf = NULL;
- gf_barrier_t *barrier = NULL;
- gf_barrier_payload_t *stub = NULL;
gf_boolean_t barriered = _gf_false;
GF_VALIDATE_OR_GOTO ("server", req, ret);
@@ -153,7 +150,6 @@ server_submit_reply (call_frame_t *frame, rpcsvc_request_t *req, void *arg,
state = CALL_STATE (frame);
frame->local = NULL;
client = frame->root->client;
- conf = (server_conf_t *) client->this->private;
}
if (client)
@@ -176,32 +172,6 @@ server_submit_reply (call_frame_t *frame, rpcsvc_request_t *req, void *arg,
iobref_add (iobref, iob);
- if (conf)
- barrier = conf->barrier;
- if (barrier) {
- /* todo: write's with fd flags set to O_SYNC and O_DIRECT */
- LOCK (&barrier->lock);
- {
- if (is_fop_barriered (barrier->fops, req->procnum) &&
- (barrier_add_to_queue (barrier))) {
- stub = gf_barrier_payload (req, &rsp, frame,
- payload,
- payloadcount, iobref,
- iob, new_iobref);
- if (stub) {
- gf_barrier_enqueue (barrier, stub);
- barriered = _gf_true;
- } else {
- gf_log ("", GF_LOG_ERROR, "Failed to "
- " barrier fop %"PRIu64,
- ((uint64_t)1 << req->procnum));
- }
- }
- }
- UNLOCK (&barrier->lock);
- if (barriered == _gf_true)
- goto out;
- }
/* Then, submit the message for transmission. */
ret = rpcsvc_submit_generic (req, &rsp, 1, payload, payloadcount,
iobref);
@@ -243,7 +213,6 @@ ret:
if (new_iobref) {
iobref_unref (iobref);
}
-out:
return ret;
}
@@ -827,8 +796,6 @@ init (xlator_t *this)
server_conf_t *conf = NULL;
rpcsvc_listener_t *listener = NULL;
char *statedump_path = NULL;
- gf_barrier_t *barrier = NULL;
- char *str = NULL;
GF_VALIDATE_OR_GOTO ("init", this, out);
if (this->children == NULL) {
@@ -977,37 +944,6 @@ init (xlator_t *this)
}
}
#endif
- /* barrier related */
- barrier = GF_CALLOC (1, sizeof (*barrier),1);
- if (!barrier) {
- gf_log (this->name, GF_LOG_WARNING,
- "WARNING: Failed to allocate barrier");
- ret = -1;
- goto out;
- }
-
- LOCK_INIT (&barrier->lock);
- INIT_LIST_HEAD (&barrier->queue);
- barrier->on = _gf_false;
-
- GF_OPTION_INIT ("barrier-queue-length", barrier->max_size,
- int64, out);
- GF_OPTION_INIT ("barrier-timeout", barrier->time_out,
- uint64, out);
-
- ret = dict_get_str (this->options, "barrier-fops", &str);
- if (ret) {
- gf_log (this->name, GF_LOG_DEBUG,
- "setting barrier fops to default value");
- }
- ret = gf_barrier_fops_configure (this, barrier, str);
- if (ret) {
- gf_log (this->name, GF_LOG_ERROR,
- "invalid barrier fops specified");
- goto out;
- }
-
- conf->barrier = barrier;
this->private = conf;
ret = 0;
@@ -1072,37 +1008,10 @@ notify (xlator_t *this, int32_t event, void *data, ...)
va_end (ap);
switch (event) {
- case GF_EVENT_VOLUME_BARRIER_OP:
- ret = dict_get_int32 (dict, "barrier", &val);
- if (ret) {
- gf_log (this->name, GF_LOG_ERROR,
- "Wrong BARRIER event");
- goto out;
- }
- /* !val un-barrier, if val, barrier */
- if (val) {
- ret = gf_barrier_start (this);
- if (ret)
- gf_log (this->name, GF_LOG_ERROR,
- "Barrier start failed");
- } else {
- ret = gf_barrier_stop (this);
- if (ret)
- gf_log (this->name, GF_LOG_ERROR,
- "Barrier stop failed");
- }
- ret = dict_set_int32 (output, "barrier-status", ret);
- if (ret)
- gf_log (this->name, GF_LOG_ERROR,
- "Failed to set barrier-status in dict");
- break;
-
- /* todo: call default_notify to make other xlators handle it.*/
default:
default_notify (this, event, data);
break;
}
-out:
return ret;
}
@@ -1226,23 +1135,5 @@ struct volume_options options[] = {
"overrides the auth.allow option. By default, all"
" connections are allowed."
},
- {.key = {"barrier-timeout"},
- .type = GF_OPTION_TYPE_INT,
- .default_value = "60",
- .min = 0,
- .max = 360,
- .description = "Barrier timeout in seconds",
- },
- {.key = {"barrier-queue-length"},
- .type = GF_OPTION_TYPE_INT,
- .default_value = "4096",
- .min = 0,
- .max = 16384,
- .description = "Barrier queue length",
- },
- {.key = {"barrier-fops"},
- .type = GF_OPTION_TYPE_STR,
- .description = "Allow a comma seperated fop lists",
- },
{ .key = {NULL} },
};