diff options
| author | Harshavardhana <harsha@harshavardhana.net> | 2013-08-13 17:35:20 -0700 |
|---|---|---|
| committer | Anand Avati <avati@redhat.com> | 2013-08-13 23:31:49 -0700 |
| commit | 0d756dc618c1a4b659a3531aec449506ce577f50 (patch) | |
| tree | 54480f2ebea66081e85a68ddf12b17347f11e136 /xlators | |
| parent | a78f7dcddbc3d2048a184b7f4f45a63586eb5580 (diff) | |
posix: Default value for `batch-fsync-delay-usec` should be '0'
Also fixes for failing testcase `./tests/bugs/bug-888174.t`,
which has been failing sporadically for many patches.
Change-Id: Ic7d2c95da5d3126623cec403207afadd449bf950
BUG: 927146
Signed-off-by: Harshavardhana <harsha@harshavardhana.net>
Reviewed-on: http://review.gluster.org/5620
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Anand Avati <avati@redhat.com>
Diffstat (limited to 'xlators')
| -rw-r--r-- | xlators/storage/posix/src/posix.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/xlators/storage/posix/src/posix.c b/xlators/storage/posix/src/posix.c index e9502bfe5f4..49d1effbcab 100644 --- a/xlators/storage/posix/src/posix.c +++ b/xlators/storage/posix/src/posix.c @@ -629,7 +629,7 @@ _posix_fallocate(call_frame_t *frame, xlator_t *this, fd_t *fd, int32_t keep_siz ret = posix_do_fallocate(frame, this, fd, flags, offset, len, &statpre, &statpost); - if (ret < 0) + if (ret < 0) goto err; STACK_UNWIND_STRICT(fallocate, frame, 0, 0, &statpre, &statpost, NULL); @@ -651,7 +651,7 @@ posix_discard(call_frame_t *frame, xlator_t *this, fd_t *fd, off_t offset, ret = posix_do_fallocate(frame, this, fd, flags, offset, len, &statpre, &statpost); - if (ret < 0) + if (ret < 0) goto err; STACK_UNWIND_STRICT(discard, frame, 0, 0, &statpre, &statpost, NULL); @@ -4994,7 +4994,7 @@ struct volume_options options[] = { }, { .key = {"batch-fsync-delay-usec"}, .type = GF_OPTION_TYPE_INT, - .default_value = "1000000", + .default_value = "0", .description = "Num of usecs to wait for aggregating fsync" " requests", }, |