netdev - Re: [PATCH net-next 6/6] team: remove unused input parameters in lb_htpm_select_tx_port and lb_hash_select_tx

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date: Sat, 5 Aug 2023 10:04:10 +0800
From: shaozhengchao <shaozhengchao@...wei.com>
To: Jiri Pirko <jiri@...nulli.us>
CC: <netdev@...r.kernel.org>, <bpf@...r.kernel.org>, <davem@...emloft.net>,
	<edumazet@...gle.com>, <kuba@...nel.org>, <pabeni@...hat.com>,
	<weiyongjun1@...wei.com>, <yuehaibing@...wei.com>
Subject: Re: [PATCH net-next 6/6] team: remove unused input parameters in
 lb_htpm_select_tx_port and lb_hash_select_tx_port



On 2023/8/5 0:56, Jiri Pirko wrote:
> Fri, Aug 04, 2023 at 04:06:39PM CEST, shaozhengchao@...wei.com wrote:
>>
>>
>> On 2023/8/4 21:45, Jiri Pirko wrote:
>>> Fri, Aug 04, 2023 at 02:31:16PM CEST, shaozhengchao@...wei.com 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@...wei.com>
>>>> ---
>>>> 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!
>>>
>> Hi Jiri:
>> 	Thank you for your review. I will change it and send v2.
> 
> Did you hear about netdev rules by any chance?
> 
> https://www.kernel.org/doc/html/v6.5-rc4/process/maintainer-netdev.html?highlight=24h
> 
> Could you please read it?
> 
Thank you. I will read it.

Zhengchao Shao
> 
>>
>> Zhengchao Shao
>>>
>>>> -						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
>>>>
>>>
>