Re: [PATCH] scsi: vmw_pscsi: Rearrange code to avoid multiple calls to free_irq during unload

From: Dan Carpenter
Date: Tue Nov 27 2018 - 15:12:43 EST


On Tue, Nov 27, 2018 at 02:28:53PM -0500, Cathy Avery wrote:
> Currently pvscsi_remove calls free_irq more than once as
> pvscsi_release_resources and __pvscsi_shutdown both call
> pvscsi_shutdown_intr. This results in a 'Trying to free
> already-free IRQ' warning and stack trace. To solve the problem
> pvscsi_shutdown_intr has been moved out of pvscsi_release_resources.
>
> Signed-off-by: Cathy Avery <cavery@xxxxxxxxxx>
> Reviewed-by: Ewan D. Milne <emilne@xxxxxxxxxx>

That bug has been there since the driver was introduced in 2009. It's
so amazing that no one has fixed it until now...

Reviewed-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>

regards,
dan carpenter