Subject: Re: vop_symlink and unused vpp?
To: Bill Studenmund <wrstuden@zembu.com>
From: Assar Westerlund <assar@netbsd.org>
List: tech-kern
Date: 07/19/2001 14:45:46
Bill Studenmund <wrstuden@zembu.com> writes:
> On 17 Jul 2001, Assar Westerlund wrote:
>
> > Bill Studenmund <wrstuden@zembu.com> writes:
> > > No objections, just a suggestion. There are a few places where you added
> > > if (error), but didn't indent the following vput. Please do so. Oh, also
> > > please indent the nfs fragment too, though maybe only 4 characters to
> > > prevent the line from wrapping. Your call.
> >
> > My fault for doing the diff with `-w'. All the code is actually indented.
>
> Ahhh.. Ok. :-)
I did vop_mknod too, and I got another question for you. Is
layer_bypass going to be happy and should I apply this comment fix?
/assar
Index: layer_vnops.c
===================================================================
RCS file: /cvsroot/syssrc/sys/miscfs/genfs/layer_vnops.c,v
retrieving revision 1.6
diff -u -w -u -w -r1.6 layer_vnops.c
--- layer_vnops.c 2001/06/07 13:32:47 1.6
+++ layer_vnops.c 2001/07/19 12:45:50
@@ -393,8 +393,7 @@
descp->vdesc_vpp_offset,ap);
/*
* Only vop_lookup, vop_create, vop_makedir, vop_bmap,
- * vop_mknod, and vop_symlink return vpp's. The latter
- * two are VPP_WILLRELE, so we won't get here, and vop_bmap
+ * vop_mknod, and vop_symlink return vpp's. vop_bmap
* doesn't call bypass as the lower vpp is fine (we're just
* going to do i/o on it). vop_loookup doesn't call bypass
* as a lookup on "." would generate a locking error.