netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Ido Schimmel <idosch@idosch.org>
To: David Ahern <dsahern@gmail.com>
Cc: netdev@vger.kernel.org, davem@davemloft.net,
	roopa@cumulusnetworks.com, nikolay@cumulusnetworks.com,
	tom@herbertland.com
Subject: Re: [PATCH v3 net-next 00/10] net/ipv6: Add support for path selection using hash of 5-tuple
Date: Fri, 2 Mar 2018 18:42:06 +0200	[thread overview]
Message-ID: <20180302164206.GA20477@splinter> (raw)
In-Reply-To: <20180302163221.26954-1-dsahern@gmail.com>

On Fri, Mar 02, 2018 at 08:32:11AM -0800, David Ahern wrote:
> Hardware supports multipath selection using the standard L4 5-tuple
> instead of just L3 and the flow label. In addition, some network
> operators prefer IPv6 path selection to use the 5-tuple. To that end,
> add support to IPv6 for multipath hash policy similar to
> bf4e0a3db97eb ("net: ipv4: add support for ECMP hash policy choice").
> The default is still L3 which covers source and destination addresses
> along with flow label and IPv6 protocol. This gives users a choice in
> hash algorithms if they believe L3 only and the IPv6 flow label are not
> sufficient for their use case.
> 
> A separate sysctl is added for IPv6, allowing IPv4 and IPv6 to use
> different algorithms if desired.
> 
> The first 3 patches modify the IPv4 variant so that at the end of the
> patch set the ipv4 and ipv6 implementations are direct parallels.
> 
> Patch 4 refactors the existing rt6_multipath_hash in preparation for
> adding the policy option.
> 
> Patch 5 renames the existing netevent to have IPv4 in the name so ipv4
> changes can be distinguished from IPv6 if the netevent handler cares.
> 
> Patch 6 adds the skb as an argument through the FIB lookup functions
> to the multipath selection. Needed for the forwarding case.
> 
> Patch 7 adds the L4 hash support.
> 
> Patch 8 adds the hook for the netevent to the spectrum driver to update
> the ASIC.
> 
> Patch 9 removes no longer used code.
> 
> Patch 10 adds a testcase for IPv6 multipath with L4 hash.

David, Great work. Thanks a lot!

  parent reply	other threads:[~2018-03-02 16:42 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-03-02 16:32 [PATCH v3 net-next 00/10] net/ipv6: Add support for path selection using hash of 5-tuple David Ahern
2018-03-02 16:32 ` [PATCH v3 net-next 01/10] net/ipv4: Pass net to fib_multipath_hash instead of fib_info David Ahern
2018-03-02 16:40   ` Ido Schimmel
2018-03-02 17:43   ` Nikolay Aleksandrov
2018-03-02 16:32 ` [PATCH v3 net-next 02/10] net: Align ip_multipath_l3_keys and ip6_multipath_l3_keys David Ahern
2018-03-02 17:44   ` Nikolay Aleksandrov
2018-03-02 16:32 ` [PATCH v3 net-next 03/10] net/ipv4: Simplify fib_multipath_hash with optional flow keys David Ahern
2018-03-02 17:44   ` Nikolay Aleksandrov
2018-03-02 16:32 ` [PATCH v3 net-next 04/10] net/ipv6: Make rt6_multipath_hash similar to fib_multipath_hash David Ahern
2018-03-02 17:45   ` Nikolay Aleksandrov
2018-03-02 16:32 ` [PATCH v3 net-next 05/10] net: Rename NETEVENT_MULTIPATH_HASH_UPDATE David Ahern
2018-03-02 17:45   ` Nikolay Aleksandrov
2018-03-02 16:32 ` [PATCH v3 net-next 06/10] net/ipv6: Pass skb to route lookup David Ahern
2018-03-02 17:47   ` Nikolay Aleksandrov
2018-03-02 16:32 ` [PATCH v3 net-next 07/10] net/ipv6: Add support for path selection using hash of 5-tuple David Ahern
2018-03-02 17:49   ` Nikolay Aleksandrov
2018-03-02 16:32 ` [PATCH v3 net-next 08/10] mlxsw: spectrum_router: Add support for ipv6 hash policy update David Ahern
2018-03-02 16:32 ` [PATCH v3 net-next 09/10] net: Remove unused get_hash_from_flow functions David Ahern
2018-03-02 17:49   ` Nikolay Aleksandrov
2018-03-02 16:32 ` [PATCH v3 net-next 10/10] selftests: forwarding: Add multipath test for L4 hashing David Ahern
2018-03-02 16:42 ` Ido Schimmel [this message]
2018-03-04 18:04 ` [PATCH v3 net-next 00/10] net/ipv6: Add support for path selection using hash of 5-tuple 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=20180302164206.GA20477@splinter \
    --to=idosch@idosch.org \
    --cc=davem@davemloft.net \
    --cc=dsahern@gmail.com \
    --cc=netdev@vger.kernel.org \
    --cc=nikolay@cumulusnetworks.com \
    --cc=roopa@cumulusnetworks.com \
    --cc=tom@herbertland.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).