diff options
| author | Arkadiusz Kubalewski <arkadiusz.kubalewski@intel.com> | 2021-03-26 19:43:40 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2021-04-14 08:42:01 +0200 |
| commit | 89e406e95278628ae0337ae3fa3c85659c88514f (patch) | |
| tree | dd2c81c949eec7330177936d29bd6371121063c9 /net/sched/sch_fifo.c | |
| parent | 12e1438a0946434906303b3682d363e0d0efc3a2 (diff) | |
| download | kernel_replicant_linux-89e406e95278628ae0337ae3fa3c85659c88514f.tar.gz kernel_replicant_linux-89e406e95278628ae0337ae3fa3c85659c88514f.tar.bz2 kernel_replicant_linux-89e406e95278628ae0337ae3fa3c85659c88514f.zip | |
i40e: Fix sparse errors in i40e_txrx.c
commit 12738ac4754ec92a6a45bf3677d8da780a1412b3 upstream.
Remove error handling through pointers. Instead use plain int
to return value from i40e_run_xdp(...).
Previously:
- sparse errors were produced during compilation:
i40e_txrx.c:2338 i40e_run_xdp() error: (-2147483647) too low for ERR_PTR
i40e_txrx.c:2558 i40e_clean_rx_irq() error: 'skb' dereferencing possible ERR_PTR()
- sk_buff* was used to return value, but it has never had valid
pointer to sk_buff. Returned value was always int handled as
a pointer.
Fixes: 0c8493d90b6b ("i40e: add XDP support for pass and drop actions")
Fixes: 2e6893123830 ("i40e: split XDP_TX tail and XDP_REDIRECT map flushing")
Signed-off-by: Aleksandr Loktionov <aleksandr.loktionov@intel.com>
Signed-off-by: Arkadiusz Kubalewski <arkadiusz.kubalewski@intel.com>
Tested-by: Dave Switzer <david.switzer@intel.com>
Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'net/sched/sch_fifo.c')
0 files changed, 0 insertions, 0 deletions