From: John Garry <john.garry@huawei.com>
To: Marc Zyngier <maz@kernel.org>, <linux-kernel@vger.kernel.org>,
<netdev@vger.kernel.org>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Marcin Wojtas <mw@semihalf.com>,
Russell King <linux@armlinux.org.uk>,
"David S. Miller" <davem@davemloft.net>,
Jakub Kicinski <kuba@kernel.org>,
Thomas Gleixner <tglx@linutronix.de>, <kernel-team@android.com>
Subject: Re: [PATCH 1/2] genirq: Extract irq_set_affinity_masks() from devm_platform_get_irqs_affinity()
Date: Thu, 17 Feb 2022 17:07:46 +0000 [thread overview]
Message-ID: <bdfe935b-6ee0-b588-e1e8-776d85f91813@huawei.com> (raw)
In-Reply-To: <20220216090845.1278114-2-maz@kernel.org>
On 16/02/2022 09:08, Marc Zyngier wrote:
Hi Marc,
> In order to better support drivers that deal with interrupts in a more
> "hands-on" way, extract the core of devm_platform_get_irqs_affinity()
> and expose it as irq_set_affinity_masks().
>
> This helper allows a driver to provide a set of wired interrupts that
> are to be configured as managed interrupts. As with the original helper,
> this is exported as EXPORT_SYMBOL_GPL.
I know you mentioned it in 2/2, but it would be interesting to see how
network controller drivers can handle the problem of missing in-flight
IO completions for managed irq shutdown. For storage controllers this is
all now safely handled in the block layer.
>
> Signed-off-by: Marc Zyngier <maz@kernel.org>
Just a small comment, below.
Tested-by: John Garry <john.garry@huawei.com> #D05
Thanks,
John
> ---
> drivers/base/platform.c | 20 +++-----------------
> include/linux/interrupt.h | 8 ++++++++
> kernel/irq/affinity.c | 27 +++++++++++++++++++++++++++
> 3 files changed, 38 insertions(+), 17 deletions(-)
>
> diff --git a/drivers/base/platform.c b/drivers/base/platform.c
> index 6cb04ac48bf0..b363cf6ce5be 100644
> --- a/drivers/base/platform.c
> +++ b/drivers/base/platform.c
> @@ -335,7 +335,6 @@ int devm_platform_get_irqs_affinity(struct platform_device *dev,
> int **irqs)
> {
> struct irq_affinity_devres *ptr;
> - struct irq_affinity_desc *desc;
> size_t size;
> int i, ret, nvec;
>
> @@ -376,31 +375,18 @@ int devm_platform_get_irqs_affinity(struct platform_device *dev,
> ptr->irq[i] = irq;
> }
>
> - desc = irq_create_affinity_masks(nvec, affd);
> - if (!desc) {
> - ret = -ENOMEM;
> + ret = irq_set_affinity_masks(affd, ptr->irq, nvec);
> + if (ret) {
> + dev_err(&dev->dev, "failed to update affinity descriptors (%d)\n", ret);
> goto err_free_devres;
> }
>
> - for (i = 0; i < nvec; i++) {
> - ret = irq_update_affinity_desc(ptr->irq[i], &desc[i]);
> - if (ret) {
> - dev_err(&dev->dev, "failed to update irq%d affinity descriptor (%d)\n",
> - ptr->irq[i], ret);
> - goto err_free_desc;
> - }
> - }
> -
> devres_add(&dev->dev, ptr);
>
> - kfree(desc);
> -
> *irqs = ptr->irq;
>
> return nvec;
>
> -err_free_desc:
> - kfree(desc);
> err_free_devres:
> devres_free(ptr);
> return ret;
> diff --git a/include/linux/interrupt.h b/include/linux/interrupt.h
> index 9367f1cb2e3c..6bfce96206f8 100644
> --- a/include/linux/interrupt.h
> +++ b/include/linux/interrupt.h
> @@ -381,6 +381,8 @@ irq_create_affinity_masks(unsigned int nvec, struct irq_affinity *affd);
> unsigned int irq_calc_affinity_vectors(unsigned int minvec, unsigned int maxvec,
> const struct irq_affinity *affd);
>
> +int irq_set_affinity_masks(struct irq_affinity *affd, int *irqs, int nvec);
> +
> #else /* CONFIG_SMP */
>
> static inline int irq_set_affinity(unsigned int irq, const struct cpumask *m)
> @@ -443,6 +445,12 @@ irq_calc_affinity_vectors(unsigned int minvec, unsigned int maxvec,
> return maxvec;
> }
>
> +static inline int
> +irq_set_affinity_masks(struct irq_affinity *affd, int *irqs, int nvec)
> +{
> + return -EINVAL;
> +}
> +
> #endif /* CONFIG_SMP */
>
> /*
> diff --git a/kernel/irq/affinity.c b/kernel/irq/affinity.c
> index f7ff8919dc9b..c0f868cd5b87 100644
> --- a/kernel/irq/affinity.c
> +++ b/kernel/irq/affinity.c
> @@ -512,3 +512,30 @@ unsigned int irq_calc_affinity_vectors(unsigned int minvec, unsigned int maxvec,
>
> return resv + min(set_vecs, maxvec - resv);
> }
> +
> +/*
> + * irq_set_affinity_masks - Set the affinity masks of a number of interrupts
> + * for multiqueue spreading
> + * @affd: Description of the affinity requirements
> + * @irqs: An array of interrupt numbers
> + * @nvec: The total number of interrupts
> + */
> +int irq_set_affinity_masks(struct irq_affinity *affd, int *irqs, int nvec)
> +{
> + struct irq_affinity_desc *desc;
> + int i, err = 0;
nit: it might be worth doing something similar to how
pci_alloc_irq_vectors_affinity() handles sets with no pre- and
post-vectors with msi_default_affd
> +
> + desc = irq_create_affinity_masks(nvec, affd);
> + if (!desc)
> + return -ENOMEM;
> +
> + for (i = 0; i < nvec; i++) {
> + err = irq_update_affinity_desc(irqs[i], desc + i);
> + if (err)
> + break;
> + }
> +
> + kfree(desc);
> + return err;
> +}
> +EXPORT_SYMBOL_GPL(irq_set_affinity_masks);
next prev parent reply other threads:[~2022-02-17 17:07 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-02-16 9:08 [PATCH 0/2] net: mvpp2: Survive CPU hotplug events Marc Zyngier
2022-02-16 9:08 ` [PATCH 1/2] genirq: Extract irq_set_affinity_masks() from devm_platform_get_irqs_affinity() Marc Zyngier
2022-02-16 10:56 ` Greg Kroah-Hartman
2022-02-17 17:07 ` John Garry [this message]
2022-02-17 17:17 ` Marc Zyngier
2022-02-18 8:41 ` John Garry
2022-03-15 14:25 ` Thomas Gleixner
2022-02-16 9:08 ` [PATCH 2/2] net: mvpp2: Convert to managed interrupts to fix CPU HP issues Marc Zyngier
2022-02-16 11:38 ` Marc Zyngier
2022-02-16 13:19 ` [PATCH 0/2] net: mvpp2: Survive CPU hotplug events Marcin Wojtas
2022-02-16 13:29 ` Marc Zyngier
2022-02-16 13:32 ` Marcin Wojtas
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=bdfe935b-6ee0-b588-e1e8-776d85f91813@huawei.com \
--to=john.garry@huawei.com \
--cc=davem@davemloft.net \
--cc=gregkh@linuxfoundation.org \
--cc=kernel-team@android.com \
--cc=kuba@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@armlinux.org.uk \
--cc=maz@kernel.org \
--cc=mw@semihalf.com \
--cc=netdev@vger.kernel.org \
--cc=tglx@linutronix.de \
/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;
as well as URLs for NNTP newsgroup(s).