Re: [PATCH 1/1] sched: Make schedstats a runtime tunable that is disabled by default v3

From: Srikar Dronamraju
Date: Tue Feb 02 2016 - 04:33:50 EST



Dont we need changes for sched_info_on()?

If we disable schedstats dynamically but CONFIG_TASK_DELAY_ACCT is not
set, then sched_info_on will return true, This could impact guest steal
time stats as well as data read from /proc/<PID>/schedstat

Also when schedstats is dynamically disabled, and user tries to enable
kernel sleep profiling profile_setup(), the kernel may not be able to do
the right profiling since enqueue_sleeper() may not get called. Should
we alert the user saying kernel sleep profiling is disabled?

--
Thanks and Regards
Srikar Dronamraju