[lvc-project] [PATCH] chelsio: delete the line with the pIdx initialization

[Fedor Pchelkin]

Для патчей в сетевую подсистему требуют указывать целевое гит-дерево в
теме письма.  Если багфикс, то net, иначе — net-next.
https://www.kernel.org/doc/html/latest/process/maintainer-netdev.html#tl-dr

On Mon, 25. May 15:43, Markov Gleb wrote:
> From: Gleb Markov <markov.gi at npc-ksb.ru>
> 
> The value of pIdx is overwritten immediately after exiting
> the "if" block.
> 
> Remove pIdx ptr initialization string from conditional block.

Насколько вижу, переменная называется pidx, не pIdx. (по крайней мере в
текущем состоянии master ветки).

> 
> Found by Linux Verification Center (linuxtesting.org) with SVACE.
> 
> Fixes: f1d3d38af757 ("[PATCH] chelsio: add support for other 10G boards")

Тег Fixes точно стоит убрать.  Вообще говоря, этот патч прямо очень
низкого приоритета и не обязателен по Программе.  Учитывая достаточно
большое количество других более важных с точки зрения severity задач,
советую в первую очередь браться за них и доводить до приемлемого
исправления.

Текущее удаление лишнего кода очень низкоприоритетно.

> Signed-off-by: Gleb Markov <markov.gi at npc-ksb.ru>
> ---
>  drivers/net/ethernet/chelsio/cxgb/sge.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/net/ethernet/chelsio/cxgb/sge.c b/drivers/net/ethernet/chelsio/cxgb/sge.c
> index 861edff5ed89..4e93d82d5892 100644
> --- a/drivers/net/ethernet/chelsio/cxgb/sge.c
> +++ b/drivers/net/ethernet/chelsio/cxgb/sge.c
> @@ -1743,7 +1743,6 @@ static int t1_sge_tx(struct sk_buff *skb, struct adapter *adapter,
>  			spin_unlock(&q->lock);
>  			return NETDEV_TX_OK;
>  		}
> -		pidx = q->pidx;
>  		count = 1 + skb_shinfo(skb)->nr_frags;
>  		count += compute_large_page_tx_descs(skb);
>  	}
> -- 
> 2.43.0