Re: [PATCH] perf record: Fix to honor user freq/interval properly

From: Andi Kleen
Date: Mon Jun 09 2014 - 12:54:38 EST


On Mon, Jun 09, 2014 at 02:43:37PM +0900, Namhyung Kim wrote:
> When configuring event perf checked a wrong condition that user
> specified both of freq (-F) and period (-c) or the event has no
> default value. This worked because most of events don't have default
> value and only tracepoint events have default of 1 (and it's not
> desirable to change it for those events).
>
> However, Andi's downloadable event patch changes the situation so it
> cannot change the value for those events. Fix it by allowing override
> the default value if user gives one of the options.

Thanks. Looks good. I'll drop my version.

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