From: Julia Lawall <julia.lawall@inria.fr>
To: Mauro Carvalho Chehab <mchehab@kernel.org>
Cc: linux-media@vger.kernel.org, linuxarm@huawei.com,
mauro.chehab@huawei.com, linux-media@vger.kernel.org,
"Gustavo A. R. Silva" <gustavoars@kernel.org>,
Allen Pais <allen.lkml@gmail.com>,
Chuhong Yuan <hslester96@gmail.com>,
Ezequiel Garcia <ezequiel@collabora.com>,
Hans Verkuil <hverkuil@xs4all.nl>,
Lubomir Rintel <lkundrak@v3.sk>,
kbuild-all@lists.01.org
Subject: Re: [PATCH 02/78] media: marvel-ccic: fix some issues when getting pm_runtime (fwd)
Date: Sun, 25 Apr 2021 19:38:46 +0200 (CEST) [thread overview]
Message-ID: <alpine.DEB.2.22.394.2104251936530.2930@hadrien> (raw)
[-- Attachment #1: Type: text/plain, Size: 6059 bytes --]
Should there be an unlock before line 1619?
thanks,
julia
---------- Forwarded message ----------
Date: Sat, 24 Apr 2021 19:12:42 +0800
From: kernel test robot <lkp@intel.com>
To: kbuild@lists.01.org
Cc: lkp@intel.com, Julia Lawall <julia.lawall@lip6.fr>
Subject: Re: [PATCH 02/78] media: marvel-ccic: fix some issues when getting
pm_runtime
CC: kbuild-all@lists.01.org
In-Reply-To: <93a8c81827fd7fdd4b50ba5c5f63387519559bf5.1619191723.git.mchehab+huawei@kernel.org>
References: <93a8c81827fd7fdd4b50ba5c5f63387519559bf5.1619191723.git.mchehab+huawei@kernel.org>
TO: Mauro Carvalho Chehab <mchehab@kernel.org>
CC: linux-media@vger.kernel.org
CC: linuxarm@huawei.com
CC: mauro.chehab@huawei.com
CC: Mauro Carvalho Chehab <mchehab@kernel.org>
CC: linux-media@vger.kernel.org
CC: "Gustavo A. R. Silva" <gustavoars@kernel.org>
CC: Allen Pais <allen.lkml@gmail.com>
CC: Chuhong Yuan <hslester96@gmail.com>
CC: Ezequiel Garcia <ezequiel@collabora.com>
CC: Hans Verkuil <hverkuil@xs4all.nl>
CC: Lubomir Rintel <lkundrak@v3.sk>
Hi Mauro,
I love your patch! Perhaps something to improve:
[auto build test WARNING on linuxtv-media/master]
[also build test WARNING on rockchip/for-next tegra/for-next v5.12-rc8 next-20210423]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Mauro-Carvalho-Chehab/media-use-pm_runtime_resume_and_get-instead-of-pm_runtime_get_sync/20210424-145029
base: git://linuxtv.org/media_tree.git master
:::::: branch date: 4 hours ago
:::::: commit date: 4 hours ago
config: i386-randconfig-c021-20210424 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Julia Lawall <julia.lawall@lip6.fr>
cocci warnings: (new ones prefixed by >>)
>> drivers/media/platform/marvell-ccic/mcam-core.c:1619:3-9: preceding lock on line 1609
vim +1619 drivers/media/platform/marvell-ccic/mcam-core.c
abfa3df36c01a3 drivers/media/video/marvell-ccic/mcam-core.c Jonathan Corbet 2011-06-11 1599
abfa3df36c01a3 drivers/media/video/marvell-ccic/mcam-core.c Jonathan Corbet 2011-06-11 1600 /* ---------------------------------------------------------------------- */
abfa3df36c01a3 drivers/media/video/marvell-ccic/mcam-core.c Jonathan Corbet 2011-06-11 1601 /*
d43dae75cc1140 drivers/media/video/marvell-ccic/mcam-core.c Jonathan Corbet 2011-07-08 1602 * Our various file operations.
abfa3df36c01a3 drivers/media/video/marvell-ccic/mcam-core.c Jonathan Corbet 2011-06-11 1603 */
d43dae75cc1140 drivers/media/video/marvell-ccic/mcam-core.c Jonathan Corbet 2011-07-08 1604 static int mcam_v4l_open(struct file *filp)
d43dae75cc1140 drivers/media/video/marvell-ccic/mcam-core.c Jonathan Corbet 2011-07-08 1605 {
d43dae75cc1140 drivers/media/video/marvell-ccic/mcam-core.c Jonathan Corbet 2011-07-08 1606 struct mcam_camera *cam = video_drvdata(filp);
949bd4081ffe95 drivers/media/platform/marvell-ccic/mcam-core.c Hans Verkuil 2015-03-05 1607 int ret;
abfa3df36c01a3 drivers/media/video/marvell-ccic/mcam-core.c Jonathan Corbet 2011-06-11 1608
d43dae75cc1140 drivers/media/video/marvell-ccic/mcam-core.c Jonathan Corbet 2011-07-08 @1609 mutex_lock(&cam->s_mutex);
949bd4081ffe95 drivers/media/platform/marvell-ccic/mcam-core.c Hans Verkuil 2015-03-05 1610 ret = v4l2_fh_open(filp);
d43dae75cc1140 drivers/media/video/marvell-ccic/mcam-core.c Jonathan Corbet 2011-07-08 1611 if (ret)
d43dae75cc1140 drivers/media/video/marvell-ccic/mcam-core.c Jonathan Corbet 2011-07-08 1612 goto out;
949bd4081ffe95 drivers/media/platform/marvell-ccic/mcam-core.c Hans Verkuil 2015-03-05 1613 if (v4l2_fh_is_singular_file(filp)) {
81a409bfd5517d drivers/media/platform/marvell-ccic/mcam-core.c Lubomir Rintel 2019-05-28 1614 ret = sensor_call(cam, core, s_power, 1);
05fed81625bf75 drivers/media/platform/marvell-ccic/mcam-core.c Libin Yang 2013-07-03 1615 if (ret)
05fed81625bf75 drivers/media/platform/marvell-ccic/mcam-core.c Libin Yang 2013-07-03 1616 goto out;
52f5b74e58d396 drivers/media/platform/marvell-ccic/mcam-core.c Mauro Carvalho Chehab 2021-04-24 1617 ret = pm_runtime_resume_and_get(cam->dev);
52f5b74e58d396 drivers/media/platform/marvell-ccic/mcam-core.c Mauro Carvalho Chehab 2021-04-24 1618 if (ret < 0)
52f5b74e58d396 drivers/media/platform/marvell-ccic/mcam-core.c Mauro Carvalho Chehab 2021-04-24 @1619 return ret;
d43dae75cc1140 drivers/media/video/marvell-ccic/mcam-core.c Jonathan Corbet 2011-07-08 1620 __mcam_cam_reset(cam);
d43dae75cc1140 drivers/media/video/marvell-ccic/mcam-core.c Jonathan Corbet 2011-07-08 1621 mcam_set_config_needed(cam, 1);
d43dae75cc1140 drivers/media/video/marvell-ccic/mcam-core.c Jonathan Corbet 2011-07-08 1622 }
d43dae75cc1140 drivers/media/video/marvell-ccic/mcam-core.c Jonathan Corbet 2011-07-08 1623 out:
d43dae75cc1140 drivers/media/video/marvell-ccic/mcam-core.c Jonathan Corbet 2011-07-08 1624 mutex_unlock(&cam->s_mutex);
44fbcb10cf35e3 drivers/media/platform/marvell-ccic/mcam-core.c Hans Verkuil 2015-03-05 1625 if (ret)
44fbcb10cf35e3 drivers/media/platform/marvell-ccic/mcam-core.c Hans Verkuil 2015-03-05 1626 v4l2_fh_release(filp);
d43dae75cc1140 drivers/media/video/marvell-ccic/mcam-core.c Jonathan Corbet 2011-07-08 1627 return ret;
a9b36e850782db drivers/media/video/marvell-ccic/mcam-core.c Jonathan Corbet 2011-06-20 1628 }
abfa3df36c01a3 drivers/media/video/marvell-ccic/mcam-core.c Jonathan Corbet 2011-06-11 1629
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: Type: application/gzip, Size: 34188 bytes --]
reply other threads:[~2021-04-25 17:38 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=alpine.DEB.2.22.394.2104251936530.2930@hadrien \
--to=julia.lawall@inria.fr \
--cc=allen.lkml@gmail.com \
--cc=ezequiel@collabora.com \
--cc=gustavoars@kernel.org \
--cc=hslester96@gmail.com \
--cc=hverkuil@xs4all.nl \
--cc=kbuild-all@lists.01.org \
--cc=linux-media@vger.kernel.org \
--cc=linuxarm@huawei.com \
--cc=lkundrak@v3.sk \
--cc=mauro.chehab@huawei.com \
--cc=mchehab@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox