On Fri, Oct 26, 2012 at 12:47:48PM +0800, Robin Dong wrote:
[..]
> @@ -1084,6 +1114,16 @@ static struct cftype throtl_files[] = {
> .private = offsetof(struct tg_stats_cpu, serviced),
> .read_seq_string = tg_print_cpu_rwstat,
> },
> + {
> + .name = "throttle.io_submit_bytes",
Do we really need io_submit_bytes stats? Your need seems to be able to
figure out if there are pending IOs in the group and if you need to
increase the bandwidth. For that, isn't number of bios enough?
Also even if we retain bytes, let us change it to "io_submitted_bytes".
Thanks
Vivek
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
