On 03/13/2013 12:08 PM, Sha Zhengju wrote:
>> +static void memcg_update_root_statistics(void)
>> > +{
>> > + int cpu;
>> > + u64 pgin, pgout, faults, mjfaults;
>> > +
>> > + pgin = pgout = faults = mjfaults = 0;
>> > + for_each_online_cpu(cpu) {
>> > + struct vm_event_state *ev = &per_cpu(vm_event_states, cpu);
>> > + struct mem_cgroup_stat_cpu *memcg_stat;
>> > +
>> > + memcg_stat = per_cpu_ptr(root_mem_cgroup->stat, cpu);
>> > +
>> > + memcg_stat->events[MEM_CGROUP_EVENTS_PGPGIN] =
>> > + ev->event[PGPGIN];
>> > + memcg_stat->events[MEM_CGROUP_EVENTS_PGPGOUT] =
>> > + ev->event[PGPGOUT];
> ev->event[PGPGIN/PGPGOUT] is counted in block layer(submit_bio()) and
> represents the exactly number of pagein/pageout, but memcg
> PGPGIN/PGPGOUT events only count it as an event and ignore the page
> size. So here we can't straightforward take the ev->events for use.
>
You are right about that. Although I can't think of a straightforward
way to handle this. Well, except for the obvious of adding another
global statistic.
>> > + memcg_stat->events[MEM_CGROUP_EVENTS_PGFAULT] =
>> > + ev->event[PGFAULT];
>> > + memcg_stat->events[MEM_CGROUP_EVENTS_PGMAJFAULT] =
>> > + ev->event[PGMAJFAULT];
>> > +
>> > + memcg_stat->nr_page_events = ev->event[PGPGIN] +
>> > + ev->event[PGPGOUT];
> There's no valid memcg->nr_page_events until now, so the threshold
> notifier, but some people may use it even only root memcg exists.
> Moreover, using PGPGIN + PGPGOUT(exactly number of pagein + pageout)
> as nr_page_events is also inaccurate IMHO.
>
Humm, I believe I can zero out this. Looking at the code again, this is
not imported to userspace. It is just used to activate the thresholds
and the delta of nr_page_events is a lot more important than nr_page_events.
--
To unsubscribe from this list: send the line "unsubscribe cgroups" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
