On Fri, Jan 09, 2015 at 03:02:02PM -0800, Tom Haynes wrote:
> > DECODE_HEAD;
> > @@ -358,15 +373,10 @@ nfsd4_decode_fattr(struct nfsd4_compoundargs *argp, u32 *bmval,
> > dummy32 = be32_to_cpup(p++);
> > switch (dummy32) {
> > case NFS4_SET_TO_CLIENT_TIME:
> > - /* We require the high 32 bits of 'seconds' to be 0, and we ignore
> > - all 32 bits of 'nseconds'. */
>
> Have you done away with these requirements?
No, the comment just go lost, I'll add it bacl.
>
> > - READ_BUF(12);
> > len += 12;
>
> I think this code makes it clear that the magic number 12 is the
> same on both lines. With the change, that gets lost.
>
> Do I think that the 12 will ever change? No.
>
> Do I think this becomes more "magic"? Yes.
Sure. but the whole counting the number to be decoded in setattr
is magic to start with. I guess we could replace it with some magic
pointer arithmetic on argp->p, but is that really worth it? Should
be a separate patch for sure.
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
