RE: [Intel-wired-lan] [PATCH v4 net-next 4/4] ixgbevf: Fix passing 0 to ERR_PTR in ixgbevf_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 4/4] ixgbevf: Fix passing 0 to
>ERR_PTR in ixgbevf_run_xdp()
>
>ixgbevf_run_xdp() converts customed xdp action to a negative error code with
>the sk_buff pointer type which be checked with IS_ERR in
>ixgbevf_clean_rx_irq(). Remove this error pointer handing instead use plain int
>return value.
>
>Fixes: c7aec59657b6 ("ixgbevf: Add XDP support for pass and drop actions")
>Reviewed-by: Jacob Keller <jacob.e.keller@xxxxxxxxx>
>Reviewed-by: Maciej Fijalkowski <maciej.fijalkowski@xxxxxxxxx>
>Signed-off-by: Yue Haibing <yuehaibing@xxxxxxxxxx>
>---
> .../net/ethernet/intel/ixgbevf/ixgbevf_main.c | 23 ++++++++-----------
> 1 file changed, 10 insertions(+), 13 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