In test_fsync, adjust test headings to match wal_sync_method values;

add more test cases for open_sync of different sizes.

1 files changed, 63 insertions, 107 deletions

diff --git a/src/tools/fsync/test_fsync.c b/src/tools/fsync/test_fsync.c
index b1cec74fce2..9c829ba541d 100644
--- a/src/tools/fsync/test_fsync.c
+++ b/src/tools/fsync/test_fsync.c
@@ -47,6 +47,7 @@ void		test_open(void);
 void		test_non_sync(void);
 void		test_sync(int writes_per_op);
 void		test_open_syncs(void);
+void		test_open_sync(const char *msg, int writes_size);
 void		test_file_descriptor_sync(void);
 void		print_elapse(struct timeval start_t, struct timeval stop_t);
 void		die(char *str);
@@ -61,8 +62,6 @@ main(int argc, char *argv[])
 
 	test_open();
 	
-	test_non_sync();
-	
 	/* Test using 1 8k write */
 	test_sync(1);
 
@@ -73,6 +72,8 @@ main(int argc, char *argv[])
 
 	test_file_descriptor_sync();
 	
+	test_non_sync();
+	
 	unlink(filename);
 
 	return 0;
@@ -105,7 +106,7 @@ handle_args(int argc, char *argv[])
 	}
 
 	while ((option = getopt_long(argc, argv, "f:o:",
-								 long_options, &optindex)) != -1)
+			long_options, &optindex)) != -1)
 	{
 		switch (option)
 		{
@@ -126,7 +127,7 @@ handle_args(int argc, char *argv[])
 		}
 	}
 
-	printf("%d operations per test\n\n", ops_per_test);
+	printf("%d operations per test\n", ops_per_test);
 }
 
 void
@@ -162,40 +163,15 @@ test_open(void)
 }
 
 void
-test_non_sync(void)
-{
-	int			tmpfile, ops;
-
-	/*
-	 * Test a simple write without fsync
-	 */
-	printf("Simple non-sync'ed write:\n");
-	printf(LABEL_FORMAT, "8k write");
-	fflush(stdout);
-
-	gettimeofday(&start_t, NULL);
-	for (ops = 0; ops < ops_per_test; ops++)
-	{
-		if ((tmpfile = open(filename, O_RDWR, 0)) == -1)
-			die("Cannot open output file.");
-		if (write(tmpfile, buf, WRITE_SIZE) != WRITE_SIZE)
-			die("write failed");
-		close(tmpfile);
-	}
-	gettimeofday(&stop_t, NULL);
-	print_elapse(start_t, stop_t);
-}
-
-void
 test_sync(int writes_per_op)
 {
 	int			tmpfile, ops, writes;
 	bool		fs_warning = false;
 	
 	if (writes_per_op == 1)
-		printf("\nCompare file sync methods using one write:\n");
+		printf("\nCompare file sync methods using one 8k write:\n");
 	else
-		printf("\nCompare file sync methods using two writes:\n");
+		printf("\nCompare file sync methods using two 8k writes:\n");
 	printf("(in wal_sync_method preference order, except fdatasync\n");
 	printf("is Linux's default)\n");
 
@@ -203,16 +179,9 @@ test_sync(int writes_per_op)
 	 * Test open_datasync if available
 	 */
 #ifdef OPEN_DATASYNC_FLAG
-	if (writes_per_op == 1)
-		printf(LABEL_FORMAT, "open_datasync 8k write"
-#if PG_O_DIRECT != 0
-		"*"
-#endif
-		);
-	else
-	 	printf(LABEL_FORMAT, "2 open_datasync 8k writes"
+	printf(LABEL_FORMAT, "open_datasync"
 #if PG_O_DIRECT != 0
-		"*"
+		" (non-direct I/O)*"
 #endif
 		);
 	fflush(stdout);
@@ -243,10 +212,7 @@ test_sync(int writes_per_op)
 	}
 	else
 	{
-		if (writes_per_op == 1)
-			printf(LABEL_FORMAT, "open_datasync 8k direct I/O write");
-		else
-			printf(LABEL_FORMAT, "2 open_datasync 8k direct I/O writes");
+		printf(LABEL_FORMAT, "open_datasync (direct I/O)");
 		fflush(stdout);
 
 		gettimeofday(&start_t, NULL);
@@ -262,8 +228,6 @@ test_sync(int writes_per_op)
 		close(tmpfile);
 		print_elapse(start_t, stop_t);
 	}
-#else
-		printf(NA_FORMAT, "o_direct", "n/a\n");
 #endif
 
 #else
@@ -274,10 +238,7 @@ test_sync(int writes_per_op)
  * Test fdatasync if available
  */
 #ifdef HAVE_FDATASYNC
-	if (writes_per_op == 1)
-		printf(LABEL_FORMAT, "8k write, fdatasync");
-	else
-		printf(LABEL_FORMAT, "8k write, 8k write, fdatasync");
+	printf(LABEL_FORMAT, "fdatasync");
 	fflush(stdout);
 
 	if ((tmpfile = open(filename, O_RDWR, 0)) == -1)
@@ -302,10 +263,7 @@ test_sync(int writes_per_op)
 /*
  * Test fsync
  */
-	if (writes_per_op == 1)
-		printf(LABEL_FORMAT, "8k write, fsync");
-	else
-		printf(LABEL_FORMAT, "8k write, 8k write, fsync");
+	printf(LABEL_FORMAT, "fsync");
 	fflush(stdout);
 
 	if ((tmpfile = open(filename, O_RDWR, 0)) == -1)
@@ -329,10 +287,7 @@ test_sync(int writes_per_op)
  * If fsync_writethrough is available, test as well
  */	
 #ifdef HAVE_FSYNC_WRITETHROUGH
-	if (writes_per_op == 1)
-		printf(LABEL_FORMAT, "8k write, fsync_writethrough");
-	else
-		printf(LABEL_FORMAT, "8k write, 8k write, fsync_writethrough");
+	printf(LABEL_FORMAT, "fsync_writethrough");
 	fflush(stdout);
 
 	if ((tmpfile = open(filename, O_RDWR, 0)) == -1)
@@ -359,16 +314,9 @@ test_sync(int writes_per_op)
  * Test open_sync if available
  */
 #ifdef OPEN_SYNC_FLAG
-	if (writes_per_op == 1)
-		printf(LABEL_FORMAT, "open_sync 8k write"
-#if PG_O_DIRECT != 0
-		"*"
-#endif
-		);
-	else
-		printf(LABEL_FORMAT, "2 open_sync 8k writes"
+	printf(LABEL_FORMAT, "open_sync"
 #if PG_O_DIRECT != 0
-		"*"
+		" (non-direct I/O)*"
 #endif
 		);
 	fflush(stdout);
@@ -399,10 +347,7 @@ test_sync(int writes_per_op)
 	}
 	else
 	{
-		if (writes_per_op == 1)
-			printf(LABEL_FORMAT, "open_sync 8k direct I/O write");
-		else
-			printf(LABEL_FORMAT, "2 open_sync 8k direct I/O writes");
+		printf(LABEL_FORMAT, "open_sync (direct I/O)");
 		fflush(stdout);
 
 		gettimeofday(&start_t, NULL);
@@ -418,8 +363,6 @@ test_sync(int writes_per_op)
 		close(tmpfile);
 		print_elapse(start_t, stop_t);
 	}
-#else
-	printf(NA_FORMAT, "o_direct", "n/a\n");
 #endif
 
 #else
@@ -428,7 +371,7 @@ test_sync(int writes_per_op)
 
 #if defined(OPEN_DATASYNC_FLAG) || defined(OPEN_SYNC_FLAG)
 	if (PG_O_DIRECT != 0)
-		printf("* This non-direct I/O option is not used by Postgres.\n");
+		printf("* This non-direct I/O mode is not used by Postgres.\n");
 #endif
 
 	if (fs_warning)
@@ -441,14 +384,22 @@ test_sync(int writes_per_op)
 void
 test_open_syncs(void)
 {
-	int			tmpfile, ops;
+	printf("\nCompare open_sync with different write sizes:\n");
+	printf("(This is designed to compare the cost of writing 16k\n");
+	printf("in different write open_sync sizes.)\n");
+
+	test_open_sync(" 1 16k open_sync write", 16);
+	test_open_sync(" 2  8k open_sync writes", 8);
+	test_open_sync(" 4  4k open_sync writes", 4);
+	test_open_sync(" 8  2k open_sync writes", 2);
+	test_open_sync("16  1k open_sync writes", 1);
+}
 
-	/*
-	 * Compare 1 to 2 writes
-	 */
-	printf("\nCompare open_sync with different sizes:\n");
-	printf("(This is designed to compare the cost of one large\n");
-	printf("sync'ed write and two smaller sync'ed writes.)\n");
+
+void
+test_open_sync(const char *msg, int writes_size)
+{
+	int		tmpfile, ops, writes;
 
 /*
  * Test open_sync with different size files
@@ -458,14 +409,15 @@ test_open_syncs(void)
 		printf(NA_FORMAT, "o_direct", "n/a**\n");
 	else
 	{
-		printf(LABEL_FORMAT, "open_sync 16k write");
+		printf(LABEL_FORMAT, msg);
 		fflush(stdout);
 
 		gettimeofday(&start_t, NULL);
 		for (ops = 0; ops < ops_per_test; ops++)
 		{
-			if (write(tmpfile, buf, WRITE_SIZE * 2) != WRITE_SIZE * 2)
-				die("write failed");
+			for (writes = 0; writes < 16 / writes_size; writes++)
+				if (write(tmpfile, buf, writes_size) != writes_size)
+					die("write failed");
 			if (lseek(tmpfile, 0, SEEK_SET) == -1)
 				die("seek failed");
 		}
@@ -474,27 +426,6 @@ test_open_syncs(void)
 		print_elapse(start_t, stop_t);
 	}
 	
-	if ((tmpfile = open(filename, O_RDWR | OPEN_SYNC_FLAG | PG_O_DIRECT, 0)) == -1)
-		printf(NA_FORMAT, "o_direct", "n/a**\n");
-	else
-	{
-		printf(LABEL_FORMAT, "2 open_sync 8k writes");
-		fflush(stdout);
-
-		gettimeofday(&start_t, NULL);
-		for (ops = 0; ops < ops_per_test; ops++)
-		{
-			if (write(tmpfile, buf, WRITE_SIZE) != WRITE_SIZE)
-				die("write failed");
-			if (write(tmpfile, buf, WRITE_SIZE) != WRITE_SIZE)
-				die("write failed");
-			if (lseek(tmpfile, 0, SEEK_SET) == -1)
-				die("seek failed");
-		}
-		gettimeofday(&stop_t, NULL);
-		close(tmpfile);
-		print_elapse(start_t, stop_t);
-	}
 #else
 	printf(NA_FORMAT, "open_sync", "n/a\n");
 #endif
@@ -520,7 +451,7 @@ test_file_descriptor_sync(void)
 	 * first write, fsync and close, which is the 
 	 * normal behavior without multiple descriptors
 	 */
-	printf(LABEL_FORMAT, "8k write, fsync, close");
+	printf(LABEL_FORMAT, "write, fsync, close");
 	fflush(stdout);
 
 	gettimeofday(&start_t, NULL);
@@ -549,7 +480,7 @@ test_file_descriptor_sync(void)
 	 * This simulates processes fsyncing each other's
 	 * writes.
 	 */
- 	printf(LABEL_FORMAT, "8k write, close, fsync");
+ 	printf(LABEL_FORMAT, "write, close, fsync");
  	fflush(stdout);
 
 	gettimeofday(&start_t, NULL);
@@ -572,6 +503,31 @@ test_file_descriptor_sync(void)
 
 }
 
+void
+test_non_sync(void)
+{
+	int			tmpfile, ops;
+
+	/*
+	 * Test a simple write without fsync
+	 */
+	printf("\nNon-sync'ed 8k writes:\n");
+	printf(LABEL_FORMAT, "write");
+	fflush(stdout);
+
+	gettimeofday(&start_t, NULL);
+	for (ops = 0; ops < ops_per_test; ops++)
+	{
+		if ((tmpfile = open(filename, O_RDWR, 0)) == -1)
+			die("Cannot open output file.");
+		if (write(tmpfile, buf, WRITE_SIZE) != WRITE_SIZE)
+			die("write failed");
+		close(tmpfile);
+	}
+	gettimeofday(&stop_t, NULL);
+	print_elapse(start_t, stop_t);
+}
+
 /* 
  * print out the writes per second for tests
  */