Date: Wed, 31 Aug 2011 18:08:44 -0400 (EDT) From: Rick Macklem <rmacklem@uoguelph.ca> To: George Liaskos <geo.liaskos@gmail.com> Cc: freebsd-fs@freebsd.org, Pawel Jakub Dawidek <pjd@freebsd.org> Subject: Re: NFSv4: After upgrade to 9 users can no longer list files. (sounds like a ZFS issue?) Message-ID: <1463532532.632117.1314828524787.JavaMail.root@erie.cs.uoguelph.ca> In-Reply-To: <CANcjpOB3ri-zN%2B1LEka-VRk%2B9vuV27eTX=VSSVpSU0VQnZEA2A@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
George Liaskos wrote: > On Wed, Aug 31, 2011 at 3:49 PM, Rick Macklem <rmacklem@uoguelph.ca> > wrote: > > Well, I can't imagine why this would matter, but you can try this > > patch, > > which fixes a problem introduced by r224810 where Lookup ".." no > > longer > > works. (It's at http://people.freebsd.org/~rmacklem/dotdot.patch, in > > case > > the white space gets munged.) > > Index: fs/nfsserver/nfs_nfsdport.c > > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > > --- fs/nfsserver/nfs_nfsdport.c (revision 225270) > > +++ fs/nfsserver/nfs_nfsdport.c (working copy) > > @@ -282,6 +282,7 @@ nfsvno_namei(struct nfsrv_descript *nd, struct > > nam > > > > =C2=A0 =C2=A0 =C2=A0 =C2=A0*retdirp =3D NULL; > > =C2=A0 =C2=A0 =C2=A0 =C2=A0cnp->cn_nameptr =3D cnp->cn_pnbuf; > > + ndp->ni_strictrelative =3D 0; > > =C2=A0 =C2=A0 =C2=A0 =C2=A0/* > > =C2=A0 =C2=A0 =C2=A0 =C2=A0 * Extract and set starting directory. > > =C2=A0 =C2=A0 =C2=A0 =C2=A0 */ > > Index: nfsserver/nfs_serv.c > > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > > --- nfsserver/nfs_serv.c (revision 225270) > > +++ nfsserver/nfs_serv.c (working copy) > > @@ -157,6 +157,7 @@ ndclear(struct nameidata *nd) > > =C2=A0 =C2=A0 =C2=A0 =C2=A0nd->ni_vp =3D NULL; > > =C2=A0 =C2=A0 =C2=A0 =C2=A0nd->ni_dvp =3D NULL; > > =C2=A0 =C2=A0 =C2=A0 =C2=A0nd->ni_startdir =3D NULL; > > + nd->ni_strictrelative =3D 0; > > =C2=A0} > > > > =C2=A0/* > > > > rick >=20 > This patch works for me. :) >=20 Ah, good. (I can't think of why root vs non-root would have mattered, but if it fixed the problem. Maybe just a side effect, since without being initialized, it would be whatever happened to be on the stack.) Thanks for doing the legwork on this and letting us know. This patch is in the re@ queue, rick. > Regards, > George
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1463532532.632117.1314828524787.JavaMail.root>