[PATCH] net/bnxt: fix deadlock in mgr timer cb
Ajit Khaparde
ajit.khaparde at broadcom.com
Thu Feb 8 22:45:13 CET 2024
On Fri, Nov 3, 2023 at 9:07 PM Weiguo Li <liwg06 at foxmail.com> wrote:
>
> The function 'ulp_ha_mgr_timer_cb' acquires a lock on the context
> entry at the beginning with bnxt_ulp_cntxt_entry_acquire(). This lock
> is expected to be released by bnxt_ulp_cntxt_entry_release() at the
> end of the function.
>
> However, the second early return statement in the function could
> potentially bypass the lock release. To fix this issue, add
> bnxt_ulp_cntxt_entry_release() before the return statement.
>
> Fixes: 1993b267dbcb ("net/bnxt: cleanup ULP parser and mapper)
> CC: stable at dpdk.org
>
> Signed-off-by: Weiguo Li <liweiguo at xencore.cn>
Acked-by: Ajit Khaparde <ajit.khaparde at broadcom.com>
Fixed up the author and signed-off-by mismatch and merged in
brcm-next-net for-next-net branch.
> ---
> drivers/net/bnxt/tf_ulp/ulp_ha_mgr.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/net/bnxt/tf_ulp/ulp_ha_mgr.c b/drivers/net/bnxt/tf_ulp/ulp_ha_mgr.c
> index f3f5bda890..852deef3b4 100644
> --- a/drivers/net/bnxt/tf_ulp/ulp_ha_mgr.c
> +++ b/drivers/net/bnxt/tf_ulp/ulp_ha_mgr.c
> @@ -253,6 +253,7 @@ ulp_ha_mgr_timer_cb(void *arg)
>
> myclient_cnt = bnxt_ulp_cntxt_num_shared_clients_get(ulp_ctx);
> if (myclient_cnt == 0) {
> + bnxt_ulp_cntxt_entry_release();
> BNXT_TF_DBG(ERR,
> "PANIC Client Count is zero kill timer\n.");
> return;
> --
> 2.34.1
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 4218 bytes
Desc: S/MIME Cryptographic Signature
URL: <http://mails.dpdk.org/archives/stable/attachments/20240208/74a84a9a/attachment.bin>
More information about the stable
mailing list