On Sat, Dec 06, 2014 at 05:09:29PM +0000, Dilger, Andreas wrote: > > > >diff --git a/drivers/staging/lustre/lustre/fld/lproc_fld.c > >b/drivers/staging/lustre/lustre/fld/lproc_fld.c > >index 74b4db9..7a55941 100644 > >--- a/drivers/staging/lustre/lustre/fld/lproc_fld.c > >+++ b/drivers/staging/lustre/lustre/fld/lproc_fld.c > >@@ -96,7 +96,7 @@ fld_proc_hash_seq_write(struct file *file, > > char *name; > > int i; > > > >- if (count > 80) > >+ if (count > LUSTRE_MDT_MAXNAMELEN) > > return -ENAMETOOLONG; > > Sorry for the late reply, but this has nothing to do with an MDT service > name, so using this #define is misleading here. As I wrote in my other > email, this only needs to be at most 8 characters. >
I see that now, I will change that according to the other email. Do you want me to redo this patch and still define the 80 as LUSTRE_MDT_MAXNAMELEN but without this last part (just the define and the string length declaration), or should we just let this go? Thanks _______________________________________________ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
