[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <77cda378-37e7-918c-05e6-09ac7b3ed5c7@linux.vnet.ibm.com>
Date: Wed, 14 Dec 2016 22:33:21 +0530
From: Hari Bathini <hbathini@...ux.vnet.ibm.com>
To: "Eric W. Biederman" <ebiederm@...ssion.com>,
Peter Zijlstra <peterz@...radead.org>
Cc: ast@...com, lkml <linux-kernel@...r.kernel.org>, acme@...nel.org,
alexander.shishkin@...ux.intel.com, mingo@...hat.com,
daniel@...earbox.net, rostedt@...dmis.org,
Ananth N Mavinakayanahalli <ananth@...ux.vnet.ibm.com>,
sargun@...gun.me, Aravinda Prasad <aravinda@...ux.vnet.ibm.com>,
brendan.d.gregg@...il.com
Subject: Re: [PATCH v3 3/3] perf tool: add cgroup identifier entry in perf
report
On Wednesday 14 December 2016 09:22 PM, Eric W. Biederman wrote:
> Peter Zijlstra <peterz@...radead.org> writes:
>
>> On Wed, Dec 14, 2016 at 08:56:43AM +1300, Eric W. Biederman wrote:
>>> I would just make the identifier a structure containing the
>>> device number and the inode number. It didn't look like perf required
>>> the identifier to be a simple integer.
>> Right, perf doesn't care at all here, its just a transport.
> perf report? In that case I think perf cares enough to know there is
> some identifier it is reporting things by.
Let me post v4 with this change..
Thanks
Hari
Powered by blists - more mailing lists