kernel_samsung_a53x/drivers/base
Andy Shevchenko 732c3b1ad7 regmap: irq: Set lockdep class for hierarchical IRQ domains
[ Upstream commit 953e549471cabc9d4980f1da2e9fa79f4c23da06 ]

Lockdep gives a false positive splat as it can't distinguish the lock
which is taken by different IRQ descriptors from different IRQ chips
that are organized in a way of a hierarchy:

   ======================================================
   WARNING: possible circular locking dependency detected
   6.12.0-rc5-next-20241101-00148-g9fabf8160b53 #562 Tainted: G        W
   ------------------------------------------------------
   modprobe/141 is trying to acquire lock:
   ffff899446947868 (intel_soc_pmic_bxtwc:502:(&bxtwc_regmap_config)->lock){+.+.}-{4:4}, at: regmap_update_bits_base+0x33/0x90

   but task is already holding lock:
   ffff899446947c68 (&d->lock){+.+.}-{4:4}, at: __setup_irq+0x682/0x790

   which lock already depends on the new lock.

   -> #3 (&d->lock){+.+.}-{4:4}:
   -> #2 (&desc->request_mutex){+.+.}-{4:4}:
   -> #1 (ipclock){+.+.}-{4:4}:
   -> #0 (intel_soc_pmic_bxtwc:502:(&bxtwc_regmap_config)->lock){+.+.}-{4:4}:

   Chain exists of:
     intel_soc_pmic_bxtwc:502:(&bxtwc_regmap_config)->lock --> &desc->request_mutex --> &d->lock

    Possible unsafe locking scenario:

          CPU0                    CPU1
          ----                    ----
     lock(&d->lock);
                                  lock(&desc->request_mutex);
                                  lock(&d->lock);
     lock(intel_soc_pmic_bxtwc:502:(&bxtwc_regmap_config)->lock);

    *** DEADLOCK ***

   3 locks held by modprobe/141:
    #0: ffff8994419368f8 (&dev->mutex){....}-{4:4}, at: __driver_attach+0xf6/0x250
    #1: ffff89944690b250 (&desc->request_mutex){+.+.}-{4:4}, at: __setup_irq+0x1a2/0x790
    #2: ffff899446947c68 (&d->lock){+.+.}-{4:4}, at: __setup_irq+0x682/0x790

Set a lockdep class when we map the IRQ so that it doesn't warn about
a lockdep bug that doesn't exist.

Fixes: 4af8be67fd99 ("regmap: Convert regmap_irq to use irq_domain")
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://patch.msgid.link/20241101165553.4055617-1-andriy.shevchenko@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
2024-12-17 13:24:03 +01:00
..
firmware_loader Revert "firmware_loader: Block path traversal" 2024-11-24 00:23:16 +01:00
power PM: sleep: wakeirq: fix wake irq warning in system suspend 2024-11-19 09:22:17 +01:00
regmap regmap: irq: Set lockdep class for hierarchical IRQ domains 2024-12-17 13:24:03 +01:00
test Import A536BXXU9EXDC 2024-06-15 16:02:09 -03:00
arch_topology.c arch_topology: Rename freq_scale as arch_freq_scale 2024-11-17 17:45:22 +01:00
attribute_container.c Import A536BXXU9EXDC 2024-06-15 16:02:09 -03:00
base.h Import A536BXXU9EXDC 2024-06-15 16:02:09 -03:00
bus.c Revert "driver core: bus: Return -EIO instead of 0 when show/store invalid bus attribute" 2024-11-24 00:22:58 +01:00
cacheinfo.c Import A536BXXU9EXDC 2024-06-15 16:02:09 -03:00
class.c Import A536BXXU9EXDC 2024-06-15 16:02:09 -03:00
component.c Import A536BXXU9EXDC 2024-06-15 16:02:09 -03:00
container.c Import A536BXXU9EXDC 2024-06-15 16:02:09 -03:00
core.c Revert "driver core: Fix uevent_show() vs driver detach race" 2024-11-23 23:22:05 +01:00
cpu.c x86/rfds: Mitigate Register File Data Sampling (RFDS) 2024-11-19 09:22:40 +01:00
dd.c driver core: Release all resources during unbind before updating device links 2024-11-18 12:11:17 +01:00
devcoredump.c devcoredump: Send uevent once devcd is ready 2024-11-18 12:11:48 +01:00
devres.c Revert "devres: Initialize an uninitialized struct member" 2024-11-24 00:23:40 +01:00
devtmpfs.c Import A536BXXU9EXDC 2024-06-15 16:02:09 -03:00
driver.c driver: platform: Add helper for safer setting of driver_override 2024-11-18 10:58:45 +01:00
firmware.c Import A536BXXU9EXDC 2024-06-15 16:02:09 -03:00
hypervisor.c Import A536BXXU9EXDC 2024-06-15 16:02:09 -03:00
init.c Import A536BXXU9EXDC 2024-06-15 16:02:09 -03:00
isa.c Import A536BXXU9EXDC 2024-06-15 16:02:09 -03:00
Kconfig Import A536BXXU9EXDC 2024-06-15 16:02:09 -03:00
Makefile Import A536BXXU9EXDC 2024-06-15 16:02:09 -03:00
map.c Import A536BXXU9EXDC 2024-06-15 16:02:09 -03:00
memory.c Import A536BXXU9EXDC 2024-06-15 16:02:09 -03:00
module.c Revert "driver core: Fix uevent_show() vs driver detach race" 2024-11-23 23:22:05 +01:00
node.c Import A536BXXU9EXDC 2024-06-15 16:02:09 -03:00
pinctrl.c Import A536BXXU9EXDC 2024-06-15 16:02:09 -03:00
platform-mock.c Import A536BXXU9EXDC 2024-06-15 16:02:09 -03:00
platform-msi.c Import A536BXXU9EXDC 2024-06-15 16:02:09 -03:00
platform.c driver: platform: Add helper for safer setting of driver_override 2024-11-18 10:58:45 +01:00
property.c Revert "device property: Add const qualifier to device_get_match_data() parameter" 2024-11-24 00:23:27 +01:00
soc.c Import A536BXXU9EXDC 2024-06-15 16:02:09 -03:00
swnode.c software node: Let args be NULL in software_node_get_reference_args 2024-11-18 12:12:49 +01:00
syscore.c Import A536BXXU9EXDC 2024-06-15 16:02:09 -03:00
topology.c Import A536BXXU9EXDC 2024-06-15 16:02:09 -03:00
transport_class.c Import A536BXXU9EXDC 2024-06-15 16:02:09 -03:00