Em Fri, Sep 04, 2020 at 09:25:02PM +0800, Yuehaibing escreveu:
> On 2020/9/4 2:55, Arnaldo Carvalho de Melo wrote:
> > Em Thu, Sep 03, 2020 at 03:54:51PM -0300, Arnaldo Carvalho de Melo escreveu:
> >>>> Fixes: 13edc237200c ("perf bench: Add a multi-threaded synthesize
> >>>> benchmark")
> >>>> Signed-off-by: YueHaibing <[email protected]>
> >>> Thanks, applied, kudos for adding the Fixes: tag, appreciated!
> >> But...
> OOps, sorry for this, I'll pay attention next time.
Thanks!
- Arnaldo
> >>>> +++ b/tools/perf/bench/synthesize.c
> >>>> @@ -162,8 +162,8 @@ static int do_run_multi_threaded(struct target
> >>>> *target,
> >>>> init_stats(&event_stats);
> >>>> for (i = 0; i < multi_iterations; i++) {
> >>>> session = perf_session__new(NULL, false, NULL);
> >>>> - if (!session)
> >>>> - return -ENOMEM;
> >>>> + if (IS_ERR(session)) {
> >>>> + return PTR_ERR(session);
> >> This doesn't compile, so I take back that kudo ;-\
> >> I'm fixing this by removing that needless '{'.
