* [PATCH -next] media: sunxi: sun8i-rotate: fix PM reference leak in rotate_start_streaming()
@ 2021-04-09 6:46 Xiang Yang
2021-04-11 18:16 ` Jernej Škrabec
0 siblings, 1 reply; 2+ messages in thread
From: Xiang Yang @ 2021-04-09 6:46 UTC (permalink / raw)
To: xiangyang3, Jernej Skrabec, Mauro Carvalho Chehab, Maxime Ripard,
Chen-Yu Tsai, Philipp Zabel
Cc: linux-media, linux-arm-kernel, linux-sunxi, kernel-janitors,
linux-kernel, Hulk Robot
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: Xiang Yang <xiangyang3@huawei.com>
---
.../platform/sunxi/sun8i-rotate/sun8i_rotate.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/platform/sunxi/sun8i-rotate/sun8i_rotate.c b/drivers/media/platform/sunxi/sun8i-rotate/sun8i_rotate.c
index 3f81dd17755c..fbcca59a0517 100644
--- a/drivers/media/platform/sunxi/sun8i-rotate/sun8i_rotate.c
+++ b/drivers/media/platform/sunxi/sun8i-rotate/sun8i_rotate.c
@@ -494,7 +494,7 @@ static int rotate_start_streaming(struct vb2_queue *vq, unsigned int count)
struct device *dev = ctx->dev->dev;
int ret;
- ret = pm_runtime_get_sync(dev);
+ ret = pm_runtime_resume_and_get(dev);
if (ret < 0) {
dev_err(dev, "Failed to enable module\n");
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH -next] media: sunxi: sun8i-rotate: fix PM reference leak in rotate_start_streaming()
2021-04-09 6:46 [PATCH -next] media: sunxi: sun8i-rotate: fix PM reference leak in rotate_start_streaming() Xiang Yang
@ 2021-04-11 18:16 ` Jernej Škrabec
0 siblings, 0 replies; 2+ messages in thread
From: Jernej Škrabec @ 2021-04-11 18:16 UTC (permalink / raw)
To: xiangyang3, Mauro Carvalho Chehab, Maxime Ripard, Chen-Yu Tsai,
Philipp Zabel, Xiang Yang
Cc: linux-media, linux-arm-kernel, linux-sunxi, kernel-janitors,
linux-kernel, Hulk Robot
Dne petek, 09. april 2021 ob 08:46:58 CEST je Xiang Yang napisal(a):
> 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: Xiang Yang <xiangyang3@huawei.com>
Acked-by: Jernej Skrabec <jernej.skrabec@siol.net>
Best regards,
Jernej
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-04-11 18:18 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-09 6:46 [PATCH -next] media: sunxi: sun8i-rotate: fix PM reference leak in rotate_start_streaming() Xiang Yang
2021-04-11 18:16 ` Jernej Škrabec
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).