netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Krisztian Ivancso <github-ivan@ivancso.net>
To: Ding Tianhong <dingtianhong@huawei.com>
Cc: netdev@vger.kernel.org
Subject: Re: [PATCH net-next] bonding: lacp_port_id setting for 802.3ad
Date: Tue, 13 Aug 2013 13:00:29 +0200	[thread overview]
Message-ID: <520A11CD.8060400@ivancso.net> (raw)
In-Reply-To: <5209FEBC.4050305@huawei.com>

On 08/13/2013 11:39 AM, Ding Tianhong wrote:
> On 2013/8/13 17:20, Krisztian Ivancso wrote:
>> On 08/13/2013 03:07 AM, Ding Tianhong wrote:
>>> On 2013/8/12 19:19, Krisztian Ivancso wrote:
>>>> >From 472fffa5a8f170daed9e4cc677af8e2560b86be2 Mon Sep 17 00:00:00 2001
>>>> From: Krisztian Ivancso <github-ivan@ivancso.net>
>>>> Date: Sun, 11 Aug 2013 20:30:44 +0200
>>>> Subject: [PATCH net-next] bonding: lacp_port_id setting for 802.3ad ports
> 
> ok, for example: the bonding has four slave, slave1 and slave2 aggregation to 1 group,
> and slave3 and slave4 aggregtion to 2 group, how you distinguish the 1 and 2 group by initialize id.

this is not possible, because all slave have to be a member of the same
aggregation group.

i think we misunderstood each other.

here is a new example:
- switch1 is a switch with a configured lag with two members ports
  (member1 and member2)
- two linux (linux1 and linux2) box with a configured bonding device
  (bond0) with the same MAC set in both box and one
  slave on each
- lacp_port_id is set to 10 in linux1 and 20 in linux2

you can attach the slave from both linux boxes to the same
lag on switch1. (slave from linux1 to port member1 and
slave from linux2 to port member2 on switch1)

port id must be unique within a system.
bonding implementation set a unique system id for every bonding device
which is derived from MAC of one of the slave interfaces.

if we use the current bonding implementation second linux box can't be
a member on switch1 because port id is 1 in both linux bonding device.

if we can set different starting port id for bonding in different boxes
the second box can be a member also.

  reply	other threads:[~2013-08-13 11:00 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-08-12 11:19 [PATCH net-next] bonding: lacp_port_id setting for 802.3ad Krisztian Ivancso
2013-08-13  1:07 ` Ding Tianhong
2013-08-13  9:20   ` Krisztian Ivancso
2013-08-13  9:39     ` Ding Tianhong
2013-08-13 11:00       ` Krisztian Ivancso [this message]
2013-08-13 18:25         ` Jay Vosburgh
2013-08-13 23:34           ` Krisztian Ivancso
2013-08-14  1:46             ` Ding Tianhong
2013-08-16  2:07               ` Krisztian Ivancso

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=520A11CD.8060400@ivancso.net \
    --to=github-ivan@ivancso.net \
    --cc=dingtianhong@huawei.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 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).