OK - Thanks for the review. New patches to follow.. -->Andy On Mon, Aug 31, 2009 at 9:08 AM, J. Bruce Fields<bfields@xxxxxxxxxxxx> wrote: > On Mon, Aug 31, 2009 at 02:10:55AM +0300, Benny Halevy wrote: >> On Aug. 29, 2009, 0:33 +0300, "J. Bruce Fields" <bfields@xxxxxxxxxxxx> wrote: >> > I think this is OK for now, but maybe stick something like: >> > >> > BUILD_BUG_ON(NFSD_MAX_SLOTS_PER_SESSION * sizeof(struct nfsd4_slot) >> > + sizeof(struct nfsd4_session) > PAGE_SIZE); >> > >> > in state.h just to warn anyone who wants to blindly bump up >> >> I think that the BUILD_BUG_ON should be placed in a function so >> it gets to compile, > > Whoops, thanks--I hadn't noticed that. > >> so it'd better be defined here and not in state.h >> (where it also is in the right context) > > Sounds fine.--b. > _______________________________________________ > pNFS mailing list > pNFS@xxxxxxxxxxxxx > http://linux-nfs.org/cgi-bin/mailman/listinfo/pnfs > -- 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