On Wed, May 30, 2012 at 10:59:04PM +0800, jeff.liu@xxxxxxxxxx wrote: > dquot_free_inode(inode); > + ns_dquot_free_inode(inode); It looks like you are adding ns_dquot_<foo>() with exactly the same arguments after dquot_<foo>() calls, for various values of <foo>. Instead of needing to change all of the file systems, would it be possible simply to change the quota layer to call the ns_dquot_* functions? It might make your life simpler, and it would certainly reduce the number of filesystem maintainers that would need to look through and review your changes. You'd just have to work with Jan Kara as the quota system maintainer. Regards, - Ted -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
