Re: [PATCH 01/14] perf tool: Introduce perf_hpp__list for period related columns

From: Namhyung Kim
Date: Thu Nov 29 2012 - 05:52:10 EST


Hi,

On Thu, 29 Nov 2012 16:44:59 +0900, Namhyung Kim wrote:
> Hi Jiri,
>
> On Wed, 28 Nov 2012 14:52:36 +0100, Jiri Olsa wrote:
>> Adding perf_hpp__list list to register and contain all period
>> related columns the command is interested in.
>>
>> This way we get rid of static array holding all possible
>> columns and enable commands to register their own columns.
>
> But it seems you didn't get rid of the array? :)

Ah, understood. You wanted to add new hpp columns dynamically!
Looks good to me then.

Acked-by: Namhyung Kim <namhyung@xxxxxxxxxx>

Thanks,
Namhyung
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/