Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1604503 > unrolled thread

Re: [PATCH 1/2] perf probe: Fix concat_probe_trace_events

Started byRavi Bangoria <ravi.bangoria@linux.vnet.ibm.com>
First post2017-03-20 13:50 +0100
Last post2017-03-20 14:50 +0100
Articles 2 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH 1/2] perf probe: Fix concat_probe_trace_events Ravi Bangoria <ravi.bangoria@linux.vnet.ibm.com> - 2017-03-20 13:50 +0100
    Re: [PATCH 1/2] perf probe: Fix concat_probe_trace_events Arnaldo Carvalho de Melo <acme@kernel.org> - 2017-03-20 14:50 +0100

#1604503 — Re: [PATCH 1/2] perf probe: Fix concat_probe_trace_events

FromRavi Bangoria <ravi.bangoria@linux.vnet.ibm.com>
Date2017-03-20 13:50 +0100
SubjectRe: [PATCH 1/2] perf probe: Fix concat_probe_trace_events
Message-ID<tn4V4-4Nu-35@gated-at.bofh.it>

On Wednesday 08 March 2017 02:07 PM, Masami Hiramatsu wrote:
> On Wed,  8 Mar 2017 12:29:07 +0530
> Ravi Bangoria <ravi.bangoria@linux.vnet.ibm.com> wrote:
>
>> '*ntevs' contains number of elements present in 'tevs' array. If
>> there are no elements in array, 'tevs2' can be directly assigned
>> to 'tevs' without allocating more space. So the condition should
>> be  '*ntevs == 0'  not  'ntevs == 0'.
> Oops, good catch!
>
> Acked-by: Masami Hiramatsu <mhiramat@kernel.org>

Hi Arnaldo,

Can you please pull this patch.

Thanks,
Ravi

[toc] | [next] | [standalone]


#1604587

FromArnaldo Carvalho de Melo <acme@kernel.org>
Date2017-03-20 14:50 +0100
Message-ID<tn5R8-5oQ-17@gated-at.bofh.it>
In reply to#1604503
Em Mon, Mar 20, 2017 at 03:03:05PM +0530, Ravi Bangoria escreveu:
> 
> 
> On Wednesday 08 March 2017 02:07 PM, Masami Hiramatsu wrote:
> > On Wed,  8 Mar 2017 12:29:07 +0530
> > Ravi Bangoria <ravi.bangoria@linux.vnet.ibm.com> wrote:
> >
> >> '*ntevs' contains number of elements present in 'tevs' array. If
> >> there are no elements in array, 'tevs2' can be directly assigned
> >> to 'tevs' without allocating more space. So the condition should
> >> be  '*ntevs == 0'  not  'ntevs == 0'.
> > Oops, good catch!
> >
> > Acked-by: Masami Hiramatsu <mhiramat@kernel.org>
> 
> Hi Arnaldo,
> 
> Can you please pull this patch.

Sure, done.

- Arnaldo

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web