Re: [patch 2/2] perf jit: remove some no-op error handling

2016-07-16 Thread Stephane Eranian
On Fri, Jul 15, 2016 at 3:27 PM, Arnaldo Carvalho de Melo wrote: > Em Sat, Jul 16, 2016 at 12:08:36AM +0300, Dan Carpenter escreveu: >> info.e_machine is a uint16_t so m is never less than zero. It looks >> like this was maybe left over code from earlier versions so I've just >>

Re: [patch 2/2] perf jit: remove some no-op error handling

2016-07-16 Thread Stephane Eranian
On Fri, Jul 15, 2016 at 3:27 PM, Arnaldo Carvalho de Melo wrote: > Em Sat, Jul 16, 2016 at 12:08:36AM +0300, Dan Carpenter escreveu: >> info.e_machine is a uint16_t so m is never less than zero. It looks >> like this was maybe left over code from earlier versions so I've just >> removed it. > >

Re: [patch 2/2] perf jit: remove some no-op error handling

2016-07-15 Thread Arnaldo Carvalho de Melo
Em Sat, Jul 16, 2016 at 12:08:36AM +0300, Dan Carpenter escreveu: > info.e_machine is a uint16_t so m is never less than zero. It looks > like this was maybe left over code from earlier versions so I've just > removed it. Yeah, it sure looks like that, applying, thanks, - Arnaldo >

Re: [patch 2/2] perf jit: remove some no-op error handling

2016-07-15 Thread Arnaldo Carvalho de Melo
Em Sat, Jul 16, 2016 at 12:08:36AM +0300, Dan Carpenter escreveu: > info.e_machine is a uint16_t so m is never less than zero. It looks > like this was maybe left over code from earlier versions so I've just > removed it. Yeah, it sure looks like that, applying, thanks, - Arnaldo >

[patch 2/2] perf jit: remove some no-op error handling

2016-07-15 Thread Dan Carpenter
info.e_machine is a uint16_t so m is never less than zero. It looks like this was maybe left over code from earlier versions so I've just removed it. Signed-off-by: Dan Carpenter diff --git a/tools/perf/jvmti/jvmti_agent.c b/tools/perf/jvmti/jvmti_agent.c index

[patch 2/2] perf jit: remove some no-op error handling

2016-07-15 Thread Dan Carpenter
info.e_machine is a uint16_t so m is never less than zero. It looks like this was maybe left over code from earlier versions so I've just removed it. Signed-off-by: Dan Carpenter diff --git a/tools/perf/jvmti/jvmti_agent.c b/tools/perf/jvmti/jvmti_agent.c index 91bf333..55daeff 100644 ---