On Tue, 6 Mar 2012 19:21:39 -0800 (PST)
David Rientjes <rientjes@xxxxxxxxxx> wrote:
> -static void __sync_task_rss_stat(struct task_struct *task, struct mm_struct *mm)
> +static void __sync_task_rss_stat(struct mm_struct *mm)
> {
> int i;
>
> for (i = 0; i < NR_MM_COUNTERS; i++) {
> - if (task->rss_stat.count[i]) {
> - add_mm_counter(mm, i, task->rss_stat.count[i]);
> - task->rss_stat.count[i] = 0;
> + if (current->rss_stat.count[i]) {
> + add_mm_counter(mm, i, current->rss_stat.count[i]);
> + current->rss_stat.count[i] = 0;
> }
> }
> - task->rss_stat.events = 0;
> + current->rss_stat.events = 0;
> }
hm, with my gcc it's beneficial to cache `current' in a local. But
when I tried that, Weird Things happened, because gcc has gone and
decided to inline __sync_task_rss_stat() into its callers. I don't see
how that could have been the right thing to do.
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@xxxxxxxxx. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>