All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marc Zyngier <maz@kernel.org>
To: "Marek Behún" <kabel@kernel.org>
Cc: linux-kernel@vger.kernel.org,
	Thomas Gleixner <tglx@linutronix.de>,
	pali@kernel.org
Subject: Re: irqdomain API: how to set affinity of parent irq of chained irqs?
Date: Mon, 02 May 2022 10:31:11 +0100	[thread overview]
Message-ID: <87mtg0i8m8.wl-maz@kernel.org> (raw)
In-Reply-To: <20220502102137.764606ee@thinkpad>

On Mon, 02 May 2022 09:21:37 +0100,
Marek Behún <kabel@kernel.org> wrote:
> 
> Dear Marc, Thomas,
> 
> we have encountered the following problem that can hopefully be put
> some light onto: What is the intended way to set affinity (and possibly
> other irq attributes) of parent IRQ of chained IRQs, when using the
> irqdomain API?

Simples: you can't. What sense does it make to change the affinity of
the parent interrupt, given that its fate is tied to *all* of the
other interrupts that are muxed to it?

Moving the parent interrupt breaks userspace's view of how interrupt
affinity is managed (change the affinity of one interrupt, see all the
others move the same way). Which is why we don't expose this interrupt
to userspace, as this can only lead to bad things.

Note that this has nothing to do with the irqdomain API, but
everything to do with the userspace ABI.

> 
> We are working on a driver that
> - registers an irqchip and adds an irqdomain
> - calls irq_set_chained_handler_and_data(parent_irq, handler)
>   where handler triggers handling of child IRQs
> - but since parent_irq isn't requested for with request_thread_irq(),
>   it does not show up in proc/sysfs, only in debugfs
> - the HW does not support setting affinity for the chained IRQs, only 
>   the parent (which comes from a GIC chip)
> 
> The problem is that he parent IRQ, as mentioned in the third point, does
> not show up in proc/sysfs.
> 
> Is there some precedent for this?

There were precedents of irqchips doing terrible things, such as
implementing a set_affinity() callback in the chained irqchip.
Thankfully, they have been either fixed or eradicated.

	M.

-- 
Without deviation from the norm, progress is not possible.

  reply	other threads:[~2022-05-02  9:30 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-05-02  8:21 Marek Behún
2022-05-02  9:31 ` Marc Zyngier [this message]
2022-05-02 15:45   ` Marek Behún
2022-05-03  9:32     ` Marc Zyngier

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=87mtg0i8m8.wl-maz@kernel.org \
    --to=maz@kernel.org \
    --cc=kabel@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pali@kernel.org \
    --cc=tglx@linutronix.de \
    --subject='Re: irqdomain API: how to set affinity of parent irq of chained irqs?' \
    /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

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.