[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAM9d7cju-Z0_+WuhCZ+Y-4Hm6MCmXbF6XVBGd-n8BP_yoici5Q@mail.gmail.com>
Date: Tue, 22 Mar 2022 15:21:51 -0700
From: Namhyung Kim <namhyung@...nel.org>
To: Peter Zijlstra <peterz@...radead.org>
Cc: Chengming Zhou <zhouchengming@...edance.com>,
Ingo Molnar <mingo@...hat.com>,
Arnaldo Carvalho de Melo <acme@...nel.org>,
Mark Rutland <mark.rutland@....com>,
Alexander Shishkin <alexander.shishkin@...ux.intel.com>,
Jiri Olsa <jolsa@...nel.org>,
Stephane Eranian <eranian@...gle.com>,
linux-perf-users <linux-perf-users@...r.kernel.org>,
linux-kernel <linux-kernel@...r.kernel.org>,
duanxiongchun@...edance.com, songmuchun@...edance.com
Subject: Re: [PATCH v2 2/6] perf/core: Introduce percpu perf_cgroup
On Tue, Mar 22, 2022 at 6:01 AM Peter Zijlstra <peterz@...radead.org> wrote:
>
> On Tue, Mar 22, 2022 at 08:08:30PM +0800, Chengming Zhou wrote:
> > Although we don't have incosistency problem any more, we can
> > have other problem like:
> >
> > CPU1 CPU2
> > (in context_switch) (attach running task)
> > prev->cgroups = cgrp2
> > perf_cgroup_sched_switch(prev, next)
> > cgrp2 == cgrp2 is True
> >
>
> Again, I'm not following, how can you attach to a running task from
> another CPU ?
I think it's supported from the beginning by writing PID to a file
in the cgroupfs.
Thanks,
Namhyung
Powered by blists - more mailing lists