diff mbox

[2/5] block: use fdatasync instead of fsync if possible

Message ID 20090904170132.GB22964@lst.de
State Superseded
Headers show

Commit Message

Christoph Hellwig Sept. 4, 2009, 5:01 p.m. UTC 
If we are flushing the caches for our image files we only care about the
data (including the metadata required for accessing it) but not things
like timestamp updates.  So try to use fdatasync instead of fsync to
implement the flush operations.

Unfortunately many operating systems still do not support fdatasync,
so we add a qemu_fdatasync wrapper that uses fdatasync if available
as per the _POSIX_SYNCHRONIZED_IO feature macro or fsync otherwise.


Signed-off-by: Christoph Hellwig <hch@lst.de>
diff mbox

Patch

Index: qemu/block/cow.c
===================================================================
--- qemu.orig/block/cow.c	2009-09-04 13:41:43.114022303 -0300
+++ qemu/block/cow.c	2009-09-04 13:45:19.146522312 -0300
@@ -258,7 +258,7 @@  static int cow_create(const char *filena
 static void cow_flush(BlockDriverState *bs)
 {
     BDRVCowState *s = bs->opaque;
-    fsync(s->fd);
+    qemu_fdatasync(s->fd);
 }
 
 static QEMUOptionParameter cow_create_options[] = {
Index: qemu/block/raw-posix.c
===================================================================
--- qemu.orig/block/raw-posix.c	2009-09-04 13:41:43.122026044 -0300
+++ qemu/block/raw-posix.c	2009-09-04 13:59:28.402521899 -0300
@@ -723,7 +723,7 @@  static int raw_create(const char *filena
 static void raw_flush(BlockDriverState *bs)
 {
     BDRVRawState *s = bs->opaque;
-    fsync(s->fd);
+    qemu_fdatasync(s->fd);
 }
 
 
Index: qemu/cutils.c
===================================================================
--- qemu.orig/cutils.c	2009-09-04 13:41:43.130023149 -0300
+++ qemu/cutils.c	2009-09-04 13:43:58.230523144 -0300
@@ -115,6 +115,22 @@  int qemu_fls(int i)
     return 32 - clz32(i);
 }
 
+/*
+ * Make sure data goes on disk, but if possible do not bother to
+ * write out the inode just for timestamp updates.
+ *
+ * Unfortunately even in 2009 many operating systems do not support
+ * fdatasync and have to fall back to fsync.
+ */
+int qemu_fdatasync(int fd)
+{
+#ifdef _POSIX_SYNCHRONIZED_IO
+    return fdatasync(fd);
+#else
+    return fsync(fd);
+#endif
+}
+
 /* io vectors */
 
 void qemu_iovec_init(QEMUIOVector *qiov, int alloc_hint)
Index: qemu/qemu-common.h
===================================================================
--- qemu.orig/qemu-common.h	2009-09-04 13:41:43.134022470 -0300
+++ qemu/qemu-common.h	2009-09-04 13:43:58.234522745 -0300
@@ -114,6 +114,7 @@  int stristart(const char *str, const cha
 int qemu_strnlen(const char *s, int max_len);
 time_t mktimegm(struct tm *tm);
 int qemu_fls(int i);
+int qemu_fdatasync(int fd);
 
 /* path.c */
 void init_paths(const char *prefix);