|
Thomas Bogendoerfer |
aef6c4 |
From: Jiasheng Jiang <jiasheng@iscas.ac.cn>
|
|
Thomas Bogendoerfer |
aef6c4 |
Date: Mon, 20 Dec 2021 22:03:44 +0800
|
|
Thomas Bogendoerfer |
aef6c4 |
Subject: sfc: falcon: Check null pointer of rx_queue->page_ring
|
|
Thomas Bogendoerfer |
aef6c4 |
Patch-mainline: v5.16-rc7
|
|
Thomas Bogendoerfer |
aef6c4 |
Git-commit: 9b8bdd1eb5890aeeab7391dddcf8bd51f7b07216
|
|
Thomas Bogendoerfer |
aef6c4 |
References: git-fixes
|
|
Thomas Bogendoerfer |
aef6c4 |
|
|
Thomas Bogendoerfer |
aef6c4 |
Because of the possible failure of the kcalloc, it should be better to
|
|
Thomas Bogendoerfer |
aef6c4 |
set rx_queue->page_ptr_mask to 0 when it happens in order to maintain
|
|
Thomas Bogendoerfer |
aef6c4 |
the consistency.
|
|
Thomas Bogendoerfer |
aef6c4 |
|
|
Thomas Bogendoerfer |
aef6c4 |
Fixes: 5a6681e22c14 ("sfc: separate out SFC4000 ("Falcon") support into new sfc-falcon driver")
|
|
Thomas Bogendoerfer |
aef6c4 |
Signed-off-by: Jiasheng Jiang <jiasheng@iscas.ac.cn>
|
|
Thomas Bogendoerfer |
aef6c4 |
Acked-by: Martin Habets <habetsm.xilinx@gmail.com>
|
|
Thomas Bogendoerfer |
aef6c4 |
Link: https://lore.kernel.org/r/20211220140344.978408-1-jiasheng@iscas.ac.cn
|
|
Thomas Bogendoerfer |
aef6c4 |
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
|
|
Thomas Bogendoerfer |
aef6c4 |
Acked-by: Thomas Bogendoerfer <tbogendoerfer@suse.de>
|
|
Thomas Bogendoerfer |
aef6c4 |
---
|
|
Thomas Bogendoerfer |
aef6c4 |
drivers/net/ethernet/sfc/falcon/rx.c | 5 ++++-
|
|
Thomas Bogendoerfer |
aef6c4 |
1 file changed, 4 insertions(+), 1 deletion(-)
|
|
Thomas Bogendoerfer |
aef6c4 |
|
|
Thomas Bogendoerfer |
aef6c4 |
--- a/drivers/net/ethernet/sfc/falcon/rx.c
|
|
Thomas Bogendoerfer |
aef6c4 |
+++ b/drivers/net/ethernet/sfc/falcon/rx.c
|
|
Thomas Bogendoerfer |
aef6c4 |
@@ -726,7 +726,10 @@ static void ef4_init_rx_recycle_ring(str
|
|
Thomas Bogendoerfer |
aef6c4 |
efx->rx_bufs_per_page);
|
|
Thomas Bogendoerfer |
aef6c4 |
rx_queue->page_ring = kcalloc(page_ring_size,
|
|
Thomas Bogendoerfer |
aef6c4 |
sizeof(*rx_queue->page_ring), GFP_KERNEL);
|
|
Thomas Bogendoerfer |
aef6c4 |
- rx_queue->page_ptr_mask = page_ring_size - 1;
|
|
Thomas Bogendoerfer |
aef6c4 |
+ if (!rx_queue->page_ring)
|
|
Thomas Bogendoerfer |
aef6c4 |
+ rx_queue->page_ptr_mask = 0;
|
|
Thomas Bogendoerfer |
aef6c4 |
+ else
|
|
Thomas Bogendoerfer |
aef6c4 |
+ rx_queue->page_ptr_mask = page_ring_size - 1;
|
|
Thomas Bogendoerfer |
aef6c4 |
}
|
|
Thomas Bogendoerfer |
aef6c4 |
|
|
Thomas Bogendoerfer |
aef6c4 |
void ef4_init_rx_queue(struct ef4_rx_queue *rx_queue)
|