[prev in list] [next in list] [prev in thread] [next in thread]
List: netbsd-tech-net
Subject: Re: CVS commit: src/sys
From: yamt () mwd ! biglobe ! ne ! jp (YAMAMOTO Takashi)
Date: 2009-06-09 22:37:57
Message-ID: 20090609223758.100A863B126 () mail ! netbsd ! org
[Download RAW message or body]
hi,
thanks for working on this.
> Module Name: src
> Committed By: darran
> Date: Fri May 29 04:57:05 UTC 2009
>
> Modified Files:
> src/sys/dev/pci: if_wm.c
> src/sys/net: if_ethersubr.c
> src/sys/net/agr: ieee8023ad_lacp.c if_agr.c if_agrether.c
> if_agrvar_impl.h
>
> Log Message:
> Add vlan support and hardware offload capabilities to agr.
> These changes allow vlans to be layered above agr, with the attach
> and detach propogated to the member ports in the aggregation.
> Note the agr interface must be up before the vlan is attached.
>
> Adds SIOCINITIFADDR support to the wm driver for setting the AF_LINK
> address, necessary for agr to be able to set the mac addresses of each
> port to the agr address (i.e. so it can receive all intended traffic
> at the hardware level).
>
> Adds support for disabling the LACP protocol by setting LINK1 on the agr
> interface (e.g. ifconfig agr0 link1).
i'm not happy with using link1 for the purpose.
1 bit is not enough to specify a configuration mechanism.
IMO it should be able to be specified with a string like
"lacp" "static" etc. (or probably with a plist with configuration
mechanism specific parameters like port priority etc.)
> In consultation with tls@.
>
>
> To generate a diff of this commit:
> cvs rdiff -u -r1.174 -r1.175 src/sys/dev/pci/if_wm.c
> cvs rdiff -u -r1.171 -r1.172 src/sys/net/if_ethersubr.c
> cvs rdiff -u -r1.8 -r1.9 src/sys/net/agr/ieee8023ad_lacp.c
i'm not happy with using _lacp.c file for static configuration.
with static configuration, the lacp state machine should not run
in the first place.
> cvs rdiff -u -r1.22 -r1.23 src/sys/net/agr/if_agr.c
DNH_DEBUG?
> cvs rdiff -u -r1.6 -r1.7 src/sys/net/agr/if_agrether.c
as your comment suggets, capability should be intersection of
underlying ports. what's wrong with doing it in _portinit/portfini?
YAMAMOTO Takashi
> cvs rdiff -u -r1.7 -r1.8 src/sys/net/agr/if_agrvar_impl.h
>
> Please note that diffs are not public domain; they are subject to the
> copyright notices on the relevant files.
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic