lkml.org 
[lkml]   [2018]   [Dec]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH 3/7] tracing: Use var_refs[] for hist trigger reference checking
From
Date
Hi Steve,

On Wed, 2018-12-19 at 10:36 -0500, Steven Rostedt wrote:
> On Wed, 19 Dec 2018 09:01:43 -0600
> Tom Zanussi <zanussi@kernel.org> wrote:
>
> > > > + if (hist_field && hist_field->var.idx == var_idx &&
> > > > + hist_field->var.hist_data == var_data)
> > > > + found = hist_field;
> > > >
> > > > return found;
> > >
> > > It seems we don't need "found" var here. Just return hist_field
> > > or
> > > NULL.
> > >
> >
> > OK, will change these and resubmit shortly.
>
> Tom,
>
> If this is the only patch in the series that needs updating, can you
> just reply to this patch with the v2 of 3/7? That is have a subject
> of:
>
> [PATCH v2 3/7] tracing: Use var_refs[] for hist trigger reference
> checking
>
> and reply to the [PATCH 3/7]
>
>
> This isn't the normal way of updates, but I want to start taking this
> patches in ASAP, and I figured this may be the easiest for both of
> us.
>
> It's fine to send a v2 of the entire patch series (which other
> maintainers require), so I'll leave it up to you.
>
> If there's any other patch that needs changing, then a full v2 series
> is required. But if this is the only change and you want to send just
> this patch, then I'll take that as well.
>

Sorry for the delay - had an emergency appointment for the dog at the
vet this morning..

Anyway, yeah, I'll reply with the patch - it's the only one that needs
updating in this series.

Thanks,

Tom

> -- Steve

\
 
 \ /
  Last update: 2018-12-19 20:09    [W:0.051 / U:0.864 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site