Date: Thu, 6 Jun 2019 10:35:16 -0700 (PDT) From: "Rodney W. Grimes" <freebsd@gndrsh.dnsmgr.net> To: Alan Somers <asomers@freebsd.org> Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r348737 - head/sys/kern Message-ID: <201906061735.x56HZGIJ058845@gndrsh.dnsmgr.net> In-Reply-To: <201906061504.x56F4odw034764@repo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
> Author: asomers > Date: Thu Jun 6 15:04:50 2019 > New Revision: 348737 > URL: https://svnweb.freebsd.org/changeset/base/348737 > > Log: > Add a testing facility to manually reclaim a vnode > > Add the debug.try_reclaim_vnode sysctl. When a pathname is written to it, it > will be reclaimed, as long as it isn't already or doomed. The purpose is to > gain test coverage for vnode reclamation, which is otherwise hard to > achieve. > > Add the debug.ftry_reclaim_vnode sysctl. It does the same thing, except > that its argument is a file descriptor instead of a pathname. Should not this all be wrapped in some #ifdef or other protection, is it really a good idea to have this on every single box running FreeBSD? > > Reviewed by: kib > MFC after: 2 weeks > Sponsored by: The FreeBSD Foundation > Differential Revision: https://reviews.freebsd.org/D20519 > > Modified: > head/sys/kern/vfs_subr.c > > Modified: head/sys/kern/vfs_subr.c > ============================================================================== > --- head/sys/kern/vfs_subr.c Thu Jun 6 12:44:43 2019 (r348736) > +++ head/sys/kern/vfs_subr.c Thu Jun 6 15:04:50 2019 (r348737) > @@ -50,6 +50,7 @@ __FBSDID("$FreeBSD$"); > #include <sys/systm.h> > #include <sys/bio.h> > #include <sys/buf.h> > +#include <sys/capsicum.h> > #include <sys/condvar.h> > #include <sys/conf.h> > #include <sys/counter.h> > @@ -338,6 +339,93 @@ SYSCTL_ULONG(_kern, OID_AUTO, minvnodes, CTLFLAG_RW, > static int vnlru_nowhere; > SYSCTL_INT(_debug, OID_AUTO, vnlru_nowhere, CTLFLAG_RW, > &vnlru_nowhere, 0, "Number of times the vnlru process ran without success"); > + > +static int > +sysctl_try_reclaim_vnode(SYSCTL_HANDLER_ARGS) > +{ > + struct vnode *vp; > + struct nameidata nd; > + char *buf; > + unsigned long ndflags; > + int error; > + > + if (req->newptr == NULL) > + return (EINVAL); > + if (req->newlen > PATH_MAX) > + return (E2BIG); > + > + buf = malloc(PATH_MAX + 1, M_TEMP, M_WAITOK); > + error = SYSCTL_IN(req, buf, req->newlen); > + if (error != 0) > + goto out; > + > + buf[req->newlen] = '\0'; > + > + ndflags = LOCKLEAF | NOFOLLOW | AUDITVNODE1 | NOCACHE | SAVENAME; > + NDINIT(&nd, LOOKUP, ndflags, UIO_SYSSPACE, buf, curthread); > + if ((error = namei(&nd)) != 0) > + goto out; > + vp = nd.ni_vp; > + > + if ((vp->v_iflag & VI_DOOMED) != 0) { > + /* > + * This vnode is being recycled. Return != 0 to let the caller > + * know that the sysctl had no effect. Return EAGAIN because a > + * subsequent call will likely succeed (since namei will create > + * a new vnode if necessary) > + */ > + error = EAGAIN; > + goto putvnode; > + } > + > + counter_u64_add(recycles_count, 1); > + vgone(vp); > +putvnode: > + NDFREE(&nd, 0); > +out: > + free(buf, M_TEMP); > + return (error); > +} > + > +static int > +sysctl_ftry_reclaim_vnode(SYSCTL_HANDLER_ARGS) > +{ > + struct thread *td = curthread; > + struct vnode *vp; > + struct file *fp; > + int error; > + int fd; > + > + if (req->newptr == NULL) > + return (EBADF); > + > + error = sysctl_handle_int(oidp, &fd, 0, req); > + if (error != 0) > + return (error); > + error = getvnode(curthread, fd, &cap_fcntl_rights, &fp); > + if (error != 0) > + return (error); > + vp = fp->f_vnode; > + > + error = vn_lock(vp, LK_EXCLUSIVE); > + if (error != 0) > + goto drop; > + > + counter_u64_add(recycles_count, 1); > + vgone(vp); > + VOP_UNLOCK(vp, 0); > +drop: > + fdrop(fp, td); > + return (error); > +} > + > +SYSCTL_PROC(_debug, OID_AUTO, try_reclaim_vnode, > + CTLTYPE_STRING | CTLFLAG_MPSAFE | CTLFLAG_WR, NULL, 0, > + sysctl_try_reclaim_vnode, "A", "Try to reclaim a vnode by its pathname"); > +SYSCTL_PROC(_debug, OID_AUTO, ftry_reclaim_vnode, > + CTLTYPE_INT | CTLFLAG_MPSAFE | CTLFLAG_WR, NULL, 0, > + sysctl_ftry_reclaim_vnode, "I", > + "Try to reclaim a vnode by its file descriptor"); > > /* Shift count for (uintptr_t)vp to initialize vp->v_hash. */ > static int vnsz2log; > > -- Rod Grimes rgrimes@freebsd.org
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201906061735.x56HZGIJ058845>