netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jiri Pirko <jiri@resnulli.us>
To: Sun Shouxin <sunshouxin@chinatelecom.cn>
Cc: j.vosburgh@gmail.com, vfalico@gmail.com, andy@greyhouse.net,
	davem@davemloft.net, kuba@kernel.org, yoshfuji@linux-ipv6.org,
	dsahern@kernel.org, oliver@neukum.org, netdev@vger.kernel.org,
	linux-kernel@vger.kernel.org, huyd12@chinatelecom.cn
Subject: Re: [PATCH v4] net:bonding:Add support for IPV6 RLB to balance-alb mode
Date: Thu, 17 Mar 2022 09:11:25 +0100	[thread overview]
Message-ID: <YjLtLdH9gmg7yaNl@nanopsycho> (raw)
In-Reply-To: <20220317061521.23985-1-sunshouxin@chinatelecom.cn>

Thu, Mar 17, 2022 at 07:15:21AM CET, sunshouxin@chinatelecom.cn wrote:
>This patch is implementing IPV6 RLB for balance-alb mode.
>
>Suggested-by: Hu Yadi <huyd12@chinatelecom.cn>
>Signed-off-by: Sun Shouxin <sunshouxin@chinatelecom.cn>


Could you please reply to my question I asked for v1:
Out of curiosity, what is exactly your usecase? I'm asking because
I don't see any good reason to use RLB/ALB modes. I have to be missing
something.

This is adding a lot of code in bonding that needs to be maintained.
However, if there is no particular need to add it, why would we?

Could you please spell out why exactly do you need this? I'm pretty sure
that in the end well find out, that you really don't need this at all.

Thanks!


  reply	other threads:[~2022-03-17  8:11 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-03-17  6:15 [PATCH v4] net:bonding:Add support for IPV6 RLB to balance-alb mode Sun Shouxin
2022-03-17  8:11 ` Jiri Pirko [this message]
2022-03-18  9:49   ` 孙守鑫
2022-03-18 11:34     ` Jiri Pirko
2022-03-21  1:17       ` 孙守鑫
2022-03-21  9:52         ` Jiri Pirko
2022-03-17 18:49 ` David Ahern
2022-03-17 20:10   ` Jay Vosburgh
2022-03-18  9:53     ` 孙守鑫
2022-03-18  9:50   ` 孙守鑫

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=YjLtLdH9gmg7yaNl@nanopsycho \
    --to=jiri@resnulli.us \
    --cc=andy@greyhouse.net \
    --cc=davem@davemloft.net \
    --cc=dsahern@kernel.org \
    --cc=huyd12@chinatelecom.cn \
    --cc=j.vosburgh@gmail.com \
    --cc=kuba@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=oliver@neukum.org \
    --cc=sunshouxin@chinatelecom.cn \
    --cc=vfalico@gmail.com \
    --cc=yoshfuji@linux-ipv6.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 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).