On Wed, Apr 22, 2009 at 11:06:33AM +0530, Krishna Kumar2 wrote: > "J. Bruce Fields" <bfields@xxxxxxxxxxxx> wrote on 04/22/2009 04:18:38 AM: > > > > +/* > > > + * This is a cache of file handles to quicken file lookup. This also > > > + * helps prevent multiple open/close of a file when a client reads it. > > > + * > > > + * If you increase the number of cached files very much, you'll need > to > > > + * add a hash table here. > > > > I'm a little confused here--what do you mean by "add a hash table here"? > > That is also in the original code. I feel the comment is wrong as we > already have a hash table in raparm_hbucket, but I didn't want to change > typos as part of this patch. This patch doesn't change anything--it only adds new lines. If you're trying to make minimal changes, that's admirable, but break up the patches in such a way that each patch shows that minimal change. --b. > > thanks, > > - KK > -- 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
