| Date | Wed, 29 Sep 2010 21:52:14 -0400 | From | Christoph Hellwig <> | Subject | Re: [PATCH 12/17] fs: Introduce per-bucket inode hash locks |
| |
Instead of doing the lock overkill on a still fundamentally global data structure what about replacing this with something better. I know you've already done this with the XFS icache, and while the per-AG concept obviously can't be generic at least some of the lessons could be applied.
then again how much testing did this get anyway given that you benchmark ran mostly XFS which doesn't hit this at all?
If it was up to me I'd dtop this (and the bl_list addition) from the series for now and wait for people who care about the scalability of the generic icache code to come up with a better data structure.
|