On Wed, Aug 18, 2010 at 4:27 PM, Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote: >> +++ b/include/linux/nfs4_pnfs.h >> @@ -0,0 +1,15 @@ >> +/* >> + * include/linux/nfs4_pnfs.h > > Please don't include these kinds of comments, they only purpose they > serve is frequently getting out of date. That applies to just about > every file added in this series. > >> + * >> + * Common data structures needed by the pnfs client and pnfs layout driver. >> + * >> + * Copyright (c) 2002 The Regents of the University of Michigan. >> + * All rights reserved. >> + * >> + * Dean Hildebrand <dhildebz@xxxxxxxxxxxxxx> >> + */ >> + >> +#ifndef LINUX_NFS4_PNFS_H >> +#define LINUX_NFS4_PNFS_H >> + >> +#endif /* LINUX_NFS4_PNFS_H */ > > Adding a file that only contains copyrights and inclusion headers is > rather odd. I think you want your split a little more corse grained. Agreed. This patch set is really a first go at squashing 281 patches into 50. We have more know re-org and cleanup, just wanted to do an initial re-org keeping the tree the same as a first step. Thanks for your comments. -->Andy > > -- > 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 > -- 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
