diff options
author | Frederic Weisbecker <fweisbec@gmail.com> | 2013-07-12 13:05:14 -0400 |
---|---|---|
committer | Frederic Weisbecker <fweisbec@gmail.com> | 2013-08-12 18:40:44 -0400 |
commit | 5b206d48e58204e84d249c4eb18651a1ff7a1274 (patch) | |
tree | c97a8158df0ff854d1d6389ddde5cc1fa53bae92 /kernel/sched/cputime.c | |
parent | 2d854e5738cded368a0759f85b1197f5c044513d (diff) |
vtime: Update a few comments
Update a stale comment from the old vtime era and document some
locking that might be non obvious.
Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Steven Rostedt <rostedt@goodmis.org>
Cc: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Borislav Petkov <bp@alien8.de>
Cc: Li Zhong <zhong@linux.vnet.ibm.com>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Kevin Hilman <khilman@linaro.org>
Diffstat (limited to 'kernel/sched/cputime.c')
-rw-r--r-- | kernel/sched/cputime.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/kernel/sched/cputime.c b/kernel/sched/cputime.c index a7959e05a9d5..223a35efa0a6 100644 --- a/kernel/sched/cputime.c +++ b/kernel/sched/cputime.c | |||
@@ -712,6 +712,13 @@ void vtime_user_enter(struct task_struct *tsk) | |||
712 | 712 | ||
713 | void vtime_guest_enter(struct task_struct *tsk) | 713 | void vtime_guest_enter(struct task_struct *tsk) |
714 | { | 714 | { |
715 | /* | ||
716 | * The flags must be updated under the lock with | ||
717 | * the vtime_snap flush and update. | ||
718 | * That enforces a right ordering and update sequence | ||
719 | * synchronization against the reader (task_gtime()) | ||
720 | * that can thus safely catch up with a tickless delta. | ||
721 | */ | ||
715 | write_seqlock(&tsk->vtime_seqlock); | 722 | write_seqlock(&tsk->vtime_seqlock); |
716 | __vtime_account_system(tsk); | 723 | __vtime_account_system(tsk); |
717 | current->flags |= PF_VCPU; | 724 | current->flags |= PF_VCPU; |