Re: [PATCH] Add unaccounted time to timeslice_used.

From: Justin TerAvest
Date: Sat Mar 12 2011 - 00:35:47 EST


On Fri, Mar 11, 2011 at 8:17 PM, Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:
> On Fri, Mar 11, 2011 at 01:06:12PM -0800, Justin TerAvest wrote:
>> There are two kind of times that tasks are not charged for: the first
>> seek and the extra time slice used over the allocated timeslice. Both
>> of these exported as a new unaccounted_time stat.
>>
>> I think it would be good to have this reported in 'time' as well, but
>> that is probably a separate discussion.
>>
>> Signed-off-by: Justin TerAvest <teravest@xxxxxxxxxx>
>> ---
>>  block/blk-cgroup.c  |   16 +++++++++++++++-
>>  block/blk-cgroup.h  |   12 ++++++++++--
>>  block/cfq-iosched.c |   21 +++++++++++++--------
>>  block/cfq.h         |    6 +++---
>>  4 files changed, 41 insertions(+), 14 deletions(-)
>
> Hi! Just an FYI, could you add what subsystem you are changing into the
> subject. Like:
>
> [PATCH] block: Add unaccounted time to timeslice_used
>
> Otherwise, the subject looks like it's modifying the kernel internal time
> source.

Sorry about that. I'll be more careful in the future.

Thanks,
Justin

>
> Thanks,
>
> -- Steve
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/