Re: [PATCH for-rc 0/2] Fix poor reference counting in advice_mr work

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

 



On Fri, Jan 11, 2019 at 07:31:22PM -0700, Jason Gunthorpe wrote:
> From: Jason Gunthorpe <jgg@xxxxxxxxxxxx>
> 
> Work needs to hold a reference on the pointers that it plans to use otherwise
> they can become free'd or unregistered. Expose the existing registration lock
> that netlink uses for this purpose.
> 
> Jason Gunthorpe (2):
>   RDMA/device: Expose ib_device_try_get(()
>   IB/mlx5: Fix how advise_mr() launches async work

I've applied both of these to for-rc, with the change in comment that
Parav gave.

We can revisit if get_device/put_device is the best way to handle this
when the work queue issue is resolved, most probably we can drop the
get/put in the driver once the work queue is flushed prior to
deallocing the device.

Jason



[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