bonding: use plain memcpy in __record_default()
authorHolger Eitzenberger <holger@eitzenberger.org>
Thu, 18 Dec 2008 03:08:46 +0000 (19:08 -0800)
committerDavid S. Miller <davem@davemloft.net>
Thu, 18 Dec 2008 03:08:46 +0000 (19:08 -0800)
Also remove the pointless comment at the top.

Signed-off-by: Holger Eitzenberger <holger@eitzenberger.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/bonding/bond_3ad.c

index b789dcb776269a05e45e7ec7d03b7214c4f9d714..0674d4149808ca575ca57addcd31c11007f74920 100644 (file)
@@ -516,15 +516,10 @@ static void __record_pdu(struct lacpdu *lacpdu, struct port *port)
  */
 static void __record_default(struct port *port)
 {
-       // validate the port
        if (port) {
                // record the partner admin parameters
-               port->partner_oper.port_number = port->partner_admin.port_number;
-               port->partner_oper.port_priority = port->partner_admin.port_priority;
-               port->partner_oper.system = port->partner_admin.system;
-               port->partner_oper.system_priority = port->partner_admin.system_priority;
-               port->partner_oper.key = port->partner_admin.key;
-               port->partner_oper.port_state = port->partner_admin.port_state;
+               memcpy(&port->partner_oper, &port->partner_admin,
+                      sizeof(struct port_params));
 
                // set actor_oper_port_state.defaulted to true
                port->actor_oper_port_state |= AD_STATE_DEFAULTED;