Remove SYNC_ATTR check
[zfs.git] / module / zfs / zfs_vnops.c
index 5899c7f..e5e187b 100644 (file)
@@ -74,6 +74,7 @@
 #include <sys/zfs_ctldir.h>
 #include <sys/zfs_fuid.h>
 #include <sys/zfs_sa.h>
+#include <sys/zfs_vnops.h>
 #include <sys/dnlc.h>
 #include <sys/zfs_rlock.h>
 #include <sys/extdirent.h>
@@ -443,14 +444,14 @@ offset_t zfs_read_chunk_size = 1024 * 1024; /* Tunable */
  *     vp - atime updated if byte count > 0
  */
 /* ARGSUSED */
-static int
+int
 zfs_read(vnode_t *vp, uio_t *uio, int ioflag, cred_t *cr, caller_context_t *ct)
 {
        znode_t         *zp = VTOZ(vp);
        zfsvfs_t        *zfsvfs = zp->z_zfsvfs;
        objset_t        *os;
        ssize_t         n, nbytes;
-       int             error;
+       int             error = 0;
        rl_t            *rl;
        xuio_t          *xuio = NULL;
 
@@ -563,9 +564,11 @@ out:
        zfs_range_unlock(rl);
 
        ZFS_ACCESSTIME_STAMP(zfsvfs, zp);
+       zfs_inode_update(zp);
        ZFS_EXIT(zfsvfs);
        return (error);
 }
+EXPORT_SYMBOL(zfs_read);
 
 /*
  * Write the bytes to a file.
@@ -587,7 +590,7 @@ out:
  */
 
 /* ARGSUSED */
-static int
+int
 zfs_write(vnode_t *vp, uio_t *uio, int ioflag, cred_t *cr, caller_context_t *ct)
 {
        znode_t         *zp = VTOZ(vp);
@@ -763,8 +766,8 @@ again:
                            max_blksz);
                        ASSERT(abuf != NULL);
                        ASSERT(arc_buf_size(abuf) == max_blksz);
-                       if (error = uiocopy(abuf->b_data, max_blksz,
-                           UIO_WRITE, uio, &cbytes)) {
+                       if ((error = uiocopy(abuf->b_data, max_blksz,
+                           UIO_WRITE, uio, &cbytes))) {
                                dmu_return_arcbuf(abuf);
                                break;
                        }
@@ -936,9 +939,11 @@ again:
            zfsvfs->z_os->os_sync == ZFS_SYNC_ALWAYS)
                zil_commit(zilog, zp->z_id);
 
+       zfs_inode_update(zp);
        ZFS_EXIT(zfsvfs);
        return (0);
 }
+EXPORT_SYMBOL(zfs_write);
 
 void
 zfs_get_done(zgd_t *zgd, int error)
@@ -1152,7 +1157,7 @@ specvp_check(vnode_t **vpp, cred_t *cr)
  *     NA
  */
 /* ARGSUSED */
-static int
+int
 zfs_lookup(vnode_t *dvp, char *nm, vnode_t **vpp, struct pathname *pnp,
     int flags, vnode_t *rdir, cred_t *cr,  caller_context_t *ct,
     int *direntflags, pathname_t *realpnp)
@@ -1223,7 +1228,7 @@ zfs_lookup(vnode_t *dvp, char *nm, vnode_t **vpp, struct pathname *pnp,
                        return (EINVAL);
                }
 
-               if (error = zfs_get_xattrdir(VTOZ(dvp), vpp, cr, flags)) {
+               if ((error = zfs_get_xattrdir(VTOZ(dvp), vpp, cr, flags))) {
                        ZFS_EXIT(zfsvfs);
                        return (error);
                }
@@ -1232,8 +1237,8 @@ zfs_lookup(vnode_t *dvp, char *nm, vnode_t **vpp, struct pathname *pnp,
                 * Do we have permission to get into attribute directory?
                 */
 
-               if (error = zfs_zaccess(VTOZ(*vpp), ACE_EXECUTE, 0,
-                   B_FALSE, cr)) {
+               if ((error = zfs_zaccess(VTOZ(*vpp), ACE_EXECUTE, 0,
+                   B_FALSE, cr))) {
                        VN_RELE(*vpp);
                        *vpp = NULL;
                }
@@ -1251,7 +1256,7 @@ zfs_lookup(vnode_t *dvp, char *nm, vnode_t **vpp, struct pathname *pnp,
         * Check accessibility of directory.
         */
 
-       if (error = zfs_zaccess(zdp, ACE_EXECUTE, 0, B_FALSE, cr)) {
+       if ((error = zfs_zaccess(zdp, ACE_EXECUTE, 0, B_FALSE, cr))) {
                ZFS_EXIT(zfsvfs);
                return (error);
        }
@@ -1263,12 +1268,17 @@ zfs_lookup(vnode_t *dvp, char *nm, vnode_t **vpp, struct pathname *pnp,
        }
 
        error = zfs_dirlook(zdp, nm, vpp, flags, direntflags, realpnp);
-       if (error == 0)
+       if (error == 0) {
+               if (*vpp)
+                       zfs_inode_update(VTOZ(*vpp));
+
                error = specvp_check(vpp, cr);
+       }
 
        ZFS_EXIT(zfsvfs);
        return (error);
 }
+EXPORT_SYMBOL(zfs_lookup);
 
 /*
  * Attempt to create a new entry in a directory.  If the entry
@@ -1296,8 +1306,8 @@ zfs_lookup(vnode_t *dvp, char *nm, vnode_t **vpp, struct pathname *pnp,
  */
 
 /* ARGSUSED */
-static int
-zfs_create(vnode_t *dvp, char *name, vattr_t *vap, vcexcl_t excl,
+int
+zfs_create(vnode_t *dvp, char *name, vattr_t *vap, int excl,
     int mode, vnode_t **vpp, cred_t *cr, int flag, caller_context_t *ct,
     vsecattr_t *vsecp)
 {
@@ -1310,7 +1320,7 @@ zfs_create(vnode_t *dvp, char *name, vattr_t *vap, vcexcl_t excl,
        int             error;
        ksid_t          *ksid;
        uid_t           uid;
-       gid_t           gid = crgetgid(cr);
+       gid_t           gid;
        zfs_acl_ids_t   acl_ids;
        boolean_t       fuid_dirtied;
        boolean_t       have_acl = B_FALSE;
@@ -1320,6 +1330,7 @@ zfs_create(vnode_t *dvp, char *name, vattr_t *vap, vcexcl_t excl,
         * make sure file system is at proper version
         */
 
+       gid = crgetgid(cr);
        ksid = crgetsid(cr, KSID_OWNER);
        if (ksid)
                uid = ksid_getid(ksid);
@@ -1389,7 +1400,7 @@ top:
                 * Create a new file object and update the directory
                 * to reference it.
                 */
-               if (error = zfs_zaccess(dzp, ACE_ADD_FILE, 0, B_FALSE, cr)) {
+               if ((error = zfs_zaccess(dzp, ACE_ADD_FILE, 0, B_FALSE, cr))) {
                        if (have_acl)
                                zfs_acl_ids_free(&acl_ids);
                        goto out;
@@ -1518,6 +1529,8 @@ out:
                if (zp)
                        VN_RELE(ZTOV(zp));
        } else {
+               zfs_inode_update(dzp);
+               zfs_inode_update(zp);
                *vpp = ZTOV(zp);
                error = specvp_check(vpp, cr);
        }
@@ -1528,6 +1541,7 @@ out:
        ZFS_EXIT(zfsvfs);
        return (error);
 }
+EXPORT_SYMBOL(zfs_create);
 
 /*
  * Remove an entry from a directory.
@@ -1549,7 +1563,7 @@ out:
 uint64_t null_xattr = 0;
 
 /*ARGSUSED*/
-static int
+int
 zfs_remove(vnode_t *dvp, char *name, cred_t *cr, caller_context_t *ct,
     int flags)
 {
@@ -1587,8 +1601,8 @@ top:
        /*
         * Attempt to lock directory; fail if entry doesn't exist.
         */
-       if (error = zfs_dirent_lock(&dl, dzp, name, &zp, zflg,
-           NULL, realnmp)) {
+       if ((error = zfs_dirent_lock(&dl, dzp, name, &zp, zflg,
+           NULL, realnmp))) {
                if (realnmp)
                        pn_free(realnmp);
                ZFS_EXIT(zfsvfs);
@@ -1597,7 +1611,7 @@ top:
 
        vp = ZTOV(zp);
 
-       if (error = zfs_zaccess_delete(dzp, zp, cr)) {
+       if ((error = zfs_zaccess_delete(dzp, zp, cr))) {
                goto out;
        }
 
@@ -1617,7 +1631,7 @@ top:
                dnlc_remove(dvp, name);
 
        mutex_enter(&vp->v_lock);
-       may_delete_now = vp->v_count == 1 && !vn_has_cached_data(vp);
+       may_delete_now = ((vp->v_count == 1) && (!vn_has_cached_data(vp)));
        mutex_exit(&vp->v_lock);
 
        /*
@@ -1747,6 +1761,10 @@ out:
                pn_free(realnmp);
 
        zfs_dirent_unlock(dl);
+       zfs_inode_update(dzp);
+       zfs_inode_update(zp);
+       if (xzp)
+               zfs_inode_update(xzp);
 
        if (!delete_now)
                VN_RELE(vp);
@@ -1759,6 +1777,7 @@ out:
        ZFS_EXIT(zfsvfs);
        return (error);
 }
+EXPORT_SYMBOL(zfs_remove);
 
 /*
  * Create a new directory and insert it into dvp using the name
@@ -1781,7 +1800,7 @@ out:
  *      vp - ctime|mtime|atime updated
  */
 /*ARGSUSED*/
-static int
+int
 zfs_mkdir(vnode_t *dvp, char *dirname, vattr_t *vap, vnode_t **vpp, cred_t *cr,
     caller_context_t *ct, int flags, vsecattr_t *vsecp)
 {
@@ -1856,14 +1875,14 @@ zfs_mkdir(vnode_t *dvp, char *dirname, vattr_t *vap, vnode_t **vpp, cred_t *cr,
 top:
        *vpp = NULL;
 
-       if (error = zfs_dirent_lock(&dl, dzp, dirname, &zp, zf,
-           NULL, NULL)) {
+       if ((error = zfs_dirent_lock(&dl, dzp, dirname, &zp, zf,
+           NULL, NULL))) {
                zfs_acl_ids_free(&acl_ids);
                ZFS_EXIT(zfsvfs);
                return (error);
        }
 
-       if (error = zfs_zaccess(dzp, ACE_ADD_SUBDIRECTORY, 0, B_FALSE, cr)) {
+       if ((error = zfs_zaccess(dzp, ACE_ADD_SUBDIRECTORY, 0, B_FALSE, cr))) {
                zfs_acl_ids_free(&acl_ids);
                zfs_dirent_unlock(dl);
                ZFS_EXIT(zfsvfs);
@@ -1938,9 +1957,12 @@ top:
        if (zfsvfs->z_os->os_sync == ZFS_SYNC_ALWAYS)
                zil_commit(zilog, 0);
 
+       zfs_inode_update(dzp);
+       zfs_inode_update(zp);
        ZFS_EXIT(zfsvfs);
        return (0);
 }
+EXPORT_SYMBOL(zfs_mkdir);
 
 /*
  * Remove a directory subdir entry.  If the current working
@@ -1961,7 +1983,7 @@ top:
  *     dvp - ctime|mtime updated
  */
 /*ARGSUSED*/
-static int
+int
 zfs_rmdir(vnode_t *dvp, char *name, vnode_t *cwd, cred_t *cr,
     caller_context_t *ct, int flags)
 {
@@ -1987,15 +2009,15 @@ top:
        /*
         * Attempt to lock directory; fail if entry doesn't exist.
         */
-       if (error = zfs_dirent_lock(&dl, dzp, name, &zp, zflg,
-           NULL, NULL)) {
+       if ((error = zfs_dirent_lock(&dl, dzp, name, &zp, zflg,
+           NULL, NULL))) {
                ZFS_EXIT(zfsvfs);
                return (error);
        }
 
        vp = ZTOV(zp);
 
-       if (error = zfs_zaccess_delete(dzp, zp, cr)) {
+       if ((error = zfs_zaccess_delete(dzp, zp, cr))) {
                goto out;
        }
 
@@ -2066,9 +2088,12 @@ out:
        if (zfsvfs->z_os->os_sync == ZFS_SYNC_ALWAYS)
                zil_commit(zilog, 0);
 
+       zfs_inode_update(dzp);
+       zfs_inode_update(zp);
        ZFS_EXIT(zfsvfs);
        return (error);
 }
+EXPORT_SYMBOL(zfs_rmdir);
 
 /*
  * Read as many directory entries as will fit into the provided
@@ -2363,7 +2388,7 @@ update:
 
 ulong_t zfs_fsync_sync_cnt = 4;
 
-static int
+int
 zfs_fsync(vnode_t *vp, int syncflag, cred_t *cr, caller_context_t *ct)
 {
        znode_t *zp = VTOZ(vp);
@@ -2389,6 +2414,7 @@ zfs_fsync(vnode_t *vp, int syncflag, cred_t *cr, caller_context_t *ct)
        }
        return (0);
 }
+EXPORT_SYMBOL(zfs_fsync);
 
 
 /*
@@ -2407,7 +2433,7 @@ zfs_fsync(vnode_t *vp, int syncflag, cred_t *cr, caller_context_t *ct)
  *     RETURN: 0 (always succeeds)
  */
 /* ARGSUSED */
-static int
+int
 zfs_getattr(vnode_t *vp, vattr_t *vap, int flags, cred_t *cr,
     caller_context_t *ct)
 {
@@ -2442,8 +2468,8 @@ zfs_getattr(vnode_t *vp, vattr_t *vap, int flags, cred_t *cr,
         */
        if (!(zp->z_pflags & ZFS_ACL_TRIVIAL) &&
            (vap->va_uid != crgetuid(cr))) {
-               if (error = zfs_zaccess(zp, ACE_READ_ATTRIBUTES, 0,
-                   skipaclchk, cr)) {
+               if ((error = zfs_zaccess(zp, ACE_READ_ATTRIBUTES, 0,
+                   skipaclchk, cr))) {
                        ZFS_EXIT(zfsvfs);
                        return (error);
                }
@@ -2593,6 +2619,7 @@ zfs_getattr(vnode_t *vp, vattr_t *vap, int flags, cred_t *cr,
        ZFS_EXIT(zfsvfs);
        return (0);
 }
+EXPORT_SYMBOL(zfs_getattr);
 
 /*
  * Set the file attributes to the values contained in the
@@ -2613,7 +2640,7 @@ zfs_getattr(vnode_t *vp, vattr_t *vap, int flags, cred_t *cr,
  *     vp - ctime updated, mtime updated if size changed.
  */
 /* ARGSUSED */
-static int
+int
 zfs_setattr(vnode_t *vp, vattr_t *vap, int flags, cred_t *cr,
        caller_context_t *ct)
 {
@@ -3216,6 +3243,7 @@ out:
        } else {
                err2 = sa_bulk_update(zp->z_sa_hdl, bulk, count, tx);
                dmu_tx_commit(tx);
+                zfs_inode_update(zp);
        }
 
 out2:
@@ -3225,6 +3253,7 @@ out2:
        ZFS_EXIT(zfsvfs);
        return (err);
 }
+EXPORT_SYMBOL(zfs_setattr);
 
 typedef struct zfs_zlock {
        krwlock_t       *zl_rwlock;     /* lock we acquired */
@@ -3342,7 +3371,7 @@ zfs_rename_lock(znode_t *szp, znode_t *tdzp, znode_t *sdzp, zfs_zlock_t **zlpp)
  *     sdvp,tdvp - ctime|mtime updated
  */
 /*ARGSUSED*/
-static int
+int
 zfs_rename(vnode_t *sdvp, char *snm, vnode_t *tdvp, char *tnm, cred_t *cr,
     caller_context_t *ct, int flags)
 {
@@ -3520,7 +3549,7 @@ top:
         * done in a single check.
         */
 
-       if (error = zfs_zaccess_rename(sdzp, szp, tdzp, tzp, cr))
+       if ((error = zfs_zaccess_rename(sdzp, szp, tdzp, tzp, cr)))
                goto out;
 
        if (ZTOV(szp)->v_type == VDIR) {
@@ -3528,7 +3557,7 @@ top:
                 * Check to make sure rename is valid.
                 * Can't do a move like this: /usr/a/b to /usr/a/b/c/d
                 */
-               if (error = zfs_rename_lock(szp, tdzp, sdzp, &zl))
+               if ((error = zfs_rename_lock(szp, tdzp, sdzp, &zl)))
                        goto out;
        }
 
@@ -3662,13 +3691,19 @@ out:
        zfs_dirent_unlock(sdl);
        zfs_dirent_unlock(tdl);
 
+       zfs_inode_update(sdzp);
        if (sdzp == tdzp)
                rw_exit(&sdzp->z_name_lock);
 
+       if (sdzp != tdzp)
+               zfs_inode_update(tdzp);
 
+       zfs_inode_update(szp);
        VN_RELE(ZTOV(szp));
-       if (tzp)
+       if (tzp) {
+               zfs_inode_update(tzp);
                VN_RELE(ZTOV(tzp));
+       }
 
        if (zfsvfs->z_os->os_sync == ZFS_SYNC_ALWAYS)
                zil_commit(zilog, 0);
@@ -3676,6 +3711,7 @@ out:
        ZFS_EXIT(zfsvfs);
        return (error);
 }
+EXPORT_SYMBOL(zfs_rename);
 
 /*
  * Insert the indicated symbolic reference entry into the directory.
@@ -3695,7 +3731,7 @@ out:
  *     dvp - ctime|mtime updated
  */
 /*ARGSUSED*/
-static int
+int
 zfs_symlink(vnode_t *dvp, char *name, vattr_t *vap, char *link, cred_t *cr,
     caller_context_t *ct, int flags)
 {
@@ -3746,7 +3782,7 @@ top:
                return (error);
        }
 
-       if (error = zfs_zaccess(dzp, ACE_ADD_FILE, 0, B_FALSE, cr)) {
+       if ((error = zfs_zaccess(dzp, ACE_ADD_FILE, 0, B_FALSE, cr))) {
                zfs_acl_ids_free(&acl_ids);
                zfs_dirent_unlock(dl);
                ZFS_EXIT(zfsvfs);
@@ -3815,6 +3851,9 @@ top:
                txtype |= TX_CI;
        zfs_log_symlink(zilog, tx, txtype, dzp, zp, name, link);
 
+       zfs_inode_update(dzp);
+       zfs_inode_update(zp);
+
        zfs_acl_ids_free(&acl_ids);
 
        dmu_tx_commit(tx);
@@ -3829,6 +3868,7 @@ top:
        ZFS_EXIT(zfsvfs);
        return (error);
 }
+EXPORT_SYMBOL(zfs_symlink);
 
 /*
  * Return, in the buffer contained in the provided uio structure,
@@ -3848,7 +3888,7 @@ top:
  *     vp - atime updated
  */
 /* ARGSUSED */
-static int
+int
 zfs_readlink(vnode_t *vp, uio_t *uio, cred_t *cr, caller_context_t *ct)
 {
        znode_t         *zp = VTOZ(vp);
@@ -3868,9 +3908,11 @@ zfs_readlink(vnode_t *vp, uio_t *uio, cred_t *cr, caller_context_t *ct)
 
        ZFS_ACCESSTIME_STAMP(zfsvfs, zp);
 
+       zfs_inode_update(zp);
        ZFS_EXIT(zfsvfs);
        return (error);
 }
+EXPORT_SYMBOL(zfs_readlink);
 
 /*
  * Insert a new entry into directory tdvp referencing svp.
@@ -3889,7 +3931,7 @@ zfs_readlink(vnode_t *vp, uio_t *uio, cred_t *cr, caller_context_t *ct)
  *      svp - ctime updated
  */
 /* ARGSUSED */
-static int
+int
 zfs_link(vnode_t *tdvp, vnode_t *svp, char *name, cred_t *cr,
     caller_context_t *ct, int flags)
 {
@@ -3969,7 +4011,7 @@ zfs_link(vnode_t *tdvp, vnode_t *svp, char *name, cred_t *cr,
                return (EPERM);
        }
 
-       if (error = zfs_zaccess(dzp, ACE_ADD_FILE, 0, B_FALSE, cr)) {
+       if ((error = zfs_zaccess(dzp, ACE_ADD_FILE, 0, B_FALSE, cr))) {
                ZFS_EXIT(zfsvfs);
                return (error);
        }
@@ -4022,9 +4064,12 @@ top:
        if (zfsvfs->z_os->os_sync == ZFS_SYNC_ALWAYS)
                zil_commit(zilog, 0);
 
+       zfs_inode_update(dzp);
+       zfs_inode_update(szp);
        ZFS_EXIT(zfsvfs);
        return (error);
 }
+EXPORT_SYMBOL(zfs_link);
 
 /*
  * zfs_null_putapage() is used when the file system has been force
@@ -4322,6 +4367,7 @@ zfs_inactive(vnode_t *vp, cred_t *cr, caller_context_t *ct)
        zfs_zinactive(zp);
        rw_exit(&zfsvfs->z_teardown_inactive_lock);
 }
+EXPORT_SYMBOL(zfs_inactive);
 
 /*
  * Bounds-check the seek operation.
@@ -4710,7 +4756,7 @@ zfs_delmap(vnode_t *vp, offset_t off, struct as *as, caddr_t addr,
  *     vp - ctime|mtime updated
  */
 /* ARGSUSED */
-static int
+int
 zfs_space(vnode_t *vp, int cmd, flock64_t *bfp, int flag,
     offset_t offset, cred_t *cr, caller_context_t *ct)
 {
@@ -4727,7 +4773,7 @@ zfs_space(vnode_t *vp, int cmd, flock64_t *bfp, int flag,
                return (EINVAL);
        }
 
-       if (error = convoff(vp, bfp, 0, offset)) {
+       if ((error = convoff(vp, bfp, 0, offset))) {
                ZFS_EXIT(zfsvfs);
                return (error);
        }
@@ -4745,9 +4791,10 @@ zfs_space(vnode_t *vp, int cmd, flock64_t *bfp, int flag,
        ZFS_EXIT(zfsvfs);
        return (error);
 }
+EXPORT_SYMBOL(zfs_space);
 
 /*ARGSUSED*/
-static int
+int
 zfs_fid(vnode_t *vp, fid_t *fidp, caller_context_t *ct)
 {
        znode_t         *zp = VTOZ(vp);
@@ -4806,6 +4853,7 @@ zfs_fid(vnode_t *vp, fid_t *fidp, caller_context_t *ct)
        ZFS_EXIT(zfsvfs);
        return (0);
 }
+EXPORT_SYMBOL(zfs_fid);
 
 static int
 zfs_pathconf(vnode_t *vp, int cmd, ulong_t *valp, cred_t *cr,
@@ -4878,7 +4926,7 @@ zfs_pathconf(vnode_t *vp, int cmd, ulong_t *valp, cred_t *cr,
 }
 
 /*ARGSUSED*/
-static int
+int
 zfs_getsecattr(vnode_t *vp, vsecattr_t *vsecp, int flag, cred_t *cr,
     caller_context_t *ct)
 {
@@ -4894,9 +4942,10 @@ zfs_getsecattr(vnode_t *vp, vsecattr_t *vsecp, int flag, cred_t *cr,
 
        return (error);
 }
+EXPORT_SYMBOL(zfs_getsecattr);
 
 /*ARGSUSED*/
-static int
+int
 zfs_setsecattr(vnode_t *vp, vsecattr_t *vsecp, int flag, cred_t *cr,
     caller_context_t *ct)
 {
@@ -4917,6 +4966,7 @@ zfs_setsecattr(vnode_t *vp, vsecattr_t *vsecp, int flag, cred_t *cr,
        ZFS_EXIT(zfsvfs);
        return (error);
 }
+EXPORT_SYMBOL(zfs_setsecattr);
 
 /*
  * Tunable, both must be a power of 2.