Re: [PATCH] inet/connection_sock: prefer _THIS_IP_ to current_text_addr

From: Nick Desaulniers
Date: Mon Aug 13 2018 - 17:10:33 EST


On Thu, Aug 2, 2018 at 3:17 PM David Miller <davem@xxxxxxxxxxxxx> wrote:
>
> From: Nick Desaulniers <ndesaulniers@xxxxxxxxxx>
> Date: Thu, 2 Aug 2018 15:10:00 -0700
>
> > On Thu, Aug 2, 2018 at 2:42 PM David Miller <davem@xxxxxxxxxxxxx> wrote:
> >>
> >> From: Nick Desaulniers <ndesaulniers@xxxxxxxxxx>
> >> Date: Wed, 1 Aug 2018 14:57:59 -0700
> >>
> >> > As part of the effort to reduce the code duplication between _THIS_IP_
> >> > and current_text_addr(), let's consolidate callers of
> >> > current_text_addr() to use _THIS_IP_.
> >> >
> >> > Signed-off-by: Nick Desaulniers <ndesaulniers@xxxxxxxxxx>
> >>
> >> I'll ACk this for now:
> >>
> >> Acked-by: David S. Miller <davem@xxxxxxxxxxxxx>
> >
> > Thank you David. Should you take it into the net tree, can you please add:
> >
> > Cc: stable@xxxxxxxxxxxxxxx # 4.17, 4.14, 4.9, 4.4
> >
> > to the commit message? This will help us as Nathan stated previously
> > in this thread.
>
> Why in the world is this a stable candidate? It doesn't fix a bug.
>
> My understanding is that you are trying to do a consolidation of code
> and clean things up so that only one interface is used.
>
> That means it should, at best, go to the net-next tree.

David,
Can you please pick this up for the net-next tree then?
--
Thanks,
~Nick Desaulniers