Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files. =================================================================== RCS file: /ftp/cvs/cvsroot/src/sys/ufs/ffs/ffs_vnops.c,v rcsdiff: /ftp/cvs/cvsroot/src/sys/ufs/ffs/ffs_vnops.c,v: warning: Unknown phrases like `commitid ...;' are present. retrieving revision 1.31.2.1 retrieving revision 1.77.8.3 diff -u -p -r1.31.2.1 -r1.77.8.3 --- src/sys/ufs/ffs/ffs_vnops.c 2000/12/14 23:36:48 1.31.2.1 +++ src/sys/ufs/ffs/ffs_vnops.c 2006/05/24 10:59:25 1.77.8.3 @@ -1,4 +1,4 @@ -/* $NetBSD: ffs_vnops.c,v 1.31.2.1 2000/12/14 23:36:48 he Exp $ */ +/* $NetBSD: ffs_vnops.c,v 1.77.8.3 2006/05/24 10:59:25 yamt Exp $ */ /* * Copyright (c) 1982, 1986, 1989, 1993 @@ -12,11 +12,7 @@ * 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. All advertising materials mentioning features or use of this software - * must display the following acknowledgement: - * This product includes software developed by the University of - * California, Berkeley and its contributors. - * 4. Neither the name of the University nor the names of its contributors + * 3. Neither the name of the University nor the names of its contributors * may be used to endorse or promote products derived from this software * without specific prior written permission. * @@ -35,6 +31,9 @@ * @(#)ffs_vnops.c 8.15 (Berkeley) 5/14/95 */ +#include +__KERNEL_RCSID(0, "$NetBSD: ffs_vnops.c,v 1.77.8.3 2006/05/24 10:59:25 yamt Exp $"); + #include #include #include @@ -42,21 +41,18 @@ #include #include #include +#include #include #include #include #include #include - -#include - -#include +#include #include #include #include -#include #include #include #include @@ -65,11 +61,13 @@ #include #include -static int ffs_full_fsync __P((void *)); +#include + +static int ffs_full_fsync(void *); /* Global vfs data structures for ufs. */ -int (**ffs_vnodeop_p) __P((void *)); -struct vnodeopv_entry_desc ffs_vnodeop_entries[] = { +int (**ffs_vnodeop_p)(void *); +const struct vnodeopv_entry_desc ffs_vnodeop_entries[] = { { &vop_default_desc, vn_default_error }, { &vop_lookup_desc, ufs_lookup }, /* lookup */ { &vop_create_desc, ufs_create }, /* create */ @@ -86,6 +84,7 @@ struct vnodeopv_entry_desc ffs_vnodeop_e { &vop_ioctl_desc, ufs_ioctl }, /* ioctl */ { &vop_fcntl_desc, ufs_fcntl }, /* fcntl */ { &vop_poll_desc, ufs_poll }, /* poll */ + { &vop_kqfilter_desc, genfs_kqfilter }, /* kqfilter */ { &vop_revoke_desc, ufs_revoke }, /* revoke */ { &vop_mmap_desc, ufs_mmap }, /* mmap */ { &vop_fsync_desc, ffs_fsync }, /* fsync */ @@ -109,21 +108,22 @@ struct vnodeopv_entry_desc ffs_vnodeop_e { &vop_islocked_desc, ufs_islocked }, /* islocked */ { &vop_pathconf_desc, ufs_pathconf }, /* pathconf */ { &vop_advlock_desc, ufs_advlock }, /* advlock */ - { &vop_blkatoff_desc, ffs_blkatoff }, /* blkatoff */ - { &vop_valloc_desc, ffs_valloc }, /* valloc */ - { &vop_balloc_desc, ffs_balloc }, /* balloc */ - { &vop_reallocblks_desc, ffs_reallocblks }, /* reallocblks */ - { &vop_vfree_desc, ffs_vfree }, /* vfree */ - { &vop_truncate_desc, ffs_truncate }, /* truncate */ - { &vop_update_desc, ffs_update }, /* update */ { &vop_bwrite_desc, vn_bwrite }, /* bwrite */ - { (struct vnodeop_desc*)NULL, (int(*) __P((void*)))NULL } + { &vop_getpages_desc, ffs_getpages }, /* getpages */ + { &vop_putpages_desc, genfs_putpages }, /* putpages */ + { &vop_openextattr_desc, ffs_openextattr }, /* openextattr */ + { &vop_closeextattr_desc, ffs_closeextattr }, /* closeextattr */ + { &vop_getextattr_desc, ffs_getextattr }, /* getextattr */ + { &vop_setextattr_desc, ffs_setextattr }, /* setextattr */ + { &vop_listextattr_desc, ffs_listextattr }, /* listextattr */ + { &vop_deleteextattr_desc, ffs_deleteextattr }, /* deleteextattr */ + { NULL, NULL } }; -struct vnodeopv_desc ffs_vnodeop_opv_desc = +const struct vnodeopv_desc ffs_vnodeop_opv_desc = { &ffs_vnodeop_p, ffs_vnodeop_entries }; -int (**ffs_specop_p) __P((void *)); -struct vnodeopv_entry_desc ffs_specop_entries[] = { +int (**ffs_specop_p)(void *); +const struct vnodeopv_entry_desc ffs_specop_entries[] = { { &vop_default_desc, vn_default_error }, { &vop_lookup_desc, spec_lookup }, /* lookup */ { &vop_create_desc, spec_create }, /* create */ @@ -139,6 +139,7 @@ struct vnodeopv_entry_desc ffs_specop_en { &vop_ioctl_desc, spec_ioctl }, /* ioctl */ { &vop_fcntl_desc, ufs_fcntl }, /* fcntl */ { &vop_poll_desc, spec_poll }, /* poll */ + { &vop_kqfilter_desc, spec_kqfilter }, /* kqfilter */ { &vop_revoke_desc, spec_revoke }, /* revoke */ { &vop_mmap_desc, spec_mmap }, /* mmap */ { &vop_fsync_desc, ffs_fsync }, /* fsync */ @@ -162,20 +163,22 @@ struct vnodeopv_entry_desc ffs_specop_en { &vop_islocked_desc, ufs_islocked }, /* islocked */ { &vop_pathconf_desc, spec_pathconf }, /* pathconf */ { &vop_advlock_desc, spec_advlock }, /* advlock */ - { &vop_blkatoff_desc, spec_blkatoff }, /* blkatoff */ - { &vop_valloc_desc, spec_valloc }, /* valloc */ - { &vop_reallocblks_desc, spec_reallocblks }, /* reallocblks */ - { &vop_vfree_desc, ffs_vfree }, /* vfree */ - { &vop_truncate_desc, spec_truncate }, /* truncate */ - { &vop_update_desc, ffs_update }, /* update */ { &vop_bwrite_desc, vn_bwrite }, /* bwrite */ - { (struct vnodeop_desc*)NULL, (int(*) __P((void *)))NULL } + { &vop_getpages_desc, spec_getpages }, /* getpages */ + { &vop_putpages_desc, spec_putpages }, /* putpages */ + { &vop_openextattr_desc, ffs_openextattr }, /* openextattr */ + { &vop_closeextattr_desc, ffs_closeextattr }, /* closeextattr */ + { &vop_getextattr_desc, ffs_getextattr }, /* getextattr */ + { &vop_setextattr_desc, ffs_setextattr }, /* setextattr */ + { &vop_listextattr_desc, ffs_listextattr }, /* listextattr */ + { &vop_deleteextattr_desc, ffs_deleteextattr }, /* deleteextattr */ + { NULL, NULL } }; -struct vnodeopv_desc ffs_specop_opv_desc = +const struct vnodeopv_desc ffs_specop_opv_desc = { &ffs_specop_p, ffs_specop_entries }; -int (**ffs_fifoop_p) __P((void *)); -struct vnodeopv_entry_desc ffs_fifoop_entries[] = { +int (**ffs_fifoop_p)(void *); +const struct vnodeopv_entry_desc ffs_fifoop_entries[] = { { &vop_default_desc, vn_default_error }, { &vop_lookup_desc, fifo_lookup }, /* lookup */ { &vop_create_desc, fifo_create }, /* create */ @@ -191,6 +194,7 @@ struct vnodeopv_entry_desc ffs_fifoop_en { &vop_ioctl_desc, fifo_ioctl }, /* ioctl */ { &vop_fcntl_desc, ufs_fcntl }, /* fcntl */ { &vop_poll_desc, fifo_poll }, /* poll */ + { &vop_kqfilter_desc, fifo_kqfilter }, /* kqfilter */ { &vop_revoke_desc, fifo_revoke }, /* revoke */ { &vop_mmap_desc, fifo_mmap }, /* mmap */ { &vop_fsync_desc, ffs_fsync }, /* fsync */ @@ -214,105 +218,115 @@ struct vnodeopv_entry_desc ffs_fifoop_en { &vop_islocked_desc, ufs_islocked }, /* islocked */ { &vop_pathconf_desc, fifo_pathconf }, /* pathconf */ { &vop_advlock_desc, fifo_advlock }, /* advlock */ - { &vop_blkatoff_desc, fifo_blkatoff }, /* blkatoff */ - { &vop_valloc_desc, fifo_valloc }, /* valloc */ - { &vop_reallocblks_desc, fifo_reallocblks }, /* reallocblks */ - { &vop_vfree_desc, ffs_vfree }, /* vfree */ - { &vop_truncate_desc, fifo_truncate }, /* truncate */ - { &vop_update_desc, ffs_update }, /* update */ { &vop_bwrite_desc, vn_bwrite }, /* bwrite */ - { (struct vnodeop_desc*)NULL, (int(*) __P((void *)))NULL } + { &vop_putpages_desc, fifo_putpages }, /* putpages */ + { &vop_openextattr_desc, ffs_openextattr }, /* openextattr */ + { &vop_closeextattr_desc, ffs_closeextattr }, /* closeextattr */ + { &vop_getextattr_desc, ffs_getextattr }, /* getextattr */ + { &vop_setextattr_desc, ffs_setextattr }, /* setextattr */ + { &vop_listextattr_desc, ffs_listextattr }, /* listextattr */ + { &vop_deleteextattr_desc, ffs_deleteextattr }, /* deleteextattr */ + { NULL, NULL } }; -struct vnodeopv_desc ffs_fifoop_opv_desc = +const struct vnodeopv_desc ffs_fifoop_opv_desc = { &ffs_fifoop_p, ffs_fifoop_entries }; -int doclusterread = 1; -int doclusterwrite = 1; - #include int -ffs_fsync(v) - void *v; +ffs_fsync(void *v) { struct vop_fsync_args /* { struct vnode *a_vp; - struct ucred *a_cred; + kauth_cred_t a_cred; int a_flags; - off_t offlo; - off_t offhi; - struct proc *a_p; + off_t a_offlo; + off_t a_offhi; + struct lwp *a_l; } */ *ap = v; - struct buf *bp, *nbp, *ibp; + struct buf *bp; int s, num, error, i; struct indir ia[NIADDR + 1]; int bsize; - daddr_t blk_low, blk_high; + daddr_t blk_high; struct vnode *vp; /* * XXX no easy way to sync a range in a file with softdep. */ - if ((ap->a_offlo == 0 && ap->a_offhi == 0) || DOINGSOFTDEP(ap->a_vp)) + if ((ap->a_offlo == 0 && ap->a_offhi == 0) || DOINGSOFTDEP(ap->a_vp) || + (ap->a_vp->v_type != VREG)) return ffs_full_fsync(v); vp = ap->a_vp; bsize = ap->a_vp->v_mount->mnt_stat.f_iosize; - blk_low = ap->a_offlo / bsize; blk_high = ap->a_offhi / bsize; if (ap->a_offhi % bsize != 0) blk_high++; /* - * First, flush all data blocks in range. + * First, flush all pages in range. */ -loop: - s = splbio(); - for (bp = LIST_FIRST(&vp->v_dirtyblkhd); bp; bp = nbp) { - nbp = LIST_NEXT(bp, b_vnbufs); - if ((bp->b_flags & B_BUSY)) - continue; - if (bp->b_lblkno < blk_low || bp->b_lblkno > blk_high) - continue; - bp->b_flags |= B_BUSY | B_VFLUSH; - splx(s); - bawrite(bp); - goto loop; + + simple_lock(&vp->v_interlock); + error = VOP_PUTPAGES(vp, trunc_page(ap->a_offlo), + round_page(ap->a_offhi), PGO_CLEANIT | + ((ap->a_flags & FSYNC_WAIT) ? PGO_SYNCIO : 0)); + if (error) { + return error; } /* - * Then, flush possibly unwritten indirect blocks. Without softdeps, - * these should be the only ones left. + * Then, flush indirect blocks. */ - if (!(ap->a_flags & FSYNC_DATAONLY) && blk_high >= NDADDR) { + + s = splbio(); + if (blk_high >= NDADDR) { error = ufs_getlbns(vp, blk_high, ia, &num); - if (error != 0) + if (error) { + splx(s); return error; + } for (i = 0; i < num; i++) { - ibp = incore(vp, ia[i].in_lbn); - if (ibp != NULL && !(ibp->b_flags & B_BUSY) && - (ibp->b_flags & B_DELWRI)) { - ibp->b_flags |= B_BUSY | B_VFLUSH; - splx(s); - bawrite(ibp); - s = splbio(); + bp = incore(vp, ia[i].in_lbn); + if (bp != NULL) { + simple_lock(&bp->b_interlock); + if (!(bp->b_flags & B_BUSY) && (bp->b_flags & B_DELWRI)) { + bp->b_flags |= B_BUSY | B_VFLUSH; + simple_unlock(&bp->b_interlock); + splx(s); + bawrite(bp); + s = splbio(); + } else { + simple_unlock(&bp->b_interlock); + } } } } if (ap->a_flags & FSYNC_WAIT) { + simple_lock(&global_v_numoutput_slock); while (vp->v_numoutput > 0) { vp->v_flag |= VBWAIT; - tsleep((caddr_t)&vp->v_numoutput, PRIBIO + 1, - "fsync_range", 0); + ltsleep(&vp->v_numoutput, PRIBIO + 1, "fsync_range", 0, + &global_v_numoutput_slock); } + simple_unlock(&global_v_numoutput_slock); } - splx(s); - return (VOP_UPDATE(vp, NULL, NULL, - (ap->a_flags & FSYNC_WAIT) ? UPDATE_WAIT : 0)); + error = ffs_update(vp, NULL, NULL, + ((ap->a_flags & (FSYNC_WAIT | FSYNC_DATAONLY)) == FSYNC_WAIT) + ? UPDATE_WAIT : 0); + + if (error == 0 && ap->a_flags & FSYNC_CACHE) { + int l = 0; + VOP_IOCTL(VTOI(vp)->i_devvp, DIOCCACHESYNC, &l, FWRITE, + ap->a_l->l_proc->p_cred, ap->a_l); + } + + return error; } /* @@ -320,46 +334,64 @@ loop: */ /* ARGSUSED */ static int -ffs_full_fsync(v) - void *v; +ffs_full_fsync(void *v) { struct vop_fsync_args /* { struct vnode *a_vp; - struct ucred *a_cred; + kauth_cred_t a_cred; int a_flags; - off_t offlo; - off_t offhi; - struct proc *a_p; + off_t a_offlo; + off_t a_offhi; + struct lwp *a_l; } */ *ap = v; struct vnode *vp = ap->a_vp; struct buf *bp, *nbp; - int s, error, passes, skipmeta; + int s, error, passes, skipmeta, inodedeps_only, waitfor; if (vp->v_type == VBLK && vp->v_specmountpoint != NULL && (vp->v_specmountpoint->mnt_flag & MNT_SOFTDEP)) softdep_fsync_mountdev(vp); - /* - * Flush all dirty buffers associated with a vnode + inodedeps_only = DOINGSOFTDEP(vp) && (ap->a_flags & FSYNC_RECLAIM) + && vp->v_uobj.uo_npages == 0 && LIST_EMPTY(&vp->v_dirtyblkhd); + + /* + * Flush all dirty data associated with a vnode. */ + + if (vp->v_type == VREG || vp->v_type == VBLK) { + simple_lock(&vp->v_interlock); + error = VOP_PUTPAGES(vp, 0, 0, PGO_ALLPAGES | PGO_CLEANIT | + ((ap->a_flags & FSYNC_WAIT) ? PGO_SYNCIO : 0)); + if (error) { + return error; + } + } + passes = NIADDR + 1; skipmeta = 0; - if (ap->a_flags & (FSYNC_DATAONLY|FSYNC_WAIT)) + if (ap->a_flags & FSYNC_WAIT) skipmeta = 1; s = splbio(); + loop: - for (bp = LIST_FIRST(&vp->v_dirtyblkhd); bp; - bp = LIST_NEXT(bp, b_vnbufs)) + LIST_FOREACH(bp, &vp->v_dirtyblkhd, b_vnbufs) bp->b_flags &= ~B_SCANNED; for (bp = LIST_FIRST(&vp->v_dirtyblkhd); bp; bp = nbp) { nbp = LIST_NEXT(bp, b_vnbufs); - if (bp->b_flags & (B_BUSY | B_SCANNED)) + simple_lock(&bp->b_interlock); + if (bp->b_flags & (B_BUSY | B_SCANNED)) { + simple_unlock(&bp->b_interlock); continue; + } if ((bp->b_flags & B_DELWRI) == 0) panic("ffs_fsync: not dirty"); - if (skipmeta && bp->b_lblkno < 0) + if (skipmeta && bp->b_lblkno < 0) { + simple_unlock(&bp->b_interlock); continue; + } + simple_unlock(&bp->b_interlock); bp->b_flags |= B_BUSY | B_VFLUSH | B_SCANNED; splx(s); /* @@ -378,22 +410,21 @@ loop: */ nbp = LIST_FIRST(&vp->v_dirtyblkhd); } - if (skipmeta && !(ap->a_flags & FSYNC_DATAONLY)) { + if (skipmeta) { skipmeta = 0; goto loop; } if (ap->a_flags & FSYNC_WAIT) { + simple_lock(&global_v_numoutput_slock); while (vp->v_numoutput) { vp->v_flag |= VBWAIT; - (void) tsleep(&vp->v_numoutput, PRIBIO + 1, - "ffsfsync", 0); + (void) ltsleep(&vp->v_numoutput, PRIBIO + 1, + "ffsfsync", 0, &global_v_numoutput_slock); } + simple_unlock(&global_v_numoutput_slock); splx(s); - if (ap->a_flags & FSYNC_DATAONLY) - return (0); - - /* + /* * Ensure that any filesystem metadata associated * with the vnode has been written. */ @@ -421,26 +452,45 @@ loop: } } splx(s); - return (VOP_UPDATE(vp, NULL, NULL, - (ap->a_flags & FSYNC_WAIT) ? UPDATE_WAIT : 0)); + + if (inodedeps_only) + waitfor = 0; + else + waitfor = (ap->a_flags & FSYNC_WAIT) ? UPDATE_WAIT : 0; + error = ffs_update(vp, NULL, NULL, waitfor); + + if (error == 0 && ap->a_flags & FSYNC_CACHE) { + int i = 0; + VOP_IOCTL(VTOI(vp)->i_devvp, DIOCCACHESYNC, &i, FWRITE, + ap->a_l->l_proc->p_cred, ap->a_l); + } + + return error; } /* * Reclaim an inode so that it can be used for other purposes. */ int -ffs_reclaim(v) - void *v; +ffs_reclaim(void *v) { struct vop_reclaim_args /* { struct vnode *a_vp; - struct proc *a_p; + struct lwp *a_l; } */ *ap = v; struct vnode *vp = ap->a_vp; + struct inode *ip = VTOI(vp); + struct ufsmount *ump = ip->i_ump; int error; - if ((error = ufs_reclaim(vp, ap->a_p)) != 0) + if ((error = ufs_reclaim(vp, ap->a_l)) != 0) return (error); + if (ip->i_din.ffs1_din != NULL) { + if (ump->um_fstype == UFS1) + pool_put(&ffs_dinode1_pool, ip->i_din.ffs1_din); + else + pool_put(&ffs_dinode2_pool, ip->i_din.ffs2_din); + } /* * XXX MFS ends up here, too, to free an inode. Should we create * XXX a separate pool for MFS inodes? @@ -449,3 +499,197 @@ ffs_reclaim(v) vp->v_data = NULL; return (0); } + +int +ffs_getpages(void *v) +{ + struct vop_getpages_args /* { + struct vnode *a_vp; + voff_t a_offset; + struct vm_page **a_m; + int *a_count; + int a_centeridx; + vm_prot_t a_access_type; + int a_advice; + int a_flags; + } */ *ap = v; + struct vnode *vp = ap->a_vp; + struct inode *ip = VTOI(vp); + struct fs *fs = ip->i_fs; + + /* + * don't allow a softdep write to create pages for only part of a block. + * the dependency tracking requires that all pages be in memory for + * a block involved in a dependency. + */ + + if (ap->a_flags & PGO_OVERWRITE && + (blkoff(fs, ap->a_offset) != 0 || + blkoff(fs, *ap->a_count << PAGE_SHIFT) != 0) && + DOINGSOFTDEP(ap->a_vp)) { + if ((ap->a_flags & PGO_LOCKED) == 0) { + simple_unlock(&vp->v_interlock); + } + return EINVAL; + } + return genfs_getpages(v); +} + +/* + * Return the last logical file offset that should be written for this file + * if we're doing a write that ends at "size". + */ + +void +ffs_gop_size(struct vnode *vp, off_t size, off_t *eobp, int flags) +{ + struct inode *ip = VTOI(vp); + struct fs *fs = ip->i_fs; + daddr_t olbn, nlbn; + + olbn = lblkno(fs, ip->i_size); + nlbn = lblkno(fs, size); + if (nlbn < NDADDR && olbn <= nlbn) { + *eobp = fragroundup(fs, size); + } else { + *eobp = blkroundup(fs, size); + } +} + +int +ffs_openextattr(void *v) +{ + struct vop_openextattr_args /* { + struct vnode *a_vp; + kauth_cred_t a_cred; + struct proc *a_p; + } */ *ap = v; + struct inode *ip = VTOI(ap->a_vp); + struct fs *fs = ip->i_fs; + + /* Not supported for UFS1 file systems. */ + if (fs->fs_magic == FS_UFS1_MAGIC) + return (EOPNOTSUPP); + + /* XXX Not implemented for UFS2 file systems. */ + return (EOPNOTSUPP); +} + +int +ffs_closeextattr(void *v) +{ + struct vop_closeextattr_args /* { + struct vnode *a_vp; + int a_commit; + kauth_cred_t a_cred; + struct proc *a_p; + } */ *ap = v; + struct inode *ip = VTOI(ap->a_vp); + struct fs *fs = ip->i_fs; + + /* Not supported for UFS1 file systems. */ + if (fs->fs_magic == FS_UFS1_MAGIC) + return (EOPNOTSUPP); + + /* XXX Not implemented for UFS2 file systems. */ + return (EOPNOTSUPP); +} + +int +ffs_getextattr(void *v) +{ + struct vop_getextattr_args /* { + struct vnode *a_vp; + int a_attrnamespace; + const char *a_name; + struct uio *a_uio; + size_t *a_size; + kauth_cred_t a_cred; + struct proc *a_p; + } */ *ap = v; + struct inode *ip = VTOI(ap->a_vp); + struct fs *fs = ip->i_fs; + + if (fs->fs_magic == FS_UFS1_MAGIC) { +#ifdef UFS_EXTATTR + return (ufs_getextattr(ap)); +#else + return (EOPNOTSUPP); +#endif + } + + /* XXX Not implemented for UFS2 file systems. */ + return (EOPNOTSUPP); +} + +int +ffs_setextattr(void *v) +{ + struct vop_setextattr_args /* { + struct vnode *a_vp; + int a_attrnamespace; + const char *a_name; + struct uio *a_uio; + kauth_cred_t a_cred; + struct proc *a_p; + } */ *ap = v; + struct inode *ip = VTOI(ap->a_vp); + struct fs *fs = ip->i_fs; + + if (fs->fs_magic == FS_UFS1_MAGIC) { +#ifdef UFS_EXTATTR + return (ufs_setextattr(ap)); +#else + return (EOPNOTSUPP); +#endif + } + + /* XXX Not implemented for UFS2 file systems. */ + return (EOPNOTSUPP); +} + +int +ffs_listextattr(void *v) +{ + struct vop_listextattr_args /* { + struct vnode *a_vp; + int a_attrnamespace; + struct uio *a_uio; + size_t *a_size; + kauth_cred_t a_cred; + struct proc *a_p; + } */ *ap = v; + struct inode *ip = VTOI(ap->a_vp); + struct fs *fs = ip->i_fs; + + /* Not supported for UFS1 file systems. */ + if (fs->fs_magic == FS_UFS1_MAGIC) + return (EOPNOTSUPP); + + /* XXX Not implemented for UFS2 file systems. */ + return (EOPNOTSUPP); +} + +int +ffs_deleteextattr(void *v) +{ + struct vop_deleteextattr_args /* { + struct vnode *a_vp; + int a_attrnamespace; + kauth_cred_t a_cred; + struct proc *a_p; + } */ *ap = v; + struct inode *ip = VTOI(ap->a_vp); + struct fs *fs = ip->i_fs; + + if (fs->fs_magic == FS_UFS1_MAGIC) { +#ifdef UFS_EXTATTR + return (ufs_deleteextattr(ap)); +#else + return (EOPNOTSUPP); +#endif + } + + /* XXX Not implemented for UFS2 file systems. */ + return (EOPNOTSUPP); +}