linux.git/drivers/net/bonding, branch v2.6.29-rc2 Clone of https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git bonding: use net_device_ops 2009-01-06T18:41:56+00:00 Stephen Hemminger shemminger@vyatta.com 2009-01-06T18:41:56+00:00 53a3294e26c49622daa14c1d8540500f568ded99 Use the correct pointer in debug message. Signed-off-by: Stephen Hemminger <shemminger@vyatta.com> Signed-off-by: David S. Miller <davem@davemloft.net> 
Use the correct pointer in debug message.

Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
802.3ad: use standard ethhdr instead of ad_header 2008-12-26T21:41:53+00:00 Holger Eitzenberger holger@eitzenberger.org 2008-12-26T21:41:53+00:00 e727149e26b08550269a7786fda977aed65578f6 802.3ad has its own ethhdr-like structure in the form of an ad_header, which is at the start of both the LACPDU and marker PDU. Both are the same from the struct values, both are packed as well. It's therefore perfectly fine to replace the ad_header by the ethhdr and to remove its definition. Signed-off-by: Holger Eitzenberger <holger@eitzenberger.org> Signed-off-by: David S. Miller <davem@davemloft.net> 
802.3ad has its own ethhdr-like structure in the form of an ad_header,
which is at the start of both the LACPDU and marker PDU.  Both are
the same from the struct values, both are packed as well.

It's therefore perfectly fine to replace the ad_header by the ethhdr
and to remove its definition.

Signed-off-by: Holger Eitzenberger <holger@eitzenberger.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
802.3ad: generalize out mac address initializer 2008-12-26T21:40:48+00:00 Holger Eitzenberger holger@eitzenberger.org 2008-12-26T21:40:48+00:00 e4ac43200fd6b8b8de3b487e74e74d3742f70d79 Generalize out mac address initializer for the LACPDU multicast address and use in two places. Remove the now unused AD_MULTICAST_LACPDU_ADDR. Signed-off-by: Holger Eitzenberger <holger@eitzenberger.org> Signed-off-by: David S. Miller <davem@davemloft.net> 
Generalize out mac address initializer for the LACPDU multicast
address and use in two places.  Remove the now unused
AD_MULTICAST_LACPDU_ADDR.

Signed-off-by: Holger Eitzenberger <holger@eitzenberger.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
802.3ad: initialize ports LACPDU from const initializer 2008-12-26T21:28:33+00:00 Holger Eitzenberger holger@eitzenberger.org 2008-12-26T21:28:33+00:00 7addeef6b15c9b7cffaaf40ec5b5b0940145ec16 Save some text by initializing ports LACPDU from const initializer, then get rid of ad_initialize_lacpdu(). Signed-off-by: Holger Eitzenberger <holger@eitzenberger.org> Signed-off-by: David S. Miller <davem@davemloft.net> 
Save some text by initializing ports LACPDU from const initializer,
then get rid of ad_initialize_lacpdu().

Signed-off-by: Holger Eitzenberger <holger@eitzenberger.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
802.3ad: remove typedef around ad_system 2008-12-26T21:27:57+00:00 Holger Eitzenberger holger@eitzenberger.org 2008-12-26T21:27:57+00:00 87f422f8618c541db0f96194cca9465817f76b89 As typedefs are considered a bad thing most of the time remove the typedef around ad_system. Signed-off-by: Holger Eitzenberger <heitzenberger@astaro.com> Signed-off-by: David S. Miller <davem@davemloft.net> 
As typedefs are considered a bad thing most of the time remove the
typedef around ad_system.

Signed-off-by: Holger Eitzenberger <heitzenberger@astaro.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
802.3ad: turn ports is_individual into a bool 2008-12-26T21:27:21+00:00 Holger Eitzenberger holger@eitzenberger.org 2008-12-26T21:27:21+00:00 1624db7be0d9f77f5494efe1b066d1d44b46c96c Turn ports is_individual into a bool. There is no functional change. Signed-off-by: Holger Eitzenberger <heitzenberger@astaro.com> Signed-off-by: David S. Miller <davem@davemloft.net> 
Turn ports is_individual into a bool.  There is no functional change.

Signed-off-by: Holger Eitzenberger <heitzenberger@astaro.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
802.3ad: turn ports is_enabled into a bool 2008-12-26T21:26:54+00:00 Holger Eitzenberger holger@eitzenberger.org 2008-12-26T21:26:54+00:00 f48127b6f3270cf7234a7a9f770533332e85e8ae Turn ports is_enabled into a bool. There is no functional change. Signed-off-by: Holger Eitzenberger <holger@eitzenberger.org> Signed-off-by: David S. Miller <davem@davemloft.net> 
Turn ports is_enabled into a bool.  There is no functional change.

Signed-off-by: Holger Eitzenberger <holger@eitzenberger.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
802.3ad: make ntt bool 2008-12-26T19:18:15+00:00 Holger Eitzenberger holger@eitzenberger.org 2008-12-26T19:18:15+00:00 d238d458a70ad134cb421c3ffc66242277f30dda Turn Need-To-Transmit port variable into a bool. There is no functional change. Signed-off-by: Holger Eitzenberger <holger@eitzenberger.org> Signed-off-by: David S. Miller <davem@davemloft.net> 
Turn Need-To-Transmit port variable into a bool.  There is no
functional change.

Signed-off-by: Holger Eitzenberger <holger@eitzenberger.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/bonding: fix sparse warnings: move decls to header file 2008-12-26T07:58:57+00:00 Hannes Eder hannes@hanneseder.net 2008-12-26T07:58:57+00:00 b06715b7a3db551dcf4706f05e8d2285a66fe05f Fix this sparse warnings: drivers/net/bonding/bond_main.c:104:20: warning: symbol 'bonding_defaults' was not declared. Should it be static? drivers/net/bonding/bond_main.c:204:22: warning: symbol 'ad_select_tbl' was not declared. Should it be static? drivers/net/bonding/bond_sysfs.c:60:21: warning: symbol 'bonding_rwsem' was not declared. Should it be static? Signed-off-by: Hannes Eder <hannes@hanneseder.net> Signed-off-by: David S. Miller <davem@davemloft.net> 
Fix this sparse warnings:

  drivers/net/bonding/bond_main.c:104:20: warning: symbol 'bonding_defaults' was not declared. Should it be static?
  drivers/net/bonding/bond_main.c:204:22: warning: symbol 'ad_select_tbl' was not declared. Should it be static?
  drivers/net/bonding/bond_sysfs.c:60:21: warning: symbol 'bonding_rwsem' was not declared. Should it be static?

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
bonding: use port_params in __update_lacpdu_from_port 2008-12-18T03:13:53+00:00 Holger Eitzenberger holger@eitzenberger.org 2008-12-18T03:13:53+00:00 3b5b35d02264580950d1e1565431b99d97c20ded Signed-off-by: Holger Eitzenberger <holger@eitzenberger.org> Signed-off-by: David S. Miller <davem@davemloft.net> 
Signed-off-by: Holger Eitzenberger <holger@eitzenberger.org>
Signed-off-by: David S. Miller <davem@davemloft.net>