Commit 1055c9ab authored by Holger Eitzenberger's avatar Holger Eitzenberger Committed by David S. Miller

bonding: introduce and use port_params structure

It generally helps to handle those values in various places, using it
might make the code more readable and gives room for other improvements.

The IEEE standard talks about them as "parameter values".
Signed-off-by: default avatarHolger Eitzenberger <holger@eitzenberger.org>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent aa312819
This diff is collapsed.
...@@ -199,6 +199,15 @@ typedef struct aggregator { ...@@ -199,6 +199,15 @@ typedef struct aggregator {
u16 num_of_ports; u16 num_of_ports;
} aggregator_t; } aggregator_t;
struct port_params {
struct mac_addr system;
u16 system_priority;
u16 key;
u16 port_number;
u16 port_priority;
u16 port_state;
};
// port structure(43.4.6 in the 802.3ad standard) // port structure(43.4.6 in the 802.3ad standard)
typedef struct port { typedef struct port {
u16 actor_port_number; u16 actor_port_number;
...@@ -211,18 +220,10 @@ typedef struct port { ...@@ -211,18 +220,10 @@ typedef struct port {
u16 actor_oper_port_key; u16 actor_oper_port_key;
u8 actor_admin_port_state; u8 actor_admin_port_state;
u8 actor_oper_port_state; u8 actor_oper_port_state;
struct mac_addr partner_admin_system;
struct mac_addr partner_oper_system; struct port_params partner_admin;
u16 partner_admin_system_priority; struct port_params partner_oper;
u16 partner_oper_system_priority;
u16 partner_admin_key;
u16 partner_oper_key;
u16 partner_admin_port_number;
u16 partner_oper_port_number;
u16 partner_admin_port_priority;
u16 partner_oper_port_priority;
u8 partner_admin_port_state;
u8 partner_oper_port_state;
u16 is_enabled; // BOOLEAN u16 is_enabled; // BOOLEAN
// ****** PRIVATE PARAMETERS ****** // ****** PRIVATE PARAMETERS ******
u16 sm_vars; // all state machines variables for this port u16 sm_vars; // all state machines variables for this port
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment