Messages in this thread Patch in this message | | | Date | Fri, 9 Apr 2021 15:19:27 +0530 | From | Manivannan Sadhasivam <> | Subject | Re: [PATCH -next] spi: qup: fix PM reference leak in spi_qup_remove() |
| |
On Fri, Apr 09, 2021 at 09:54:58AM +0000, Wang Li wrote: > pm_runtime_get_sync will increment pm usage counter even it failed. > Forgetting to putting operation will result in reference leak here. > Fix it by replacing it with pm_runtime_resume_and_get to keep usage > counter balanced. > > Reported-by: Hulk Robot <hulkci@huawei.com> > Signed-off-by: Wang Li <wangli74@huawei.com>
Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Thanks, Mani
> --- > drivers/spi/spi-qup.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/spi/spi-qup.c b/drivers/spi/spi-qup.c > index 8dcb2e70735c..d39dec6d1c91 100644 > --- a/drivers/spi/spi-qup.c > +++ b/drivers/spi/spi-qup.c > @@ -1263,7 +1263,7 @@ static int spi_qup_remove(struct platform_device *pdev) > struct spi_qup *controller = spi_master_get_devdata(master); > int ret; > > - ret = pm_runtime_get_sync(&pdev->dev); > + ret = pm_runtime_resume_and_get(&pdev->dev); > if (ret < 0) > return ret; > > -- > 2.17.1 >
| |