Looks good other than the obvious style bugs, but I have a couple questions.

This line:

    max(cacheFiles, 1000);

doesn't do anything.  Maybe you meant this?

    cacheFiles = max(cacheBlocks / 32, 1000);

And why did you change the assumed average file size to 32KB?  This might
actually be a good number but I wonder if you have data to back it up.  I
just measured my own client cache and came up with 30.2KB so you could be
right.
_______________________________________________
OpenAFS-devel mailing list
[email protected]
https://lists.openafs.org/mailman/listinfo/openafs-devel

Reply via email to