[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CANjDDBjqvGEY4J=khzSJ4Gf3kH9865tH9K-JZUSBbxemO55mEg@mail.gmail.com>
Date: Thu, 8 Apr 2021 20:51:57 +0530
From: Devesh Sharma <devesh.sharma@...adcom.com>
To: Jason Gunthorpe <jgg@...dia.com>
Cc: Leon Romanovsky <leon@...nel.org>,
Doug Ledford <dledford@...hat.com>,
"David S. Miller" <davem@...emloft.net>,
Jakub Kicinski <kuba@...nel.org>,
linux-rdma <linux-rdma@...r.kernel.org>,
Michael Chan <michael.chan@...adcom.com>,
Naresh Kumar PBS <nareshkumar.pbs@...adcom.com>,
Netdev <netdev@...r.kernel.org>,
Selvin Xavier <selvin.xavier@...adcom.com>,
Somnath Kotur <somnath.kotur@...adcom.com>,
Sriharsha Basavapatna <sriharsha.basavapatna@...adcom.com>
Subject: Re: [PATCH rdma-next v2 0/5] Get rid of custom made module dependency
On Thu, Apr 8, 2021 at 5:23 PM Jason Gunthorpe <jgg@...dia.com> wrote:
>
> On Thu, Apr 08, 2021 at 02:44:45PM +0300, Leon Romanovsky wrote:
>
> > > In my internal testing, I am seeing a crash using the 3rd patch. I am
> > > spending a few cycles on debugging it. expect my input in a day or so.
> >
> > Can you please post the kernel crash report here?
> > I don't see how function rename in patch #3 can cause to the crash.
>
> I looked too, I'm also quite surprised that 1,2,3 alone have a
> bug.. Is there some condition where ulp_probe can be null?
>
> Ugh the is_bnxt_re_dev() is horribly gross too
Yeah, it is indeed. I will take this feedback to the internal team.
>
> Jason
--
-Regards
Devesh
Download attachment "smime.p7s" of type "application/pkcs7-signature" (4212 bytes)
Powered by blists - more mailing lists