[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20160628171126.5da73f41@xeon-e3>
Date: Tue, 28 Jun 2016 17:11:26 -0700
From: Stephen Hemminger <stephen@...workplumber.org>
To: David Ahern <dsa@...ulusnetworks.com>
Cc: netdev@...r.kernel.org
Subject: Re: [PATCH v2 iproute2 3/3] ss: Add support to filter on device
On Tue, 28 Jun 2016 11:40:40 -0600
David Ahern <dsa@...ulusnetworks.com> wrote:
> On 6/27/16 3:13 PM, Stephen Hemminger wrote:
> > On Mon, 27 Jun 2016 11:34:25 -0700
> > David Ahern <dsa@...ulusnetworks.com> wrote:
> >
> >> + case SSF_DEVCOND:
> >> + {
> >> + struct aafilter *a = (void *)f->pred;
> >
> > I don't like the wandering bracket left, but all the code has that.
> > After this will change it to:
> > case SSF_DEVCOND: {
> > struct aafilter *a = f->pred;
> > ...
> >
>
> meaning you'll take the patches as is since they follow current
> formatting and you will apply a patch to fix the indentation in that
> run_ssfilter? They have not shown up in your repo so want to make sure I
> understand the intent.
They all got applied today
Powered by blists - more mailing lists