Date: Tue, 30 Apr 2019 06:08:49 +0000 From: Alexey Dokuchaev <danfe@freebsd.org> To: Mark Johnston <markj@freebsd.org> Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r346932 - head/sys/ufs/ufs Message-ID: <20190430060849.GB97846@FreeBSD.org> In-Reply-To: <201904292205.x3TM5Q7Z058461@repo.freebsd.org> References: <201904292205.x3TM5Q7Z058461@repo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, Apr 29, 2019 at 10:05:26PM +0000, Mark Johnston wrote: > New Revision: 346932 > URL: https://svnweb.freebsd.org/changeset/base/346932 > > Log: > Optimize lseek(SEEK_DATA) on UFS. > > This version fixes the problems identified in r345244. > > Reviewed by: kib > @@ -56,6 +56,9 @@ __FBSDID("$FreeBSD$"); > #include <ufs/ufs/ufsmount.h> > #include <ufs/ufs/ufs_extern.h> > > +static ufs_lbn_t lbn_count(struct ufsmount *, int); > +static int readindir(struct vnode *, ufs_lbn_t, ufs2_daddr_t, struct buf **); Is the prototype for static readindir() really needed here? > +static int > +readindir(vp, lbn, daddr, bpp) > + struct vnode *vp; > + ufs_lbn_t lbn; > + ufs2_daddr_t daddr; > + struct buf **bpp; Don't we prefer ASNI declarations over K&R these days? > +static ufs_lbn_t > +lbn_count(ump, level) > + struct ufsmount *ump; > + int level; Ditto. > +int > +ufs_bmap_seekdata(vp, offp) > + struct vnode *vp; > + off_t *offp; ./danfe
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20190430060849.GB97846>