| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Fri, 6 Aug 2021 14:24:49 +0530
From: Sreekanth Reddy <sreekanth.reddy@...adcom.com>
To: Nitesh Narayan Lal <nitesh@...hat.com>
Cc: linux-kernel@...r.kernel.org,
linux-scsi <linux-scsi@...r.kernel.org>,
intel-wired-lan@...ts.osuosl.org, netdev@...r.kernel.org,
linux-api@...r.kernel.org, linux-pci@...r.kernel.org,
Thomas Gleixner <tglx@...utronix.de>,
jesse.brandeburg@...el.com, robin.murphy@....com,
mtosatti@...hat.com, Ingo Molnar <mingo@...nel.org>,
jbrandeb@...nel.org, frederic@...nel.org, juri.lelli@...hat.com,
abelits@...vell.com, Bjorn Helgaas <bhelgaas@...gle.com>,
rostedt@...dmis.org, Peter Zijlstra <peterz@...radead.org>,
David Miller <davem@...emloft.net>,
Andrew Morton <akpm@...ux-foundation.org>,
Stephen Rothwell <sfr@...b.auug.org.au>,
stephen@...workplumber.org, rppt@...ux.vnet.ibm.com,
chris.friesen@...driver.com, maz@...nel.org,
Neil Horman <nhorman@...driver.com>, pjwaskiewicz@...il.com,
sassmann@...hat.com, Tomas Henzl <thenzl@...hat.com>,
Kashyap Desai <kashyap.desai@...adcom.com>,
Sumit Saxena <sumit.saxena@...adcom.com>,
Shivasharan Srikanteshwara
<shivasharan.srikanteshwara@...adcom.com>,
Sathya Prakash Veerichetty <sathya.prakash@...adcom.com>,
Suganath Prabu Subramani
<suganath-prabu.subramani@...adcom.com>, james.smart@...adcom.com,
dick.kennedy@...adcom.com, jkc@...hat.com, faisal.latif@...el.com,
shiraz.saleem@...el.com, tariqt@...dia.com, ahleihel@...hat.com,
kheib@...hat.com, borisp@...dia.com, saeedm@...dia.com,
benve@...co.com, govind@....com, jassisinghbrar@...il.com,
ajit.khaparde@...adcom.com, sriharsha.basavapatna@...adcom.com,
somnath.kotur@...adcom.com, nilal@...hat.com,
tatyana.e.nikolova@...el.com, mustafa.ismail@...el.com,
ahs3@...hat.com, leonro@...dia.com,
Chandrakanth Patil <chandrakanth.patil@...adcom.com>,
bjorn.andersson@...aro.org, chunkuang.hu@...nel.org,
yongqiang.niu@...iatek.com, baolin.wang7@...il.com,
poros@...hat.com, minlei@...hat.com,
"Ewan D. Milne" <emilne@...hat.com>,
"James E.J. Bottomley" <jejb@...ux.ibm.com>,
"Martin K. Petersen" <martin.petersen@...cle.com>, _govind@....com,
kabel@...nel.org, viresh.kumar@...aro.org,
Tushar.Khandelwal@....com, kuba@...nel.org
Subject: Re: [PATCH v5 05/14] scsi: mpt3sas: Use irq_set_affinity_and_hint
On Wed, Jul 21, 2021 at 4:57 AM Nitesh Narayan Lal <nitesh@...hat.com> wrote:
>
> The driver uses irq_set_affinity_hint() specifically for the high IOPS
> queue interrupts for two purposes:
>
> - To set the affinity_hint which is consumed by the userspace for
> distributing the interrupts
>
> - To apply an affinity that it provides
>
> The driver enforces its own affinity to bind the high IOPS queue interrupts
> to the local NUMA node. However, irq_set_affinity_hint() applying the
> provided cpumask as an affinity (if not NULL) for the interrupt is an
> undocumented side effect.
>
> To remove this side effect irq_set_affinity_hint() has been marked
> as deprecated and new interfaces have been introduced. Hence, replace the
> irq_set_affinity_hint() with the new interface irq_set_affinity_and_hint()
> where the provided mask needs to be applied as the affinity and
> affinity_hint pointer needs to be set and replace with
> irq_update_affinity_hint() where only affinity_hint needs to be updated.
>
Changes looks good and also verified that the high iops queue's IRQs
are affinitied to local numa node.
Reviewed-by: Sreekanth Reddy <sreekanth.reddy@...adcom.com>
> Signed-off-by: Nitesh Narayan Lal <nitesh@...hat.com>
> ---
> drivers/scsi/mpt3sas/mpt3sas_base.c | 21 ++++++++++-----------
> 1 file changed, 10 insertions(+), 11 deletions(-)
>
> diff --git a/drivers/scsi/mpt3sas/mpt3sas_base.c b/drivers/scsi/mpt3sas/mpt3sas_base.c
> index c39955239d1c..c1a11962f227 100644
> --- a/drivers/scsi/mpt3sas/mpt3sas_base.c
> +++ b/drivers/scsi/mpt3sas/mpt3sas_base.c
> @@ -2991,6 +2991,7 @@ _base_check_enable_msix(struct MPT3SAS_ADAPTER *ioc)
> static void
> _base_free_irq(struct MPT3SAS_ADAPTER *ioc)
> {
> + unsigned int irq;
> struct adapter_reply_queue *reply_q, *next;
>
> if (list_empty(&ioc->reply_queue_list))
> @@ -2998,9 +2999,10 @@ _base_free_irq(struct MPT3SAS_ADAPTER *ioc)
>
> list_for_each_entry_safe(reply_q, next, &ioc->reply_queue_list, list) {
> list_del(&reply_q->list);
> - if (ioc->smp_affinity_enable)
> - irq_set_affinity_hint(pci_irq_vector(ioc->pdev,
> - reply_q->msix_index), NULL);
> + if (ioc->smp_affinity_enable) {
> + irq = pci_irq_vector(ioc->pdev, reply_q->msix_index);
> + irq_update_affinity_hint(irq, NULL);
> + }
> free_irq(pci_irq_vector(ioc->pdev, reply_q->msix_index),
> reply_q);
> kfree(reply_q);
> @@ -3056,16 +3058,13 @@ _base_request_irq(struct MPT3SAS_ADAPTER *ioc, u8 index)
> * @ioc: per adapter object
> *
> * The enduser would need to set the affinity via /proc/irq/#/smp_affinity
> - *
> - * It would nice if we could call irq_set_affinity, however it is not
> - * an exported symbol
> */
> static void
> _base_assign_reply_queues(struct MPT3SAS_ADAPTER *ioc)
> {
> - unsigned int cpu, nr_cpus, nr_msix, index = 0;
> + unsigned int cpu, nr_cpus, nr_msix, index = 0, irq;
> struct adapter_reply_queue *reply_q;
> - int local_numa_node;
> + const struct cpumask *mask;
>
> if (!_base_is_controller_msix_enabled(ioc))
> return;
> @@ -3088,11 +3087,11 @@ _base_assign_reply_queues(struct MPT3SAS_ADAPTER *ioc)
> * corresponding to high iops queues.
> */
> if (ioc->high_iops_queues) {
> - local_numa_node = dev_to_node(&ioc->pdev->dev);
> + mask = cpumask_of_node(dev_to_node(&ioc->pdev->dev));
> for (index = 0; index < ioc->high_iops_queues;
> index++) {
> - irq_set_affinity_hint(pci_irq_vector(ioc->pdev,
> - index), cpumask_of_node(local_numa_node));
> + irq = pci_irq_vector(ioc->pdev, index);
> + irq_set_affinity_and_hint(irq, mask);
> }
> }
>
> --
> 2.27.0
>
Download attachment "smime.p7s" of type "application/pkcs7-signature" (4218 bytes)
Powered by blists - more mailing lists