Re: [PATCH v2 1/3] sched/numa: advanced per-cgroup numa statistic

From: çè
Date: Sun Dec 01 2019 - 21:12:15 EST




On 2019/11/29 äå6:06, Michal Koutnà wrote:
> On Fri, Nov 29, 2019 at 01:19:33PM +0800, çè <yun.wang@xxxxxxxxxxxxxxxxx> wrote:
>> I did some research regarding cpuacct, and find cpuacct_charge() is a good
>> place to do hierarchical update, however, what we get there is the execution
>> time delta since last update_curr().
> I wouldn't extend cpuacct, I'd like to look into using the rstat
> mechanism for per-CPU runtime collection. (Most certainly I won't get
> down to this until mid December though.)
>
>> I'm afraid we can't just do local/remote accumulation since the sample period
>> now is changing, still have to accumulate the execution time into locality
>> regions.y
> My idea was to decouple time from the locality counters completely. It'd
> be up to the monitoring application to normalize differences wrt
> sampling rate (and handle wrap arounds).

I see, basically I understand your proposal as utilize cpuacct's runtime
and only expose per-cgroup local/remote counters, I'm not sure if the
locality still helpful after decouple time factor from it, both need some
investigation, anyway, once I could convince myself it's working, I'll
be happy to make things simple ;-)

Regards,
Michael Wang

>
>
> Michal
>