Browse Source

[lacp] Set "aggregatable" flag in response LACPDU

Some switches do not allow an individual link (as defined in IEEE Std
802.3ad-2000 section 43.3.5) to work alone in a link aggregation group
as described in section 43.3.6.  This is verified on Dell's
PowerConnect M6220, based on the Broadcom Strata XGS-IV chipset.

Set the LACP_STATE_AGGREGATABLE flag in the actor.state field to
announce link aggregation in the response LACPDU, which will have the
switch enable the link aggregation group and allow frames to pass.

Signed-off-by: Michael Brown <mcb30@ipxe.org>
tags/v1.20.1
Sven Ulland 10 years ago
parent
commit
de65a240b9
1 changed files with 2 additions and 1 deletions
  1. 2
    1
      src/net/eth_slow.c

+ 2
- 1
src/net/eth_slow.c View File

@@ -167,7 +167,8 @@ static int eth_slow_lacp_rx ( struct io_buffer *iobuf,
167 167
 	lacp->actor.key = htons ( 1 );
168 168
 	lacp->actor.port_priority = htons ( LACP_PORT_PRIORITY_MAX );
169 169
 	lacp->actor.port = htons ( 1 );
170
-	lacp->actor.state = ( LACP_STATE_IN_SYNC |
170
+	lacp->actor.state = ( LACP_STATE_AGGREGATABLE |
171
+			      LACP_STATE_IN_SYNC |
171 172
 			      LACP_STATE_COLLECTING |
172 173
 			      LACP_STATE_DISTRIBUTING |
173 174
 			      ( lacp->partner.state & LACP_STATE_FAST ) );

Loading…
Cancel
Save