Re: [PATCH] modify error code when perf_session__new() fail.

From: Jiri Olsa
Date: Tue Sep 23 2014 - 08:02:47 EST


hi,
subject should start with 'perf tools:'

On Tue, Sep 23, 2014 at 03:12:17PM +0900, taeung wrote:
> Because perf_session__new() could fail
> for more reasons than just ENOMEM,
> I modified error code(ENOMEM or EINVAL)
> into -1.

why the indent?

>
> Signed-off-by: taeung <treeze.taeung@xxxxxxxxx>

the log message needs some love, but the patch looks ok

Acked-by: Jiri Olsa <jolsa@xxxxxxxxxx>

thanks,
jirka
--
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/