hammer2 - code cleanup, fix kldload issue
authorMatthew Dillon <dillon@apollo.backplane.com>
Fri, 10 Feb 2012 01:54:59 +0000 (17:54 -0800)
committerMatthew Dillon <dillon@apollo.backplane.com>
Fri, 10 Feb 2012 01:54:59 +0000 (17:54 -0800)
* More code cleanups.   Formatting, add missing copyright notices,
  staticization, etc.

* Fix a missing symbol that was renamed.

sys/vfs/hammer2/hammer2.h
sys/vfs/hammer2/hammer2_mount.h
sys/vfs/hammer2/hammer2_subr.c
sys/vfs/hammer2/hammer2_vfsops.c
sys/vfs/hammer2/hammer2_vnops.c

index 1baeeea..b38eda3 100644 (file)
@@ -116,7 +116,8 @@ struct hammer2_mount {
 
 MALLOC_DECLARE(M_HAMMER2);
 
-static inline struct mount *
+static __inline
+struct mount *
 H2TOMP(struct hammer2_mount *hmp)
 {
        return (struct mount *) hmp->hm_mp;
@@ -125,7 +126,8 @@ H2TOMP(struct hammer2_mount *hmp)
 #define VTOI(vp)       ((struct hammer2_inode *) (vp)->v_data)
 #define ITOV(ip)       ((ip)->vp)
 
-static inline struct hammer2_mount *
+static __inline
+struct hammer2_mount *
 MPTOH2(struct mount *mp)
 {
        return (struct hammer2_mount *) mp->mnt_data;
@@ -135,18 +137,21 @@ extern struct vop_ops hammer2_vnode_vops;
 extern struct vop_ops hammer2_spec_vops;
 extern struct vop_ops hammer2_fifo_vops;
 
-/* hammer2_inode.c */
-
-extern int hammer2_inactive(struct vop_inactive_args *);
-extern int hammer2_reclaim(struct vop_reclaim_args *);
-
 /* hammer2_subr.c */
 
-extern struct vnode *igetv(struct hammer2_inode *, int *);
+void hammer2_inode_lock_sh(struct hammer2_inode *ip);
+void hammer2_inode_lock_up(struct hammer2_inode *ip);
+void hammer2_inode_lock_ex(struct hammer2_inode *ip);
+void hammer2_inode_unlock_ex(struct hammer2_inode *ip);
+void hammer2_inode_unlock_up(struct hammer2_inode *ip);
+void hammer2_inode_unlock_sh(struct hammer2_inode *ip);
+
+struct vnode *igetv(struct hammer2_inode *, int *);
 
-extern void hammer2_mount_exlock(struct hammer2_mount *);
-extern void hammer2_mount_shlock(struct hammer2_mount *);
-extern void hammer2_mount_unlock(struct hammer2_mount *);
+void hammer2_mount_exlock(struct hammer2_mount *);
+void hammer2_mount_shlock(struct hammer2_mount *);
+void hammer2_mount_unlock(struct hammer2_mount *);
+struct hammer2_inode *alloci(struct hammer2_mount *hmp);
 
 #endif /* !_KERNEL */
 #endif /* !_VFS_HAMMER2_HAMMER2_H_ */
index be5db5f..13018f0 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2011 The DragonFly Project.  All rights reserved.
+ * Copyright (c) 2011-2012 The DragonFly Project.  All rights reserved.
  *
  * This code is derived from software contributed to The DragonFly Project
  * by Matthew Dillon <dillon@backplane.com>
index f4f2cc3..020b50f 100644 (file)
@@ -1,3 +1,37 @@
+/*
+ * Copyright (c) 2011-2012 The DragonFly Project.  All rights reserved.
+ *
+ * This code is derived from software contributed to The DragonFly Project
+ * by Matthew Dillon <dillon@dragonflybsd.org>
+ * by Venkatesh Srinivas <vsrinivas@dragonflybsd.org>
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ *
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ *    notice, this list of conditions and the following disclaimer in
+ *    the documentation and/or other materials provided with the
+ *    distribution.
+ * 3. Neither the name of The DragonFly Project nor the names of its
+ *    contributors may be used to endorse or promote products derived
+ *    from this software without specific, prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS
+ * FOR A PARTICULAR PURPOSE ARE DISCLAIMED.  IN NO EVENT SHALL THE
+ * COPYRIGHT HOLDERS OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT,
+ * INCIDENTAL, SPECIAL, EXEMPLARY OR CONSEQUENTIAL DAMAGES (INCLUDING,
+ * BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
+ * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED
+ * AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY,
+ * OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT
+ * OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
+ * SUCH DAMAGE.
+ */
 #include <sys/cdefs.h>
 #include <sys/cdefs.h>
 #include <sys/param.h>
  * XXX: What do we use each for? How is visibility to the inode controlled?
  */
 
-void hammer2_inode_lock_sh(struct hammer2_inode *ip)
+void
+hammer2_inode_lock_sh(struct hammer2_inode *ip)
 {
        lockmgr(&ip->lk, LK_SHARED);
 }
 
-void hammer2_inode_lock_up(struct hammer2_inode *ip)
+void
+hammer2_inode_lock_up(struct hammer2_inode *ip)
 {
        lockmgr(&ip->lk, LK_EXCLUSIVE);
        ++ip->busy;
        lockmgr(&ip->lk, LK_DOWNGRADE);
 }
 
-void hammer2_inode_lock_ex(struct hammer2_inode *ip)
+void
+hammer2_inode_lock_ex(struct hammer2_inode *ip)
 {
        lockmgr(&ip->lk, LK_EXCLUSIVE);
 }
 
-void hammer2_inode_unlock_ex(struct hammer2_inode *ip)
+void
+hammer2_inode_unlock_ex(struct hammer2_inode *ip)
 {
        lockmgr(&ip->lk, LK_RELEASE);
 }
 
-void hammer2_inode_unlock_up(struct hammer2_inode *ip)
+void
+hammer2_inode_unlock_up(struct hammer2_inode *ip)
 {
        lockmgr(&ip->lk, LK_UPGRADE);
        --ip->busy;
        lockmgr(&ip->lk, LK_RELEASE);
 }
 
-void hammer2_inode_unlock_sh(struct hammer2_inode *ip)
+void
+hammer2_inode_unlock_sh(struct hammer2_inode *ip)
 {
        lockmgr(&ip->lk, LK_RELEASE);
 }
@@ -166,7 +206,9 @@ igetv(struct hammer2_inode *ip, int *error)
  *     Allocate an inode in a HAMMER2 mount. The returned inode is locked
  *     exclusively. The HAMMER2 mountpoint must be locked on entry.
  */
-struct hammer2_inode *alloci(struct hammer2_mount *hmp) {
+struct hammer2_inode *
+alloci(struct hammer2_mount *hmp)
+{
        struct hammer2_inode *ip;
 
        kprintf("alloci\n");
index 2df206a..1346fd8 100644 (file)
@@ -31,7 +31,6 @@
  * OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
  * SUCH DAMAGE.
  */
-
 #include <sys/param.h>
 #include <sys/systm.h>
 #include <sys/kernel.h>
@@ -89,7 +88,8 @@ MALLOC_DEFINE(M_HAMMER2, "HAMMER2-mount", "");
 VFS_SET(hammer2_vfsops, hammer2, 0);
 MODULE_VERSION(hammer2, 1);
 
-static int
+static
+int
 hammer2_init(struct vfsconf *conf)
 {
        int error;
@@ -131,7 +131,8 @@ hammer2_init(struct vfsconf *conf)
  * RETURNS:    0       Success
  *             !0      error number
  */
-static int
+static
+int
 hammer2_mount(struct mount *mp, char *path, caddr_t data,
              struct ucred *cred)
 {
@@ -145,7 +146,9 @@ hammer2_mount(struct mount *mp, char *path, caddr_t data,
        char *dev, *label;
        int ronly;
        int error;
+#if 0
        int rc;
+#endif
 
        hmp = NULL;
        dev = label = NULL;
@@ -300,14 +303,16 @@ hammer2_mount(struct mount *mp, char *path, caddr_t data,
        return 0;
 }
 
-static int
+static
+int
 hammer2_remount(struct mount *mp, char *path, struct vnode *devvp,
                 struct ucred *cred)
 {
        return (0);
 }
 
-static int
+static
+int
 hammer2_unmount(struct mount *mp, int mntflags)
 {
        struct hammer2_mount *hmp;
@@ -346,7 +351,8 @@ hammer2_unmount(struct mount *mp, int mntflags)
        return (error);
 }
 
-static int
+static
+int
 hammer2_vget(struct mount *mp, struct vnode *dvp,
             ino_t ino, struct vnode **vpp)
 {
@@ -354,7 +360,8 @@ hammer2_vget(struct mount *mp, struct vnode *dvp,
        return (EOPNOTSUPP);
 }
 
-static int
+static
+int
 hammer2_root(struct mount *mp, struct vnode **vpp)
 {
        struct hammer2_mount *hmp;
@@ -364,7 +371,7 @@ hammer2_root(struct mount *mp, struct vnode **vpp)
        kprintf("hammer2_root\n");
 
        hmp = MPTOH2(mp);
-       hammer2_mount_lock_ex(hmp);
+       hammer2_mount_exlock(hmp);
        if (hmp->hm_iroot == NULL) {
                *vpp = NULL;
                error = EINVAL;
@@ -379,7 +386,8 @@ hammer2_root(struct mount *mp, struct vnode **vpp)
        return (error);
 }
 
-static int
+static
+int
 hammer2_statfs(struct mount *mp, struct statfs *sbp, struct ucred *cred)
 {
        struct hammer2_mount *hmp;
@@ -402,7 +410,8 @@ hammer2_statfs(struct mount *mp, struct statfs *sbp, struct ucred *cred)
        return (0);
 }
 
-static int
+static
+int
 hammer2_statvfs(struct mount *mp, struct statvfs *sbp, struct ucred *cred)
 {
        kprintf("hammer2_statvfs\n");
@@ -425,33 +434,41 @@ hammer2_statvfs(struct mount *mp, struct statvfs *sbp, struct ucred *cred)
  *
  * THINKS: side A vs side B, to have sync not stall all I/O?
  */
-static int
+static
+int
 hammer2_sync(struct mount *mp, int waitfor)
 {
+#if 0
        struct hammer2_mount *hmp;
        struct hammer2_inode *ip;
+#endif
 
        kprintf("hammer2_sync \n");
 
-//     hmp = MPTOH2(mp);
+#if 0
+       hmp = MPTOH2(mp);
+#endif
 
        return (0);
 }
 
-static int
+static
+int
 hammer2_vptofh(struct vnode *vp, struct fid *fhp)
 {
        return (0);
 }
 
-static int
+static
+int
 hammer2_fhtovp(struct mount *mp, struct vnode *rootvp,
               struct fid *fhp, struct vnode **vpp)
 {
        return (0);
 }
 
-static int
+static
+int
 hammer2_checkexp(struct mount *mp, struct sockaddr *nam,
                 int *exflagsp, struct ucred **credanonp)
 {
index 1853a90..8737f77 100644 (file)
@@ -1,3 +1,37 @@
+/*
+ * Copyright (c) 2011-2012 The DragonFly Project.  All rights reserved.
+ *
+ * This code is derived from software contributed to The DragonFly Project
+ * by Matthew Dillon <dillon@dragonflybsd.org>
+ * by Venkatesh Srinivas <vsrinivas@dragonflybsd.org>
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ *
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ *    notice, this list of conditions and the following disclaimer in
+ *    the documentation and/or other materials provided with the
+ *    distribution.
+ * 3. Neither the name of The DragonFly Project nor the names of its
+ *    contributors may be used to endorse or promote products derived
+ *    from this software without specific, prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS
+ * FOR A PARTICULAR PURPOSE ARE DISCLAIMED.  IN NO EVENT SHALL THE
+ * COPYRIGHT HOLDERS OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT,
+ * INCIDENTAL, SPECIAL, EXEMPLARY OR CONSEQUENTIAL DAMAGES (INCLUDING,
+ * BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
+ * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED
+ * AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY,
+ * OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT
+ * OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
+ * SUCH DAMAGE.
+ */
 #include <sys/param.h>
 #include <sys/systm.h>
 #include <sys/kernel.h>
 
 #include "hammer2.h"
 
-
 /*
  * Last reference to a vnode is going away but it is still cached.
  */
+static
 int
-hammer2_inactive(struct vop_inactive_args *ap)
+hammer2_vop_inactive(struct vop_inactive_args *ap)
 {
        struct vnode *vp;
        struct hammer2_inode *ip;
+#if 0
        struct hammer2_mount *hmp;
+#endif
 
        kprintf("hammer2_inactive\n");
 
@@ -34,8 +70,9 @@ hammer2_inactive(struct vop_inactive_args *ap)
  * Reclaim a vnode so that it can be reused; after the inode is
  * disassociated, the filesystem must manage it alone.
  */
+static
 int
-hammer2_reclaim(struct vop_reclaim_args *ap)
+hammer2_vop_reclaim(struct vop_reclaim_args *ap)
 {
        struct vnode *vp;
        struct hammer2_inode *ip;
@@ -55,23 +92,25 @@ hammer2_reclaim(struct vop_reclaim_args *ap)
        return (0);
 }
 
-
+static
 int
-hammer2_fsync(struct vop_fsync_args *ap)
+hammer2_vop_fsync(struct vop_fsync_args *ap)
 {
        kprintf("hammer2_fsync\n");
        return (EOPNOTSUPP);
 }
 
+static
 int
-hammer2_access(struct vop_access_args *ap)
+hammer2_vop_access(struct vop_access_args *ap)
 {
        kprintf("hammer2_access\n");
        return (0);
 }
 
+static
 int
-hammer2_getattr(struct vop_getattr_args *ap)
+hammer2_vop_getattr(struct vop_getattr_args *ap)
 {
        struct vnode *vp;
        struct vattr *vap;
@@ -99,48 +138,55 @@ hammer2_getattr(struct vop_getattr_args *ap)
        return (0);
 }
 
+static
 int
-hammer2_readdir(struct vop_readdir_args *ap)
+hammer2_vop_readdir(struct vop_readdir_args *ap)
 {
        kprintf("hammer2_readdir\n");
        return (EOPNOTSUPP);
 }
 
+static
 int
-hammer2_read(struct vop_read_args *ap)
+hammer2_vop_read(struct vop_read_args *ap)
 {
        return (EOPNOTSUPP);
 }
 
+static
 int
-hammer2_write(struct vop_write_args *ap)
+hammer2_vop_write(struct vop_write_args *ap)
 {
        return (EOPNOTSUPP);
 }
 
+static
 int
-hammer2_nresolve(struct vop_nresolve_args *ap)
+hammer2_vop_nresolve(struct vop_nresolve_args *ap)
 {
        kprintf("hammer2_nresolve\n");
        return (EOPNOTSUPP);
 }
 
+static
 int
-hammer2_bmap(struct vop_bmap_args *ap)
+hammer2_vop_bmap(struct vop_bmap_args *ap)
 {
        kprintf("hammer2_bmap\n");
        return (EOPNOTSUPP);
 }
 
+static
 int
-hammer2_open(struct vop_open_args *ap)
+hammer2_vop_open(struct vop_open_args *ap)
 {
        kprintf("hammer2_open\n");
        return vop_stdopen(ap);
 }
 
+static
 int
-hammer2_strategy(struct vop_strategy_args *ap)
+hammer2_vop_strategy(struct vop_strategy_args *ap)
 {
        struct vnode *vp;
        struct bio *biop;
@@ -166,8 +212,9 @@ hammer2_strategy(struct vop_strategy_args *ap)
        return (error);
 }
 
+static
 int 
-hammer2_mountctl(struct vop_mountctl_args *ap)
+hammer2_vop_mountctl(struct vop_mountctl_args *ap)
 {
        struct mount *mp;
        struct hammer2_mount *hmp;
@@ -193,21 +240,21 @@ hammer2_mountctl(struct vop_mountctl_args *ap)
 
 struct vop_ops hammer2_vnode_vops = {
        .vop_default    = vop_defaultop,
-       .vop_fsync      = hammer2_fsync,
+       .vop_fsync      = hammer2_vop_fsync,
        .vop_getpages   = vop_stdgetpages,
        .vop_putpages   = vop_stdputpages,
-       .vop_access     = hammer2_access,
-       .vop_getattr    = hammer2_getattr,
-       .vop_readdir    = hammer2_readdir,
-       .vop_read       = hammer2_read,
-       .vop_write      = hammer2_write,
-       .vop_open       = hammer2_open,
-       .vop_inactive   = hammer2_inactive,
-       .vop_reclaim    = hammer2_reclaim,
-       .vop_nresolve   = hammer2_nresolve,
-       .vop_mountctl   = hammer2_mountctl,
-       .vop_bmap       = hammer2_bmap,
-       .vop_strategy   = hammer2_strategy,
+       .vop_access     = hammer2_vop_access,
+       .vop_getattr    = hammer2_vop_getattr,
+       .vop_readdir    = hammer2_vop_readdir,
+       .vop_read       = hammer2_vop_read,
+       .vop_write      = hammer2_vop_write,
+       .vop_open       = hammer2_vop_open,
+       .vop_inactive   = hammer2_vop_inactive,
+       .vop_reclaim    = hammer2_vop_reclaim,
+       .vop_nresolve   = hammer2_vop_nresolve,
+       .vop_mountctl   = hammer2_vop_mountctl,
+       .vop_bmap       = hammer2_vop_bmap,
+       .vop_strategy   = hammer2_vop_strategy,
 };
 
 struct vop_ops hammer2_spec_vops = {