From: Julia Lawall <julia.lawall@inria.fr>
To: Maxime Ripard <maxime@cerno.tech>
Cc: Linux Memory Management List <linux-mm@kvack.org>,
kbuild-all@lists.01.org
Subject: [linux-next:master 331/3266] drivers/gpu/drm/vc4/vc4_hdmi.c:1459:2-8: preceding lock on line 1456 (fwd)
Date: Sat, 27 Aug 2022 16:33:11 +0200 (CEST) [thread overview]
Message-ID: <alpine.DEB.2.22.394.2208271632220.6645@hadrien> (raw)
[-- Attachment #1: Type: text/plain, Size: 7623 bytes --]
Please check whether anunlock is needed before line 1459.
julia
---------- Forwarded message ----------
Date: Fri, 26 Aug 2022 07:43:10 +0800
From: kernel test robot <lkp@intel.com>
To: kbuild@lists.01.org
Cc: lkp@intel.com, Julia Lawall <julia.lawall@lip6.fr>
Subject: [linux-next:master 331/3266] drivers/gpu/drm/vc4/vc4_hdmi.c:1459:2-8:
preceding lock on line 1456
BCC: lkp@intel.com
CC: kbuild-all@lists.01.org
CC: Linux Memory Management List <linux-mm@kvack.org>
TO: Maxime Ripard <maxime@cerno.tech>
tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head: b5d939c951865f6fc229094e84b77c9a9e0ed0c7
commit: cd00ed5187bff03d4c2ba143a5993383dd6ed66c [331/3266] drm/vc4: hdmi: Protect device resources after removal
:::::: branch date: 18 hours ago
:::::: commit date: 6 weeks ago
config: arm64-randconfig-c44-20220824 (https://download.01.org/0day-ci/archive/20220826/202208260727.0o3WQ2wp-lkp@intel.com/config)
compiler: aarch64-linux-gcc (GCC) 12.1.0
If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Julia Lawall <julia.lawall@lip6.fr>
cocci warnings: (new ones prefixed by >>)
>> drivers/gpu/drm/vc4/vc4_hdmi.c:1459:2-8: preceding lock on line 1456
drivers/gpu/drm/vc4/vc4_hdmi.c:1429:2-8: preceding lock on line 1426
vim +1459 drivers/gpu/drm/vc4/vc4_hdmi.c
c8b75bca92cbf0 Eric Anholt 2015-03-02 1442
8d9147466776c4 Maxime Ripard 2020-12-15 1443 static void vc4_hdmi_encoder_post_crtc_enable(struct drm_encoder *encoder,
8d9147466776c4 Maxime Ripard 2020-12-15 1444 struct drm_atomic_state *state)
09c438139b8f5b Maxime Ripard 2020-09-03 1445 {
09c438139b8f5b Maxime Ripard 2020-09-03 1446 struct vc4_hdmi *vc4_hdmi = encoder_to_vc4_hdmi(encoder);
cd00ed5187bff0 Maxime Ripard 2022-07-11 1447 struct drm_device *drm = vc4_hdmi->connector.dev;
633be8c3c0c5e0 Maxime Ripard 2021-10-25 1448 struct drm_display_mode *mode = &vc4_hdmi->saved_adjusted_mode;
c3c2f38ce532c6 José Expósito 2022-04-20 1449 struct drm_display_info *display = &vc4_hdmi->connector.display_info;
8b3f90e98e1764 Maxime Ripard 2020-09-03 1450 bool hsync_pos = mode->flags & DRM_MODE_FLAG_PHSYNC;
8b3f90e98e1764 Maxime Ripard 2020-09-03 1451 bool vsync_pos = mode->flags & DRM_MODE_FLAG_PVSYNC;
81fb55e500a82a Maxime Ripard 2021-10-25 1452 unsigned long flags;
09c438139b8f5b Maxime Ripard 2020-09-03 1453 int ret;
cd00ed5187bff0 Maxime Ripard 2022-07-11 1454 int idx;
c8b75bca92cbf0 Eric Anholt 2015-03-02 1455
82cb88af12d29e Maxime Ripard 2021-10-25 @1456 mutex_lock(&vc4_hdmi->mutex);
82cb88af12d29e Maxime Ripard 2021-10-25 1457
cd00ed5187bff0 Maxime Ripard 2022-07-11 1458 if (!drm_dev_enter(drm, &idx))
cd00ed5187bff0 Maxime Ripard 2022-07-11 @1459 return;
cd00ed5187bff0 Maxime Ripard 2022-07-11 1460
81fb55e500a82a Maxime Ripard 2021-10-25 1461 spin_lock_irqsave(&vc4_hdmi->hw_lock, flags);
81fb55e500a82a Maxime Ripard 2021-10-25 1462
311e305fdb4e82 Maxime Ripard 2020-09-03 1463 HDMI_WRITE(HDMI_VID_CTL,
c8b75bca92cbf0 Eric Anholt 2015-03-02 1464 VC4_HD_VID_CTL_ENABLE |
0b066a6809d0f8 Tim Gover 2021-06-28 1465 VC4_HD_VID_CTL_CLRRGB |
c8b75bca92cbf0 Eric Anholt 2015-03-02 1466 VC4_HD_VID_CTL_UNDERFLOW_ENABLE |
8b3f90e98e1764 Maxime Ripard 2020-09-03 1467 VC4_HD_VID_CTL_FRAME_COUNTER_RESET |
8b3f90e98e1764 Maxime Ripard 2020-09-03 1468 (vsync_pos ? 0 : VC4_HD_VID_CTL_VSYNC_LOW) |
8b3f90e98e1764 Maxime Ripard 2020-09-03 1469 (hsync_pos ? 0 : VC4_HD_VID_CTL_HSYNC_LOW));
c8b75bca92cbf0 Eric Anholt 2015-03-02 1470
81d830137bdb46 Maxime Ripard 2020-09-03 1471 HDMI_WRITE(HDMI_VID_CTL,
81d830137bdb46 Maxime Ripard 2020-09-03 1472 HDMI_READ(HDMI_VID_CTL) & ~VC4_HD_VID_CTL_BLANKPIX);
c8b75bca92cbf0 Eric Anholt 2015-03-02 1473
c3c2f38ce532c6 José Expósito 2022-04-20 1474 if (display->is_hdmi) {
311e305fdb4e82 Maxime Ripard 2020-09-03 1475 HDMI_WRITE(HDMI_SCHEDULER_CONTROL,
311e305fdb4e82 Maxime Ripard 2020-09-03 1476 HDMI_READ(HDMI_SCHEDULER_CONTROL) |
c8b75bca92cbf0 Eric Anholt 2015-03-02 1477 VC4_HDMI_SCHEDULER_CONTROL_MODE_HDMI);
c8b75bca92cbf0 Eric Anholt 2015-03-02 1478
81fb55e500a82a Maxime Ripard 2021-10-25 1479 spin_unlock_irqrestore(&vc4_hdmi->hw_lock, flags);
81fb55e500a82a Maxime Ripard 2021-10-25 1480
311e305fdb4e82 Maxime Ripard 2020-09-03 1481 ret = wait_for(HDMI_READ(HDMI_SCHEDULER_CONTROL) &
2b29bf16611a1a Eric Anholt 2016-09-28 1482 VC4_HDMI_SCHEDULER_CONTROL_HDMI_ACTIVE, 1000);
c8b75bca92cbf0 Eric Anholt 2015-03-02 1483 WARN_ONCE(ret, "Timeout waiting for "
c8b75bca92cbf0 Eric Anholt 2015-03-02 1484 "VC4_HDMI_SCHEDULER_CONTROL_HDMI_ACTIVE\n");
c8b75bca92cbf0 Eric Anholt 2015-03-02 1485 } else {
311e305fdb4e82 Maxime Ripard 2020-09-03 1486 HDMI_WRITE(HDMI_RAM_PACKET_CONFIG,
311e305fdb4e82 Maxime Ripard 2020-09-03 1487 HDMI_READ(HDMI_RAM_PACKET_CONFIG) &
c8b75bca92cbf0 Eric Anholt 2015-03-02 1488 ~(VC4_HDMI_RAM_PACKET_ENABLE));
311e305fdb4e82 Maxime Ripard 2020-09-03 1489 HDMI_WRITE(HDMI_SCHEDULER_CONTROL,
311e305fdb4e82 Maxime Ripard 2020-09-03 1490 HDMI_READ(HDMI_SCHEDULER_CONTROL) &
c8b75bca92cbf0 Eric Anholt 2015-03-02 1491 ~VC4_HDMI_SCHEDULER_CONTROL_MODE_HDMI);
c8b75bca92cbf0 Eric Anholt 2015-03-02 1492
81fb55e500a82a Maxime Ripard 2021-10-25 1493 spin_unlock_irqrestore(&vc4_hdmi->hw_lock, flags);
81fb55e500a82a Maxime Ripard 2021-10-25 1494
311e305fdb4e82 Maxime Ripard 2020-09-03 1495 ret = wait_for(!(HDMI_READ(HDMI_SCHEDULER_CONTROL) &
2b29bf16611a1a Eric Anholt 2016-09-28 1496 VC4_HDMI_SCHEDULER_CONTROL_HDMI_ACTIVE), 1000);
c8b75bca92cbf0 Eric Anholt 2015-03-02 1497 WARN_ONCE(ret, "Timeout waiting for "
c8b75bca92cbf0 Eric Anholt 2015-03-02 1498 "!VC4_HDMI_SCHEDULER_CONTROL_HDMI_ACTIVE\n");
c8b75bca92cbf0 Eric Anholt 2015-03-02 1499 }
c8b75bca92cbf0 Eric Anholt 2015-03-02 1500
c3c2f38ce532c6 José Expósito 2022-04-20 1501 if (display->is_hdmi) {
81fb55e500a82a Maxime Ripard 2021-10-25 1502 spin_lock_irqsave(&vc4_hdmi->hw_lock, flags);
81fb55e500a82a Maxime Ripard 2021-10-25 1503
311e305fdb4e82 Maxime Ripard 2020-09-03 1504 WARN_ON(!(HDMI_READ(HDMI_SCHEDULER_CONTROL) &
c8b75bca92cbf0 Eric Anholt 2015-03-02 1505 VC4_HDMI_SCHEDULER_CONTROL_HDMI_ACTIVE));
c8b75bca92cbf0 Eric Anholt 2015-03-02 1506
311e305fdb4e82 Maxime Ripard 2020-09-03 1507 HDMI_WRITE(HDMI_RAM_PACKET_CONFIG,
21317b3fba5428 Eric Anholt 2016-09-29 1508 VC4_HDMI_RAM_PACKET_ENABLE);
21317b3fba5428 Eric Anholt 2016-09-29 1509
81fb55e500a82a Maxime Ripard 2021-10-25 1510 spin_unlock_irqrestore(&vc4_hdmi->hw_lock, flags);
4ae3d6c09d25a9 Dom Cobley 2022-06-13 1511 vc4_hdmi->packet_ram_enabled = true;
81fb55e500a82a Maxime Ripard 2021-10-25 1512
21317b3fba5428 Eric Anholt 2016-09-29 1513 vc4_hdmi_set_infoframes(encoder);
c8b75bca92cbf0 Eric Anholt 2015-03-02 1514 }
c8b75bca92cbf0 Eric Anholt 2015-03-02 1515
691456f622a96b Maxime Ripard 2020-09-03 1516 vc4_hdmi_recenter_fifo(vc4_hdmi);
c85695a2016e2e Maxime Ripard 2021-05-07 1517 vc4_hdmi_enable_scrambling(encoder);
82cb88af12d29e Maxime Ripard 2021-10-25 1518
cd00ed5187bff0 Maxime Ripard 2022-07-11 1519 drm_dev_exit(idx);
82cb88af12d29e Maxime Ripard 2021-10-25 1520 mutex_unlock(&vc4_hdmi->mutex);
c8b75bca92cbf0 Eric Anholt 2015-03-02 1521 }
c8b75bca92cbf0 Eric Anholt 2015-03-02 1522
--
0-DAY CI Kernel Test Service
https://01.org/lkp
next reply other threads:[~2022-08-27 14:33 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-08-27 14:33 Julia Lawall [this message]
2022-08-29 7:42 ` Maxime Ripard
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.2208271632220.6645@hadrien \
--to=julia.lawall@inria.fr \
--cc=kbuild-all@lists.01.org \
--cc=linux-mm@kvack.org \
--cc=maxime@cerno.tech \
/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