Messages in this thread | | | From | Kevin Hilman <> | Subject | Re: [RFC/PATCH 2/5] kernel_cpustat: convert to atomic 64-bit accessors | Date | Thu, 21 Feb 2013 21:57:46 -0800 |
| |
Frederic Weisbecker <fweisbec@gmail.com> writes:
> 2013/2/21 Frederic Weisbecker <fweisbec@gmail.com>: >> 2013/2/21 Kevin Hilman <khilman@linaro.org>: >>> Subject: [PATCH 2/5] kernel_cpustat: convert to atomic 64-bit accessors >>> >>> Use the atomic64_* accessors for all the kernel_cpustat fields to >>> ensure atomic access on non-64 bit platforms. >>> >>> Thanks to Mats Liljegren for CGROUP_CPUACCT related fixes. >>> >>> Cc: Mats Liljegren <mats.liljegren@enea.com> >>> Signed-off-by: Kevin Hilman <khilman@linaro.org> >> >> Funny stuff, I thought struct kernel_cpustat was made of cputime_t >> field. Actually it's u64. So the issue is independant from the new >> full dynticks cputime accounting. It was already broken before. >> >> But yeah that's not the point, we still want to fix this anyway. But >> let's just treat this patch as independant.
OK, I just sent an updated series based on your proposal.
Thanks for the review,
Kevin
| |