Looks good except for a little style nitpick: > + for (i=j = 0; i < ARRAY_SIZE(xstats); i++) { I'd rather initialize j outside of the loop, and fix the spacing to the standard one -- 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