Re: [PATCH] perf tool: save cmdline from user in file header vs whatis passed to record

From: David Ahern
Date: Thu Jul 05 2012 - 12:48:00 EST


On 7/5/12 9:48 AM, Stephane Eranian wrote:
On Tue, Jul 3, 2012 at 6:08 AM, David Ahern <dsahern@xxxxxxxxx> wrote:
Yes, you need to manually add the pfm_header__set_cmdline(). That begs

I take it you meant perf_header__set_cmdline, not pfm_header...

the question
as to whether or not it would not be better to move this call directly into
the parse_option() function given that it must be done BEFORE parsing actually
takes place.That way we would be sure all present and future commands would
be covered.

Sure, I'll move it to parse_options.

David
--
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/