Commit d299cd51 authored by Jiri Pirko's avatar Jiri Pirko Committed by David S. Miller

team: for nomode use dummy struct team_mode

That leaves team->mode and all its values valid so no checks would be
needed (for example in team_mode_option_get()).
Signed-off-by: default avatarJiri Pirko <jpirko@redhat.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 0402788a
...@@ -479,6 +479,20 @@ rx_handler_result_t team_dummy_receive(struct team *team, ...@@ -479,6 +479,20 @@ rx_handler_result_t team_dummy_receive(struct team *team,
return RX_HANDLER_ANOTHER; return RX_HANDLER_ANOTHER;
} }
static const struct team_mode __team_no_mode = {
.kind = "*NOMODE*",
};
static bool team_is_mode_set(struct team *team)
{
return team->mode != &__team_no_mode;
}
static void team_set_no_mode(struct team *team)
{
team->mode = &__team_no_mode;
}
static void team_adjust_ops(struct team *team) static void team_adjust_ops(struct team *team)
{ {
/* /*
...@@ -487,13 +501,13 @@ static void team_adjust_ops(struct team *team) ...@@ -487,13 +501,13 @@ static void team_adjust_ops(struct team *team)
*/ */
if (list_empty(&team->port_list) || if (list_empty(&team->port_list) ||
!team->mode || !team->mode->ops->transmit) !team_is_mode_set(team) || !team->mode->ops->transmit)
team->ops.transmit = team_dummy_transmit; team->ops.transmit = team_dummy_transmit;
else else
team->ops.transmit = team->mode->ops->transmit; team->ops.transmit = team->mode->ops->transmit;
if (list_empty(&team->port_list) || if (list_empty(&team->port_list) ||
!team->mode || !team->mode->ops->receive) !team_is_mode_set(team) || !team->mode->ops->receive)
team->ops.receive = team_dummy_receive; team->ops.receive = team_dummy_receive;
else else
team->ops.receive = team->mode->ops->receive; team->ops.receive = team->mode->ops->receive;
...@@ -508,7 +522,7 @@ static int __team_change_mode(struct team *team, ...@@ -508,7 +522,7 @@ static int __team_change_mode(struct team *team,
const struct team_mode *new_mode) const struct team_mode *new_mode)
{ {
/* Check if mode was previously set and do cleanup if so */ /* Check if mode was previously set and do cleanup if so */
if (team->mode) { if (team_is_mode_set(team)) {
void (*exit_op)(struct team *team) = team->ops.exit; void (*exit_op)(struct team *team) = team->ops.exit;
/* Clear ops area so no callback is called any longer */ /* Clear ops area so no callback is called any longer */
...@@ -518,7 +532,7 @@ static int __team_change_mode(struct team *team, ...@@ -518,7 +532,7 @@ static int __team_change_mode(struct team *team,
if (exit_op) if (exit_op)
exit_op(team); exit_op(team);
team_mode_put(team->mode); team_mode_put(team->mode);
team->mode = NULL; team_set_no_mode(team);
/* zero private data area */ /* zero private data area */
memset(&team->mode_priv, 0, memset(&team->mode_priv, 0,
sizeof(struct team) - offsetof(struct team, mode_priv)); sizeof(struct team) - offsetof(struct team, mode_priv));
...@@ -553,7 +567,7 @@ static int team_change_mode(struct team *team, const char *kind) ...@@ -553,7 +567,7 @@ static int team_change_mode(struct team *team, const char *kind)
return -EBUSY; return -EBUSY;
} }
if (team->mode && strcmp(team->mode->kind, kind) == 0) { if (team_is_mode_set(team) && strcmp(team->mode->kind, kind) == 0) {
netdev_err(dev, "Unable to change to the same mode the team is in\n"); netdev_err(dev, "Unable to change to the same mode the team is in\n");
return -EINVAL; return -EINVAL;
} }
...@@ -912,11 +926,9 @@ static int team_port_del(struct team *team, struct net_device *port_dev) ...@@ -912,11 +926,9 @@ static int team_port_del(struct team *team, struct net_device *port_dev)
* Net device ops * Net device ops
*****************/ *****************/
static const char team_no_mode_kind[] = "*NOMODE*";
static int team_mode_option_get(struct team *team, struct team_gsetter_ctx *ctx) static int team_mode_option_get(struct team *team, struct team_gsetter_ctx *ctx)
{ {
ctx->data.str_val = team->mode ? team->mode->kind : team_no_mode_kind; ctx->data.str_val = team->mode->kind;
return 0; return 0;
} }
...@@ -1014,6 +1026,7 @@ static int team_init(struct net_device *dev) ...@@ -1014,6 +1026,7 @@ static int team_init(struct net_device *dev)
team->dev = dev; team->dev = dev;
mutex_init(&team->lock); mutex_init(&team->lock);
team_set_no_mode(team);
team->pcpu_stats = alloc_percpu(struct team_pcpu_stats); team->pcpu_stats = alloc_percpu(struct team_pcpu_stats);
if (!team->pcpu_stats) if (!team->pcpu_stats)
......
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