From: Hangbin Liu <liuhangbin@gmail.com>
To: Jay Vosburgh <jv@jvosburgh.net>
Cc: Andrew Lunn <andrew+netdev@lunn.ch>,
"David S. Miller" <davem@davemloft.net>,
Eric Dumazet <edumazet@google.com>,
Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
Jiri Pirko <jiri@resnulli.us>,
netdev@vger.kernel.org
Subject: [Bonding Draft Proposal] Add lacp_prio Support for ad_select?
Date: Tue, 24 Jun 2025 06:53:17 +0000 [thread overview]
Message-ID: <aFpLXdT4_zbqvUTd@fedora> (raw)
Hi Jay,
We have a customer setup involving two separate switches with identical
L2/VLAN configurations. Each switch forms an independent aggregator
(port-channel), and the end host connects to both with the same number of
links and equivalent bandwidth.
As a result, the host ends up with two aggregators under a single bond
interface. Since the user cannot arbitrarily override port count or
bandwidth, they are asking for a new mechanism, lacp_prio, to influence
aggregator selection via ad_select.
Do you think this is a reasonable addition?
If yes, what would be the best way to compare priorities?
1. Port Priority Only. Currently initialized to 0xff. We could add a parameter
allowing users to configure it.
a) Use the highest port priority within each aggregator for comparison
b) Sum all port priorities in each aggregator and compare the totals
2. Full LACP Info Comparison. Compare fields such as system_priority, system,
port_priority, port_id, etc.
At present, the libteam code has implemented an approach that selects the
aggregator based on the highest system_priority/system from both local and
partner data.[1]
Looking forward to your thoughts.
Best regards,
Hangbin
[1] https://github.com/jpirko/libteam/blob/master/teamd/teamd_runner_lacp.c#L402
next reply other threads:[~2025-06-24 6:53 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-06-24 6:53 Hangbin Liu [this message]
2025-06-26 23:28 ` [Bonding Draft Proposal] Add lacp_prio Support for ad_select? Jay Vosburgh
2025-06-27 4:33 ` Hangbin Liu
2025-07-01 17:08 ` Jay Vosburgh
2025-07-02 7:44 ` Hangbin Liu
2025-07-03 19:42 ` Jay Vosburgh
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=aFpLXdT4_zbqvUTd@fedora \
--to=liuhangbin@gmail.com \
--cc=andrew+netdev@lunn.ch \
--cc=davem@davemloft.net \
--cc=edumazet@google.com \
--cc=jiri@resnulli.us \
--cc=jv@jvosburgh.net \
--cc=kuba@kernel.org \
--cc=netdev@vger.kernel.org \
--cc=pabeni@redhat.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