On Mon, Jun 14, 2010 at 08:09:17PM -0400, Benny Halevy wrote: > On Jun. 14, 2010, 16:09 -0400, Benny Halevy <bhalevy@xxxxxxxxxxx> wrote: > > On 2010-06-14 16:00, J. Bruce Fields wrote: > >> From: Eric Anderle <eanderle@xxxxxxxxx> > >> > >> We should catch errors in the format at the time the list is given to > >> the kernel, rather than just returning garbage to the client and letting > >> the client fail. > >> > >> Signed-off-by: J. Bruce Fields <bfields@xxxxxxxxxxxxxx> > >> --- > >> fs/nfsd/nfs4pnfsdlm.c | 29 +++++++++++++++++++++++------ > >> 1 files changed, 23 insertions(+), 6 deletions(-) > >> > >> diff --git a/fs/nfsd/nfs4pnfsdlm.c b/fs/nfsd/nfs4pnfsdlm.c > >> index 1cf1b69..b6d9f23 100644 > >> --- a/fs/nfsd/nfs4pnfsdlm.c > >> +++ b/fs/nfsd/nfs4pnfsdlm.c > >> @@ -26,6 +26,7 @@ > >> #include <linux/nfsd/debug.h> > >> #include <linux/nfsd/nfs4pnfsdlm.h> > >> #include <linux/nfsd/nfs4layoutxdr.h> > >> +#include <linux/sunrpc/clnt.h> > >> > >> #include "nfsfh.h" > >> #include "nfsd.h" > >> @@ -87,6 +88,24 @@ out: > >> return ret; > >> } > >> > >> +bool nfsd4_validate_pnfs_dlm_device(char *ds_list, int len, int *num_ds) > > > > Is "len" used? > > Merged at pnfs-all-2.6.35-rc3-2010-06-14 > len parameter removed. Thanks. --b. -- 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