Re: [PATCH for-rc 2/4] IB/hfi1: Call xa_destroy before unloading the module

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 3/29/2021 10:11 AM, Jason Gunthorpe wrote:
On Mon, Mar 29, 2021 at 09:48:18AM -0400, dennis.dalessandro@xxxxxxxxxxxxxxxxxxxx wrote:
From: Kaike Wan <kaike.wan@xxxxxxxxx>

Call xa_destroy for hfi1_dev_table before unloading the module to avoid
a potential memory leak.

Do you hit the WARN_ON or not?

Is this all just mindless?

If the xarray is supposed to be empty because everything was erased
then you don't need it, the WARN_ON is correct. An empty xarray needs
no further destruction.

Looking at our internal bug that corresponds to this change, I don't see a WARN_ON that had been hit. I think we should just go ahead and drop these two patches for now and if we do hit the WARN_ON we will revisit.

-Denny



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux