* [PATCH net] cxgb4: dont touch blocked freelist bitmap after free
@ 2021-08-25 21:29 Rahul Lakkireddy
2021-08-26 9:30 ` patchwork-bot+netdevbpf
0 siblings, 1 reply; 2+ messages in thread
From: Rahul Lakkireddy @ 2021-08-25 21:29 UTC (permalink / raw)
To: netdev; +Cc: davem, kuba, rajur
When adapter init fails, the blocked freelist bitmap is already freed
up and should not be touched. So, move the bitmap zeroing closer to
where it was successfully allocated. Also handle adapter init failure
unwind path immediately and avoid setting up RDMA memory windows.
Fixes: 5b377d114f2b ("cxgb4: Add debugfs facility to inject FL starvation")
Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
---
drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c | 7 +++----
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c b/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c
index dbf9a0e6601d..710cb00ce3a3 100644
--- a/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c
+++ b/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c
@@ -5068,6 +5068,7 @@ static int adap_init0(struct adapter *adap, int vpd_skip)
ret = -ENOMEM;
goto bye;
}
+ bitmap_zero(adap->sge.blocked_fl, adap->sge.egr_sz);
#endif
params[0] = FW_PARAM_PFVF(CLIP_START);
@@ -6788,13 +6789,11 @@ static int init_one(struct pci_dev *pdev, const struct pci_device_id *ent)
setup_memwin(adapter);
err = adap_init0(adapter, 0);
-#ifdef CONFIG_DEBUG_FS
- bitmap_zero(adapter->sge.blocked_fl, adapter->sge.egr_sz);
-#endif
- setup_memwin_rdma(adapter);
if (err)
goto out_unmap_bar;
+ setup_memwin_rdma(adapter);
+
/* configure SGE_STAT_CFG_A to read WC stats */
if (!is_t4(adapter->params.chip))
t4_write_reg(adapter, SGE_STAT_CFG_A, STATSOURCE_T5_V(7) |
--
2.27.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH net] cxgb4: dont touch blocked freelist bitmap after free
2021-08-25 21:29 [PATCH net] cxgb4: dont touch blocked freelist bitmap after free Rahul Lakkireddy
@ 2021-08-26 9:30 ` patchwork-bot+netdevbpf
0 siblings, 0 replies; 2+ messages in thread
From: patchwork-bot+netdevbpf @ 2021-08-26 9:30 UTC (permalink / raw)
To: Rahul Lakkireddy; +Cc: netdev, davem, kuba, rajur
Hello:
This patch was applied to netdev/net.git (refs/heads/master):
On Thu, 26 Aug 2021 02:59:42 +0530 you wrote:
> When adapter init fails, the blocked freelist bitmap is already freed
> up and should not be touched. So, move the bitmap zeroing closer to
> where it was successfully allocated. Also handle adapter init failure
> unwind path immediately and avoid setting up RDMA memory windows.
>
> Fixes: 5b377d114f2b ("cxgb4: Add debugfs facility to inject FL starvation")
> Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
>
> [...]
Here is the summary with links:
- [net] cxgb4: dont touch blocked freelist bitmap after free
https://git.kernel.org/netdev/net/c/43fed4d48d32
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-08-26 9:30 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-25 21:29 [PATCH net] cxgb4: dont touch blocked freelist bitmap after free Rahul Lakkireddy
2021-08-26 9:30 ` patchwork-bot+netdevbpf
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.