All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Ahern <dsahern@gmail.com>
To: Alexei Starovoitov <alexei.starovoitov@gmail.com>
Cc: David Miller <davem@davemloft.net>,
	netdev@vger.kernel.org, edumazet@google.com, kafai@fb.com
Subject: Re: [PATCH net-next] ipv6: Move ipv6 stubs to a separate header file
Date: Tue, 26 Mar 2019 08:19:06 -0600	[thread overview]
Message-ID: <ac429099-52c6-b6c6-1abf-841883486f83@gmail.com> (raw)
In-Reply-To: <20190326030536.kjzp2redp33y7hk6@ast-mbp>

On 3/25/19 9:05 PM, Alexei Starovoitov wrote:
> Just this patch alone is too small to see the road ahead
> and 80+ patches are too much to review properly.

I will be sending in reviewable chunks. I think I have established a
history of evolving the code in small, singly focused patches moving the
code from where it is to where I want it to be. Any time there is a
'large' patch I try to keep it to something mechanical - like a name
change or a change to a function signature.

The email I sent ("net: Improve route scalability via support for
nexthop objects") outlined the targeted steps and the intermediate prep
work needed. To be able to reach certain clear steps and still maintain
the 20'ish patches in a set requirement, I have sent a few patches which
are standalone and basically noise - like this one - to avoid
distractions on the real change. I pushed the entire set to github so
anyone can see the end goal as well as the individual patches.

I am not sure what else I can do to be open about this change and convey
the what and the why. I discussed the feature at netconf in Seoul in
October 2017. I sent RFC patches last August on the UAPI with a working
implementation as well as more details about what this change is going
to allow. I have given talks on what I believe is the best architecture
and implementation for Linux which is the motivation for improving the
kernel. And the previous email about improving scalability is almost a
novelette in length.

  reply	other threads:[~2019-03-26 14:19 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-03-22 13:06 [PATCH net-next] ipv6: Move ipv6 stubs to a separate header file David Ahern
2019-03-22 16:14 ` Alexei Starovoitov
2019-03-22 16:17   ` David Ahern
2019-03-22 17:04     ` Alexei Starovoitov
2019-03-24  1:40 ` David Miller
2019-03-24  3:55   ` Alexei Starovoitov
2019-03-24 12:56     ` David Ahern
2019-03-25  3:26       ` Alexei Starovoitov
2019-03-25 15:39         ` Stephen Suryaputra
2019-03-25 17:02         ` David Ahern
2019-03-26  3:05           ` Alexei Starovoitov
2019-03-26 14:19             ` David Ahern [this message]
2019-03-26 15:11               ` Alexei Starovoitov
2019-03-26 15:29                 ` David Ahern
2019-03-26  2:22         ` David Ahern
2019-03-29 17:55 ` David Miller

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ac429099-52c6-b6c6-1abf-841883486f83@gmail.com \
    --to=dsahern@gmail.com \
    --cc=alexei.starovoitov@gmail.com \
    --cc=davem@davemloft.net \
    --cc=edumazet@google.com \
    --cc=kafai@fb.com \
    --cc=netdev@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.