All of lore.kernel.org
 help / color / mirror / Atom feed
From: linz  <powertree@163.com>
To: "Florian Bezdeka" <florian.bezdeka@siemens.com>
Cc: "Philippe Gerum" <rpm@xenomai.org>,
	"jan.kiszka@siemens.com" <jan.kiszka@siemens.com>,
	xenomai@lists.linux.dev
Subject: [PATCH v2] irqchip: gic-v3-its: irq_pipeline: Fix OOB context in-band lock warning for ITS lock
Date: Tue, 14 Jul 2026 11:43:09 +0800 (CST)	[thread overview]
Message-ID: <55bf6b7f.363a.19f5eb8aece.Coremail.powertree@163.com> (raw)
In-Reply-To: <6ca12451-a015-4b54-9e22-0256e97d834e@siemens.com>

Thank you for your feedback

Under 6.6.63-dovetail2 environment, raw_spinlock_t for the ITS node lock triggers out‑of‑band context warning in irq‑pipeline code.
As suggested by Florian, hybrid_spinlock_t should be limited to struct irq_desc and irqchip internals. ITS node lock is more suitable to use hard_spinlock_t.

After replacing raw_spinlock_t with hard_spinlock_t, the call‑trace warning disappears and system runs stably.

In addition, the gic‑v3 irqchip already has IRQCHIP_PIPELINE_SAFE flag, while its_irq_chip lacks this flag.
Enabling IRQCHIP_PIPELINE_SAFE for its_irq_chip has no effect to resolve the warning.

Signed-off-by: zhanglin <powertree@163.com>
---
 drivers/irqchip/irq-gic-v3-its.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/irqchip/irq-gic-v3-its.c b/drivers/irqchip/irq-gic-v3-its.c
index 0e57735eac..3f2b02b8f4 100644
--- a/drivers/irqchip/irq-gic-v3-its.c
+++ b/drivers/irqchip/irq-gic-v3-its.c
@@ -94,7 +94,7 @@ struct its_device;
  * list.
  */
 struct its_node {
-	raw_spinlock_t		lock;
+	hard_spinlock_t		lock;
 	struct mutex		dev_alloc_lock;
 	struct list_head	entry;
 	void __iomem		*base;
-- 
2.34.1

      parent reply	other threads:[~2026-07-14  3:43 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-25  9:32 [PATCH] irqchip: gic-v3-its: irq_pipeline: Fix OOB context in-band lock warning for ITS lock linz
2026-07-10 10:12 ` Jan Kiszka
2026-07-10 10:59   ` Florian Bezdeka
2026-07-13 10:20     ` linz
2026-07-13 10:29       ` Jan Kiszka
2026-07-13 10:42         ` [PATCH v2] irqchip: gic-v3-its: irq_pipeline: fix OOB context in-band lock warning for ITS lock using hard spinlocks instead of hybrid spinlocks linz
2026-07-13 10:50           ` Jan Kiszka
2026-07-14  3:43         ` linz [this message]

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=55bf6b7f.363a.19f5eb8aece.Coremail.powertree@163.com \
    --to=powertree@163.com \
    --cc=florian.bezdeka@siemens.com \
    --cc=jan.kiszka@siemens.com \
    --cc=rpm@xenomai.org \
    --cc=xenomai@lists.linux.dev \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.