Re: [PATCH] perf: remove include of cgroup.h from perf_event.h

2013-03-06 Thread Stephane Eranian
On Wed, Mar 6, 2013 at 11:31 AM, Ingo Molnar wrote: > > * Stephane Eranian wrote: > >> > - * This is a per-cpu dynamically allocated data structure. >> > - */ >> > -struct perf_cgroup_info { >> > - u64 time; >> > - u64 timestamp;

Re: [PATCH] perf: remove include of cgroup.h from perf_event.h

2013-03-06 Thread Ingo Molnar
* Stephane Eranian wrote: > > - * This is a per-cpu dynamically allocated data structure. > > - */ > > -struct perf_cgroup_info { > > - u64 time; > > - u64 timestamp; > > -}; > > - > > -struct perf_cgroup { > > - struct

Re: [PATCH] perf: remove include of cgroup.h from perf_event.h

2013-03-05 Thread Tejun Heo
(cc'ing Steven and Frederic and quoting whole body) Hello, guys. On Tue, Mar 05, 2013 at 06:37:12PM +0800, Li Zefan wrote: > On 2013/3/5 16:33, Stephane Eranian wrote: > > On Tue, Mar 5, 2013 at 4:38 AM, Li Zefan wrote: > >> Move struct perf_cgroup_info and perf_cgroup to kernel/perf/core.c, > >

Re: [PATCH] perf: remove include of cgroup.h from perf_event.h

2013-03-05 Thread Li Zefan
On 2013/3/5 16:33, Stephane Eranian wrote: > On Tue, Mar 5, 2013 at 4:38 AM, Li Zefan wrote: >> Move struct perf_cgroup_info and perf_cgroup to kernel/perf/core.c, >> and then we can remove include of cgroup.h. >> >> Signed-off-by: Li Zefan >> --- >> include/linux/perf_event.h | 18 +

Re: [PATCH] perf: remove include of cgroup.h from perf_event.h

2013-03-05 Thread Stephane Eranian
On Tue, Mar 5, 2013 at 4:38 AM, Li Zefan wrote: > Move struct perf_cgroup_info and perf_cgroup to kernel/perf/core.c, > and then we can remove include of cgroup.h. > > Signed-off-by: Li Zefan > --- > include/linux/perf_event.h | 18 +- > kernel/events/core.c | 15 ++

[PATCH] perf: remove include of cgroup.h from perf_event.h

2013-03-04 Thread Li Zefan
Move struct perf_cgroup_info and perf_cgroup to kernel/perf/core.c, and then we can remove include of cgroup.h. Signed-off-by: Li Zefan --- include/linux/perf_event.h | 18 +- kernel/events/core.c | 15 +++ 2 files changed, 16 insertions(+), 17 deletions(-) dif