Author: kib
Date: Thu Sep 22 10:54:30 2016
New Revision: 306182
URL: https://svnweb.freebsd.org/changeset/base/306182

Log:
  MFC r305601:
  On rename, do not perform truncation of dirhash if the vnode truncation 
failed.

Modified:
  stable/10/sys/ufs/ufs/ufs_vnops.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/sys/ufs/ufs/ufs_vnops.c
==============================================================================
--- stable/10/sys/ufs/ufs/ufs_vnops.c   Thu Sep 22 10:53:13 2016        
(r306181)
+++ stable/10/sys/ufs/ufs/ufs_vnops.c   Thu Sep 22 10:54:30 2016        
(r306182)
@@ -1530,11 +1530,21 @@ unlockout:
         * are no longer needed.
         */
        if (error == 0 && endoff != 0) {
+               error = UFS_TRUNCATE(tdvp, endoff, IO_NORMAL | IO_SYNC,
+                   tcnp->cn_cred);
+               if (error != 0)
+                       vn_printf(tdvp, "ufs_rename: failed to truncate "
+                           "err %d", error);
 #ifdef UFS_DIRHASH
-               if (tdp->i_dirhash != NULL)
+               else if (tdp->i_dirhash != NULL)
                        ufsdirhash_dirtrunc(tdp, endoff);
 #endif
-               UFS_TRUNCATE(tdvp, endoff, IO_NORMAL | IO_SYNC, tcnp->cn_cred);
+               /*
+                * Even if the directory compaction failed, rename was
+                * succesful.  Do not propagate a UFS_TRUNCATE() error
+                * to the caller.
+                */
+               error = 0;
        }
        if (error == 0 && tdp->i_flag & IN_NEEDSYNC)
                error = VOP_FSYNC(tdvp, MNT_WAIT, td);
_______________________________________________
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to