diff options
author | Jason Low <jason.low2@hp.com> | 2015-10-14 15:07:55 -0400 |
---|---|---|
committer | Thomas Gleixner <tglx@linutronix.de> | 2015-10-15 05:23:41 -0400 |
commit | d5c373eb5610686162ff50429f63f4c00c554799 (patch) | |
tree | 1016ff693cfd585fdab1d368995f16036c9ec65c /kernel/time/posix-cpu-timers.c | |
parent | 934715a191e4be0c602d39455a7a74316f274d60 (diff) |
posix_cpu_timer: Convert cputimer->running to bool
In the next patch in this series, a new field 'checking_timer' will
be added to 'struct thread_group_cputimer'. Both this and the
existing 'running' integer field are just used as boolean values. To
save space in the structure, we can make both of these fields booleans.
This is a preparatory patch to convert the existing running integer
field to a boolean.
Suggested-by: George Spelvin <linux@horizon.com>
Signed-off-by: Jason Low <jason.low2@hp.com>
Reviewed: George Spelvin <linux@horizon.com>
Cc: Oleg Nesterov <oleg@redhat.com>
Cc: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Davidlohr Bueso <dave@stgolabs.net>
Cc: Steven Rostedt <rostedt@goodmis.org>
Cc: hideaki.kimura@hpe.com
Cc: terry.rudd@hpe.com
Cc: scott.norton@hpe.com
Cc: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1444849677-29330-4-git-send-email-jason.low2@hp.com
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Diffstat (limited to 'kernel/time/posix-cpu-timers.c')
-rw-r--r-- | kernel/time/posix-cpu-timers.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/time/posix-cpu-timers.c b/kernel/time/posix-cpu-timers.c index 6f6e252ec761..2d58153074d9 100644 --- a/kernel/time/posix-cpu-timers.c +++ b/kernel/time/posix-cpu-timers.c | |||
@@ -249,7 +249,7 @@ void thread_group_cputimer(struct task_struct *tsk, struct task_cputime *times) | |||
249 | * but barriers are not required because update_gt_cputime() | 249 | * but barriers are not required because update_gt_cputime() |
250 | * can handle concurrent updates. | 250 | * can handle concurrent updates. |
251 | */ | 251 | */ |
252 | WRITE_ONCE(cputimer->running, 1); | 252 | WRITE_ONCE(cputimer->running, true); |
253 | } | 253 | } |
254 | sample_cputime_atomic(times, &cputimer->cputime_atomic); | 254 | sample_cputime_atomic(times, &cputimer->cputime_atomic); |
255 | } | 255 | } |
@@ -918,7 +918,7 @@ static inline void stop_process_timers(struct signal_struct *sig) | |||
918 | struct thread_group_cputimer *cputimer = &sig->cputimer; | 918 | struct thread_group_cputimer *cputimer = &sig->cputimer; |
919 | 919 | ||
920 | /* Turn off cputimer->running. This is done without locking. */ | 920 | /* Turn off cputimer->running. This is done without locking. */ |
921 | WRITE_ONCE(cputimer->running, 0); | 921 | WRITE_ONCE(cputimer->running, false); |
922 | } | 922 | } |
923 | 923 | ||
924 | static u32 onecputick; | 924 | static u32 onecputick; |