From f3c7e6eaa9b14b89c1d58c0edcb5664f28914437 Mon Sep 17 00:00:00 2001
From: Ravishankar N <ravishankar@redhat.com>
Date: Tue, 4 Aug 2015 18:37:47 +0530
Subject: afr: modify afr_txn_nothing_failed()

In an AFR transaction, we need to consider something as failed only if the
failure (either in the pre-op or the FOP phase) occurs on the bricks on which a
transaction lock was obtained.

Without this, we would end up considering the transaction as failure even on the
bricks on which the lock was not obtained, resulting in unnecessary fsyncs
during the post-op phase of every write transaction for non-appending writes.

Change-Id: Iee79e5d85dc7b4c41459d8bdd04a8454bdaf9a9d
BUG: 1250170
Signed-off-by: Ravishankar N <ravishankar@redhat.com>
Reviewed-on: http://review.gluster.org/11827
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Tested-by: NetBSD Build System <jenkins@build.gluster.org>
Reviewed-by: Pranith Kumar Karampuri <pkarampu@redhat.com>
---
 tests/bugs/replicate/bug-1250170-fsync.c | 56 ++++++++++++++++++++++++++++++++
 1 file changed, 56 insertions(+)
 create mode 100644 tests/bugs/replicate/bug-1250170-fsync.c

(limited to 'tests/bugs/replicate/bug-1250170-fsync.c')

diff --git a/tests/bugs/replicate/bug-1250170-fsync.c b/tests/bugs/replicate/bug-1250170-fsync.c
new file mode 100644
index 00000000000..1d3025bcd9f
--- /dev/null
+++ b/tests/bugs/replicate/bug-1250170-fsync.c
@@ -0,0 +1,56 @@
+#include <stdio.h>
+#include <stdlib.h>
+#include <inttypes.h>
+#include <sys/types.h>
+#include <sys/stat.h>
+#include <fcntl.h>
+#include <unistd.h>
+#include <string.h>
+
+int main (int argc, char **argv)
+{
+        char *file = NULL;
+        int fd = -1;
+        char *buffer = NULL;
+        size_t buf_size = 0;
+        size_t written = 0;
+        int ret = 0;
+        off_t offset = 0;
+        int i = 0;
+        int loop_count = 5;
+
+        if (argc < 2) {
+                printf ("Usage:%s <filename>\n", argv[0]);
+                return -1;
+        }
+
+        file = argv[1];
+        buf_size = 1024;
+        buffer = calloc(1, buf_size);
+        if (!buffer) {
+                perror("calloc");
+                return -1;
+        }
+        memset (buffer, 'R', buf_size);
+
+        fd = open(file, O_WRONLY);
+        if (fd == -1) {
+                perror("open");
+                return -1;
+        }
+
+        for (i = 0; i < loop_count; i++) {
+                ret =  write (fd, buffer, buf_size);
+                if (ret == -1) {
+                        perror("write");
+                        return ret;
+                } else {
+                        written += ret;
+                }
+                offset = lseek (fd, 0 , SEEK_SET);
+        }
+
+        free(buffer);
+        return 0;
+
+}
-- 
cgit