Nick Piggin <npiggin@xxxxxxx> writes: > Hi, > Comments? > Thanks, > Nick > > -- > struct dentry is one of the most critical structures in the kernel. So it's > sad to see it going neglected. Very nice. But the sad thing is that such optimizations tend to quickly bit rot again. At least add big fat comments. How does it look like on 32bit hosts? Since the size is variable depending on word size it might be a good idea to auto adjust inline name length to always give a nice end result for slab. Also I think with some effort it would be possible to shrink it more. But since you already reached cache lines, it would just allow to increase inline name length. Ok perhaps it would help more on 32bit. Further possibilities to shrink: - Eliminate name.length. It seems of dubious utility (in general I'm not sure struct qstr is all that useful) - Change some of the children/alias list_heads to hlist_heads. I don't think these lists typically need O(1) access to the end. - If the maximum mount nest was limited d_mounted could migrate into d_flags (that would be probably desparate) -Andi -- ak@xxxxxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
