From: Mohith Kumar Thummaluru <mohith.k.kumar.thummaluru@oracle.com>
To: Jacob Keller <jacob.e.keller@intel.com>,
"saeedm@nvidia.com" <saeedm@nvidia.com>,
"leon@kernel.org" <leon@kernel.org>,
"tariqt@nvidia.com" <tariqt@nvidia.com>,
"netdev@vger.kernel.org" <netdev@vger.kernel.org>
Cc: "andrew+netdev@lunn.ch" <andrew+netdev@lunn.ch>,
"davem@davemloft.net" <davem@davemloft.net>,
"edumazet@google.com" <edumazet@google.com>,
"kuba@kernel.org" <kuba@kernel.org>,
"pabeni@redhat.com" <pabeni@redhat.com>,
"shayd@nvidia.com" <shayd@nvidia.com>,
"elic@nvidia.com" <elic@nvidia.com>,
"linux-rdma@vger.kernel.org" <linux-rdma@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
Anand Khoje <anand.a.khoje@oracle.com>,
Manjunath Patil <manjunath.b.patil@oracle.com>,
Rama Nichanamatlu <rama.nichanamatlu@oracle.com>,
Rajesh Sivaramasubramaniom
<rajesh.sivaramasubramaniom@oracle.com>,
Rohit Sajan Kumar <rohit.sajan.kumar@oracle.com>,
Qing Huang <qing.huang@oracle.com>,
Mark Bloch <mbloch@nvidia.com>
Subject: Re: [RESEND PATCH net-next 1/1] net/mlx5: Clean up only new IRQ glue on request_irq() failure
Date: Wed, 2 Jul 2025 10:37:59 +0530 [thread overview]
Message-ID: <d1b697de-ea53-4634-9821-f403090d3e09@oracle.com> (raw)
In-Reply-To: <06e99d8d-a6de-4687-b109-0d1557ac2779@intel.com>
On 02/07/25 2:21 am, Jacob Keller wrote:
>
> On 6/25/2025 10:32 PM, Mohith Kumar Thummaluru wrote:
>> The mlx5_irq_alloc() function can inadvertently free the entire rmap
>> and end up in a crash[1] when the other threads tries to access this,
>> when request_irq() fails due to exhausted IRQ vectors. This commit
>> modifies the cleanup to remove only the specific IRQ mapping that was
>> just added.
>>
>> This prevents removal of other valid mappings and ensures precise
>> cleanup of the failed IRQ allocation's associated glue object.
>>
>> Note: This error is observed when both fwctl and rds configs are enabled.
>>
> Reviewed-by: Jacob Keller <jacob.e.keller@intel.com>
Thanks for the review, Jacob!
Based on Mark Bloch's input, I’ve sent out another patch targeting the
net tree. You can find it here:
https://lore.kernel.org/netdev/1eda4785-6e3e-4660-ac04-62e474133d71@oracle.com/
Regards,
Mohith Kumar Thummaluru
next prev parent reply other threads:[~2025-07-02 5:08 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <7cb171c4-3c36-42ea-bd6f-52dfe6bc5dab@oracle.com>
2025-07-01 20:51 ` [RESEND PATCH net-next 1/1] net/mlx5: Clean up only new IRQ glue on request_irq() failure Jacob Keller
2025-07-02 5:07 ` Mohith Kumar Thummaluru [this message]
2025-07-02 17:56 ` Jacob Keller
2025-06-26 6:04 Mohith Kumar Thummaluru
2025-06-26 11:58 ` Mark Bloch
2025-06-27 6:43 ` Mohith Kumar Thummaluru
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=d1b697de-ea53-4634-9821-f403090d3e09@oracle.com \
--to=mohith.k.kumar.thummaluru@oracle.com \
--cc=anand.a.khoje@oracle.com \
--cc=andrew+netdev@lunn.ch \
--cc=davem@davemloft.net \
--cc=edumazet@google.com \
--cc=elic@nvidia.com \
--cc=jacob.e.keller@intel.com \
--cc=kuba@kernel.org \
--cc=leon@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-rdma@vger.kernel.org \
--cc=manjunath.b.patil@oracle.com \
--cc=mbloch@nvidia.com \
--cc=netdev@vger.kernel.org \
--cc=pabeni@redhat.com \
--cc=qing.huang@oracle.com \
--cc=rajesh.sivaramasubramaniom@oracle.com \
--cc=rama.nichanamatlu@oracle.com \
--cc=rohit.sajan.kumar@oracle.com \
--cc=saeedm@nvidia.com \
--cc=shayd@nvidia.com \
--cc=tariqt@nvidia.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox