Re: Fixes for ffs/logging issues for 5.0, for review

[Antti Kantee <pooka%cs.hut.fi@localhost>]

I missed the deadline for the actual issue, but ...

On Sun Feb 22 2009 at 15:11:22 +0000, Andrew Doran wrote:
> On Sun, Feb 22, 2009 at 11:29:20PM +0900, Takeshi Nakayama wrote:
> 
> > >>> Andrew Doran <ad%netbsd.org@localhost> wrote
> > 
> > > Here is an updated version of the diff:
> > > 
> > >   http://www.netbsd.org/~ad/ffs-fixes4.diff
> > 
> > | @@ -70,8 +111,8 @@ vfs_allocate_syncvnode(mp)
> > |         struct mount *mp;
> > |  {
> > |         struct vnode *vp;
> > | -       static long start, incr, next;
> > | -       int error;
> > | +       static int start, incr, next;
> > | +       int error, delay;
> > |  
> > |         /* Allocate a new vnode */
> > |         if ((error = getnewvnode(VT_VFS, mp, sync_vnodeop_p, &vp)) != 0)
> > 
> > Variable named "delay" causes build error at least on sparc64.
> 
> Thanks, I have changed the variable name.

... if someone wants a few evenings of fun manual tinkering in the kernel,
PR kern/40505 is about delay.