[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CANP3RGexoRnp6PRX6OG8obxPhdTt74J-8yjr_hNJOhzHnv1Xsw@mail.gmail.com>
Date: Thu, 20 Jul 2023 09:52:55 -0700
From: Maciej Żenczykowski <maze@...gle.com>
To: Jakub Kicinski <kuba@...nel.org>
Cc: Linux Network Development Mailing List <netdev@...r.kernel.org>, Thomas Haller <thaller@...hat.com>,
Eric Dumazet <edumazet@...gle.com>, Paolo Abeni <pabeni@...hat.com>,
"David S. Miller" <davem@...emloft.net>, David Ahern <dsahern@...nel.org>, Jiri Pirko <jiri@...nulli.us>,
Xiao Ma <xiaom@...gle.com>
Subject: Re: [PATCH net v2] ipv6 addrconf: fix bug where deleting a mngtmpaddr
can create a new temporary address
On Thu, Jul 20, 2023 at 9:34 AM Jakub Kicinski <kuba@...nel.org> wrote:
>
> On Thu, 20 Jul 2023 09:00:22 -0700 Maciej Żenczykowski wrote:
> > currently on 6.4 net/main:
> >
> > # ip link add dummy1 type dummy
> > # echo 1 > /proc/sys/net/ipv6/conf/dummy1/use_tempaddr
> > # ip link set dummy1 up
> > # ip -6 addr add 2000::1/64 mngtmpaddr dev dummy1
> > # ip -6 addr show dev dummy1
>
> FTR resending the patch as part of the same thread is really counter
> productive for the maintainers. We review patches in order, no MUA
> I know can be told to order things correctly when new versions are sent
> in reply.
Sorry, but I'm afraid as a non-maintainer I have no idea what your
work flows are like.
(and it shows up just fine on patchwork.kernel.org which is what I
thought you all used now...)
> Don't try too hard, be normal, please.
What's considered 'normal' seems to depend on the list.
I'm pretty sure I've been told to do --in-reply follow ups previously
when I didn't
(though it might not have been on netdev...).
Email is really a very poor medium for code reviews in general.
Powered by blists - more mailing lists