All unimplemented VFS ops now have entries in kern/vfs_default.c that return

reasonable defaults.

This avoids confusing and ugly casting to eopnotsupp or making dummy functions.
Bogus casting of filesystem sysctls to eopnotsupp() have been removed.

This should make *_vfsops.c more readable and reduce bloat.

Reviewed by:	msmith, eivind
Approved by:	phk
Tested by:	Jeroen Ruigrok/Asmodai <asmodai@wxs.nl>

1 files changed, 4 insertions, 50 deletions

diff --git a/sys/fs/cd9660/cd9660_vfsops.c b/sys/fs/cd9660/cd9660_vfsops.c
index 05615c3..f81cb14 100644
--- a/sys/fs/cd9660/cd9660_vfsops.c
+++ b/sys/fs/cd9660/cd9660_vfsops.c
@@ -64,14 +64,9 @@ MALLOC_DEFINE(M_ISOFSNODE, "ISOFS node", "ISOFS vnode private part");
 
 static int cd9660_mount __P((struct mount *,
 	    char *, caddr_t, struct nameidata *, struct proc *));
-static int cd9660_start __P((struct mount *, int, struct proc *));
 static int cd9660_unmount __P((struct mount *, int, struct proc *));
 static int cd9660_root __P((struct mount *, struct vnode **));
-static int cd9660_quotactl __P((struct mount *, int, uid_t, caddr_t, 
-	    struct proc *));
 static int cd9660_statfs __P((struct mount *, struct statfs *, struct proc *));
-static int cd9660_sync __P((struct mount *, int, struct ucred *, 
-	    struct proc *));
 static int cd9660_vget __P((struct mount *, ino_t, struct vnode **));
 static int cd9660_fhtovp __P((struct mount *, struct fid *, struct sockaddr *,
 	    struct vnode **, int *, struct ucred **));
@@ -79,16 +74,16 @@ static int cd9660_vptofh __P((struct vnode *, struct fid *));
 
 static struct vfsops cd9660_vfsops = {
 	cd9660_mount,
-	cd9660_start,
+	vfs_stdstart,
 	cd9660_unmount,
 	cd9660_root,
-	cd9660_quotactl,
+	vfs_stdquotactl,
 	cd9660_statfs,
-	cd9660_sync,
+	vfs_stdsync,
 	cd9660_vget,
 	cd9660_fhtovp,
 	cd9660_vptofh,
-	cd9660_init
+	cd9660_init,
 };
 VFS_SET(cd9660_vfsops, cd9660, VFCF_READONLY);
 
@@ -524,20 +519,6 @@ out:
 }
 
 /*
- * Make a filesystem operational.
- * Nothing to do at the moment.
- */
-/* ARGSUSED */
-static int
-cd9660_start(mp, flags, p)
-	struct mount *mp;
-	int flags;
-	struct proc *p;
-{
-	return 0;
-}
-
-/*
  * unmount system call
  */
 static int
@@ -593,22 +574,6 @@ cd9660_root(mp, vpp)
 }
 
 /*
- * Do operations associated with quotas, not supported
- */
-/* ARGSUSED */
-static int
-cd9660_quotactl(mp, cmd, uid, arg, p)
-	struct mount *mp;
-	int cmd;
-	uid_t uid;
-	caddr_t arg;
-	struct proc *p;
-{
-
-	return (EOPNOTSUPP);
-}
-
-/*
  * Get file system statistics.
  */
 int
@@ -636,17 +601,6 @@ cd9660_statfs(mp, sbp, p)
 	return 0;
 }
 
-/* ARGSUSED */
-static int
-cd9660_sync(mp, waitfor, cred, p)
-	struct mount *mp;
-	int waitfor;
-	struct ucred *cred;
-	struct proc *p;
-{
-	return (0);
-}
-
 /*
  * File handle to vnode
  *