RE: [Intel-wired-lan] [PATCH v4 net-next 2/4] igb: Fix passing 0 to ERR_PTR in igb_run_xdp()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




>-----Original Message-----
>From: Intel-wired-lan <intel-wired-lan-bounces@xxxxxxxxxx> On Behalf Of Yue
>Haibing
>Sent: Saturday, October 26, 2024 9:43 AM
>To: Nguyen, Anthony L <anthony.l.nguyen@xxxxxxxxx>; Kitszel, Przemyslaw
><przemyslaw.kitszel@xxxxxxxxx>; davem@xxxxxxxxxxxxx;
>edumazet@xxxxxxxxxx; kuba@xxxxxxxxxx; pabeni@xxxxxxxxxx;
>ast@xxxxxxxxxx; daniel@xxxxxxxxxxxxx; hawk@xxxxxxxxxx;
>john.fastabend@xxxxxxxxx; Fijalkowski, Maciej
><maciej.fijalkowski@xxxxxxxxx>; vedang.patel@xxxxxxxxx; Joseph, Jithu
><jithu.joseph@xxxxxxxxx>; andre.guedes@xxxxxxxxx; horms@xxxxxxxxxx; Keller,
>Jacob E <jacob.e.keller@xxxxxxxxx>; sven.auhagen@xxxxxxxxxxxx;
>alexander.h.duyck@xxxxxxxxx
>Cc: intel-wired-lan@xxxxxxxxxxxxxxxx; netdev@xxxxxxxxxxxxxxx; linux-
>kernel@xxxxxxxxxxxxxxx; bpf@xxxxxxxxxxxxxxx; yuehaibing@xxxxxxxxxx
>Subject: [Intel-wired-lan] [PATCH v4 net-next 2/4] igb: Fix passing 0 to ERR_PTR
>in igb_run_xdp()
>
>igb_run_xdp() converts customed xdp action to a negative error code with the
>sk_buff pointer type which be checked with IS_ERR in igb_clean_rx_irq().
>Remove this error pointer handing instead use plain int return value.
>
>Fixes: 9cbc948b5a20 ("igb: add XDP support")
>Reviewed-by: Maciej Fijalkowski <maciej.fijalkowski@xxxxxxxxx>
>Reviewed-by: Jacob Keller <jacob.e.keller@xxxxxxxxx>
>Signed-off-by: Yue Haibing <yuehaibing@xxxxxxxxxx>
>---
> drivers/net/ethernet/intel/igb/igb_main.c | 22 ++++++++--------------
> 1 file changed, 8 insertions(+), 14 deletions(-)
>

Tested-by: Chandan Kumar Rout <chandanx.rout@xxxxxxxxx> (A Contingent Worker at Intel)






[Index of Archives]     [Linux Samsung SoC]     [Linux Rockchip SoC]     [Linux Actions SoC]     [Linux for Synopsys ARC Processors]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]


  Powered by Linux