Skip to content

Commit

Permalink
team: use function team_port_txable() for determing enabled and up port
Browse files Browse the repository at this point in the history
Signed-off-by: Jiri Pirko <jpirko@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
Jiri Pirko authored and David S. Miller committed Jul 12, 2012
1 parent f0a70e9 commit 6e88e13
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 3 deletions.
6 changes: 6 additions & 0 deletions drivers/net/team/team.c
Original file line number Diff line number Diff line change
Expand Up @@ -677,6 +677,12 @@ bool team_port_enabled(struct team_port *port)
}
EXPORT_SYMBOL(team_port_enabled);

bool team_port_txable(struct team_port *port)
{
return port->linkup && team_port_enabled(port);
}
EXPORT_SYMBOL(team_port_txable);

/*
* Enable/disable port by adding to enabled port hashlist and setting
* port->index (Might be racy so reader could see incorrect ifindex when
Expand Down
6 changes: 3 additions & 3 deletions drivers/net/team/team_mode_roundrobin.c
Original file line number Diff line number Diff line change
Expand Up @@ -30,16 +30,16 @@ static struct team_port *__get_first_port_up(struct team *team,
{
struct team_port *cur;

if (port->linkup)
if (team_port_txable(port))
return port;
cur = port;
list_for_each_entry_continue_rcu(cur, &team->port_list, list)
if (cur->linkup)
if (team_port_txable(port))
return cur;
list_for_each_entry_rcu(cur, &team->port_list, list) {
if (cur == port)
break;
if (cur->linkup)
if (team_port_txable(port))
return cur;
}
return NULL;
Expand Down
1 change: 1 addition & 0 deletions include/linux/if_team.h
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@ struct team_port {
};

extern bool team_port_enabled(struct team_port *port);
extern bool team_port_txable(struct team_port *port);

struct team_mode_ops {
int (*init)(struct team *team);
Expand Down

0 comments on commit 6e88e13

Please sign in to comment.