Changeset 745 for trunk/server/source3/modules/vfs_syncops.c

Timestamp:

Nov 27, 2012, 4:43:17 PM (13 years ago)

Author:

Silvan Scherrer

Message:

Samba Server: updated trunk to 3.6.0

Location:

trunk/server

Files:

• . (modified) (1 prop)
• source3/modules/vfs_syncops.c (modified) (8 diffs)

trunk/server/source3/modules/vfs_syncops.c

@@ -3 +3 @@
  *
  * Copyright (C) Andrew Tridgell     2007
+
  *
  * This program is free software; you can redistribute it and/or modify
@@ -20 +21 @@
 
 #include "includes.h"
+
+
 
 /*
@@ -32 +35 @@
   On those filesystems this module provides a way to perform those
   operations safely.  
- */
-
-/*
+
   most of the performance loss with this module is in fsync on close(). 
-  You can disable that with syncops:onclose = no
- */
-static bool sync_onclose;
+  You can disable that with
+     syncops:onclose = no
+  that can be set either globally or per share.
+
+  On certain filesystems that only require the last data written to be
+  fsync()'ed, you can disable the metadata synchronization of this module with
+     syncops:onmeta = no
+  This option can be set either globally or per share.
+
+  you can also disable the module completely for a share with
+     syncops:disable = true
+
+ */
+
+struct syncops_config_data {
+        bool onclose;
+        bool onmeta;
+        bool disable;
+};
 
 /*
@@ -126 +143 @@
                           const struct smb_filename *smb_fname_dst)
 {
-        int ret = SMB_VFS_NEXT_RENAME(handle, smb_fname_src, smb_fname_dst);
-        if (ret == 0) {
+
+        int ret;
+        struct syncops_config_data *config;
+
+        SMB_VFS_HANDLE_GET_DATA(handle, config,
+                                struct syncops_config_data,
+                                return -1);
+
+        ret = SMB_VFS_NEXT_RENAME(handle, smb_fname_src, smb_fname_dst);
+        if (ret == 0 && config->onmeta && !config->disable) {
                 syncops_two_names(smb_fname_src->base_name,
                                   smb_fname_dst->base_name);
@@ -136 +161 @@
 /* handle the rest with a macro */
 #define SYNCOPS_NEXT(op, fname, args) do {   \
-        int ret = SMB_VFS_NEXT_ ## op args; \
-        if (ret == 0 && fname) syncops_name(fname); \
+        int ret; \
+        struct syncops_config_data *config; \
+        SMB_VFS_HANDLE_GET_DATA(handle, config, \
+                                struct syncops_config_data, \
+                                return -1); \
+        ret = SMB_VFS_NEXT_ ## op args; \
+        if (ret == 0 \
+                && config->onmeta && !config->disable  \
+                && fname) syncops_name(fname); \
         return ret; \
 } while (0)
 
 #define SYNCOPS_NEXT_SMB_FNAME(op, fname, args) do {   \
-        int ret = SMB_VFS_NEXT_ ## op args; \
-        if (ret == 0 && fname) syncops_smb_fname(fname); \
+        int ret; \
+        struct syncops_config_data *config; \
+        SMB_VFS_HANDLE_GET_DATA(handle, config, \
+                                struct syncops_config_data, \
+                                return -1); \
+        ret = SMB_VFS_NEXT_ ## op args; \
+        if (ret == 0 \
+        && config->onmeta && !config->disable \
+        && fname) syncops_smb_fname(fname); \
         return ret; \
 } while (0)
@@ -192 +231 @@
 static int syncops_close(vfs_handle_struct *handle, files_struct *fsp)
 {
-        if (fsp->can_write && sync_onclose) {
+        struct syncops_config_data *config;
+
+        SMB_VFS_HANDLE_GET_DATA(handle, config,
+                                struct syncops_config_data,
+                                return -1);
+
+        if (fsp->can_write && config->onclose) {
                 /* ideally we'd only do this if we have written some
                  data, but there is no flag for that in fsp yet. */
@@ -200 +245 @@
 }
 
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
 
 static struct vfs_fn_pointers vfs_syncops_fns = {
+
         .mkdir = syncops_mkdir,
         .rmdir = syncops_rmdir,
-        .open = syncops_open,
+        .open = syncops_open,
         .rename = syncops_rename,
         .unlink = syncops_unlink,
@@ -223 +302 @@
                 return ret;
 
-        sync_onclose = lp_parm_bool(-1, "syncops", "onclose", true);
-        
         return ret;
 }