Fri, Aug 04, 2023 at 02:31:16PM CEST, shaozhengchao@xxxxxxxxxx wrote:
>The input parameters "lb_priv" and "skb" in lb_htpm_select_tx_port and
>lb_hash_select_tx_port are unused, so remove them.
>
>Signed-off-by: Zhengchao Shao <shaozhengchao@xxxxxxxxxx>
>---
> drivers/net/team/team_mode_loadbalance.c | 10 ++--------
> 1 file changed, 2 insertions(+), 8 deletions(-)
>
>diff --git a/drivers/net/team/team_mode_loadbalance.c b/drivers/net/team/team_mode_loadbalance.c
>index a6021ae51d0d..00f8989c29c0 100644
>--- a/drivers/net/team/team_mode_loadbalance.c
>+++ b/drivers/net/team/team_mode_loadbalance.c
>@@ -30,8 +30,6 @@ static rx_handler_result_t lb_receive(struct team *team, struct team_port *port,
> struct lb_priv;
>
> typedef struct team_port *lb_select_tx_port_func_t(struct team *,
>- struct lb_priv *,
>- struct sk_buff *,
> unsigned char);
>
> #define LB_TX_HASHTABLE_SIZE 256 /* hash is a char */
>@@ -118,8 +116,6 @@ static void lb_tx_hash_to_port_mapping_null_port(struct team *team,
>
> /* Basic tx selection based solely by hash */
> static struct team_port *lb_hash_select_tx_port(struct team *team,
>- struct lb_priv *lb_priv,
>- struct sk_buff *skb,
> unsigned char hash)
> {
> int port_index = team_num_to_port_index(team, hash);
>@@ -129,8 +125,6 @@ static struct team_port *lb_hash_select_tx_port(struct team *team,
>
> /* Hash to port mapping select tx port */
> static struct team_port *lb_htpm_select_tx_port(struct team *team,
>- struct lb_priv *lb__priv,
Squash the previous patch in this one to avoid this odd "__" thing.
Thanks!
>- struct sk_buff *skb,
> unsigned char hash)
> {
> struct lb_priv *lb_priv = get_lb_priv(team);
>@@ -140,7 +134,7 @@ static struct team_port *lb_htpm_select_tx_port(struct team *team,
> if (likely(port))
> return port;
> /* If no valid port in the table, fall back to simple hash */
>- return lb_hash_select_tx_port(team, lb_priv, skb, hash);
>+ return lb_hash_select_tx_port(team, hash);
> }
>
> struct lb_select_tx_port {
>@@ -230,7 +224,7 @@ static bool lb_transmit(struct team *team, struct sk_buff *skb)
>
> hash = lb_get_skb_hash(lb_priv, skb);
> select_tx_port_func = rcu_dereference_bh(lb_priv->select_tx_port_func);
>- port = select_tx_port_func(team, lb_priv, skb, hash);
>+ port = select_tx_port_func(team, hash);
> if (unlikely(!port))
> goto drop;
> if (team_dev_queue_xmit(team, port, skb))
>--
>2.34.1
>
