On Mon, Jul 17, 2023 at 11:05:03AM +0200, Sebastien Marie wrote:
> On Wed, Jul 12, 2023 at 12:26:01PM +0200, thib4711 wrote:
> > make it obvious in the vfsops assignment that an op isnt supported.
>
> I agree that it is more readable.
>
> ok semarie@
OK claudio@ as well. Semarie can you commit this?
> thanks.
> --
> Sebastien Marie
>
> > diff --git sys/isofs/cd9660/cd9660_extern.h sys/isofs/cd9660/cd9660_extern.h
> > index 2a5348e1768..bd8154a27bd 100644
> > --- sys/isofs/cd9660/cd9660_extern.h
> > +++ sys/isofs/cd9660/cd9660_extern.h
> > @@ -94,10 +94,8 @@ int cd9660_vptofh(struct vnode *, struct fid *);
> > int cd9660_init(struct vfsconf *);
> > int cd9660_check_export(struct mount *, struct mbuf *, int *,
> > struct ucred **);
> > -#define cd9660_sysctl ((int (*)(int *, u_int, void *, size_t *, void *, \
> > - size_t, struct proc *))eopnotsupp)
> >
> > -int cd9660_mountroot(void);
> > +int cd9660_mountroot(void);
> >
> > extern const struct vops cd9660_vops;
> > extern const struct vops cd9660_specvops;
> > diff --git sys/isofs/cd9660/cd9660_vfsops.c sys/isofs/cd9660/cd9660_vfsops.c
> > index ef0ffbbb152..b844a2ff709 100644
> > --- sys/isofs/cd9660/cd9660_vfsops.c
> > +++ sys/isofs/cd9660/cd9660_vfsops.c
> > @@ -72,7 +72,7 @@ const struct vfsops cd9660_vfsops = {
> > .vfs_fhtovp = cd9660_fhtovp,
> > .vfs_vptofh = cd9660_vptofh,
> > .vfs_init = cd9660_init,
> > - .vfs_sysctl = cd9660_sysctl,
> > + .vfs_sysctl = (void *)eopnotsupp,
> > .vfs_checkexp = cd9660_check_export,
> > };
> >
> > diff --git sys/msdosfs/msdosfs_vfsops.c sys/msdosfs/msdosfs_vfsops.c
> > index 0de37665dfd..6b90195b5e5 100644
> > --- sys/msdosfs/msdosfs_vfsops.c
> > +++ sys/msdosfs/msdosfs_vfsops.c
> > @@ -762,27 +762,18 @@ msdosfs_check_export(struct mount *mp, struct mbuf
> > *nam, int *exflagsp,
> > return (0);
> > }
> >
> > -#define msdosfs_vget ((int (*)(struct mount *, ino_t, struct vnode **)) \
> > - eopnotsupp)
> > -
> > -#define msdosfs_quotactl ((int (*)(struct mount *, int, uid_t, caddr_t, \
> > - struct proc *))eopnotsupp)
> > -
> > -#define msdosfs_sysctl ((int (*)(int *, u_int, void *, size_t *, void *, \
> > - size_t, struct proc *))eopnotsupp)
> > -
> > const struct vfsops msdosfs_vfsops = {
> > .vfs_mount = msdosfs_mount,
> > .vfs_start = msdosfs_start,
> > .vfs_unmount = msdosfs_unmount,
> > .vfs_root = msdosfs_root,
> > - .vfs_quotactl = msdosfs_quotactl,
> > + .vfs_quotactl = (void *)eopnotsupp,
> > .vfs_statfs = msdosfs_statfs,
> > .vfs_sync = msdosfs_sync,
> > - .vfs_vget = msdosfs_vget,
> > + .vfs_vget = (void *)eopnotsupp,
> > .vfs_fhtovp = msdosfs_fhtovp,
> > .vfs_vptofh = msdosfs_vptofh,
> > .vfs_init = msdosfs_init,
> > - .vfs_sysctl = msdosfs_sysctl,
> > + .vfs_sysctl = (void *)eopnotsupp,
> > .vfs_checkexp = msdosfs_check_export,
> > };
> >
>
--
:wq Claudio
