| 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
| ||
|
Message-ID: <db916e9d-4439-2fc0-9e86-a36a3995aef3@infradead.org>
Date: Fri, 5 Feb 2021 09:27:23 -0800
From: Randy Dunlap <rdunlap@...radead.org>
To: Bhaskar Chowdhury <unixbhaskar@...il.com>,
james.smart@...adcom.com, dick.kennedy@...adcom.com,
jejb@...ux.ibm.com, martin.petersen@...cle.com,
linux-scsi@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] drivers: scsi: lpfc: Mundane spelling and sentence
construction fixes throughout the file
On 2/4/21 9:30 PM, Bhaskar Chowdhury wrote:
>
>
> Few spellings and sentence construction done throughout the file.
>
>
> Signed-off-by: Bhaskar Chowdhury <unixbhaskar@...il.com>
> ---
> drivers/scsi/lpfc/lpfc_init.c | 24 ++++++++++++------------
> 1 file changed, 12 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/scsi/lpfc/lpfc_init.c b/drivers/scsi/lpfc/lpfc_init.c
> index ac67f420ec26..923fadb7945a 100644
> --- a/drivers/scsi/lpfc/lpfc_init.c
> +++ b/drivers/scsi/lpfc/lpfc_init.c
> @@ -11022,7 +11022,7 @@ lpfc_cpu_affinity_check(struct lpfc_hba *phba, int vectors)
> /* We found a matching phys_id, so copy the IRQ info */
> cpup->eq = new_cpup->eq;
>
> - /* Bump start_cpu to the next slot to minmize the
> + /* Bump start_cpu to the next slot to minimize the
OK.
> * chance of having multiple unassigned CPU entries
> * selecting the same IRQ.
> */
> @@ -11076,7 +11076,7 @@ lpfc_cpu_affinity_check(struct lpfc_hba *phba, int vectors)
> /* We found an available entry, copy the IRQ info */
> cpup->eq = new_cpup->eq;
>
> - /* Bump start_cpu to the next slot to minmize the
> + /* Bump start_cpu to the next slot to minimize the
OK.
> * chance of having multiple unassigned CPU entries
> * selecting the same IRQ.
> */
> @@ -11246,14 +11246,14 @@ lpfc_cpuhp_get_eq(struct lpfc_hba *phba, unsigned int cpu,
> if (!maskp)
> continue;
> /*
> - * if irq is not affinitized to the cpu going
Original may not be a real word, but it conveys the meaning better
than the proposed change does.
> + * if irq is not affinities to the cpu going
> * then we don't need to poll the eq attached
> * to it.
> */
> if (!cpumask_and(tmp, maskp, cpumask_of(cpu)))
> continue;
> - /* get the cpus that are online and are affini-
> - * tized to this irq vector. If the count is
> + /* get the cpus that are online and are affinities
> + * to this irq vector. If the count is
ditto.
> * more than 1 then cpuhp is not going to shut-
> * down this vector. Since this cpu has not
> * gone offline yet, we need >1.
> @@ -11367,7 +11367,7 @@ lpfc_irq_clear_aff(struct lpfc_hba_eq_hdl *eqhdl)
> * online cpu on the phba's original_mask and migrate all offlining IRQ
> * affinities.
> *
> - * If cpu is coming online, reaffinitize the IRQ back to the onlining cpu.
> + * If cpu is coming online, again affinities the IRQ back to the on lining cpu.
ditto.
> *
> * Note: Call only if NUMA or NHT mode is enabled, otherwise rely on
> * PCI_IRQ_AFFINITY to auto-manage IRQ affinity.
> @@ -11401,7 +11401,7 @@ lpfc_irq_rebalance(struct lpfc_hba *phba, unsigned int cpu, bool offline)
>
> /* Found a valid CPU */
> if ((cpu_select < nr_cpu_ids) && (cpu_select != cpu)) {
> - /* Go through each eqhdl and ensure offlining
The original looks good to me.
> + /* Go through each eqhdl and ensure off lining
> * cpu aff_mask is migrated
> */
> for (idx = 0; idx < phba->cfg_irq_chann; idx++) {
> @@ -11597,7 +11597,7 @@ lpfc_sli4_enable_msix(struct lpfc_hba *phba)
> * this vector, set LPFC_CPU_FIRST_IRQ.
> *
> * With certain platforms its possible that irq
> - * vectors are affinitized to all the cpu's.
Original is better.
> + * vectors are affinities to all the cpu's.
> * This can result in each cpu_map.eq to be set
> * to the last vector, resulting in overwrite
> * of all the previous cpu_map.eq. Ensure that
> @@ -11635,7 +11635,7 @@ lpfc_sli4_enable_msix(struct lpfc_hba *phba)
> free_irq(eqhdl->irq, eqhdl);
> }
>
> - /* Unconfigure MSI-X capability structure */
ditto.
> + /* Not configure MSI-X capability structure */
> pci_free_irq_vectors(phba->pcidev);
>
> vec_fail_out:
> @@ -11744,7 +11744,7 @@ lpfc_sli4_enable_intr(struct lpfc_hba *phba, uint32_t cfg_mode)
> }
> }
>
> - /* Fallback to INTx if both MSI-X/MSI initalization failed */
> + /* Fallback to INTx if both MSI-X/MSI initialization failed */
OK.
> if (phba->intr_type == NONE) {
> retval = request_irq(phba->pcidev->irq, lpfc_sli4_intr_handler,
> IRQF_SHARED, LPFC_DRIVER_NAME, phba);
> @@ -12479,7 +12479,7 @@ lpfc_pci_probe_one_s3(struct pci_dev *pdev, const struct pci_device_id *pid)
> * lpfc_pci_remove_one_s3 - PCI func to unreg SLI-3 device from PCI subsystem.
> * @pdev: pointer to PCI device
> *
> - * This routine is to be called to disattach a device with SLI-3 interface
detach
> + * This routine is to be called to not attach a device with SLI-3 interface
> * spec from PCI subsystem. When an Emulex HBA with SLI-3 interface spec is
> * removed from PCI bus, it performs all the necessary cleanup for the HBA
> * device to be removed from the PCI subsystem properly.
> @@ -12940,7 +12940,7 @@ lpfc_log_write_firmware_error(struct lpfc_hba *phba, uint32_t offset,
> /* Three cases: (1) FW was not supported on the detected adapter.
> * (2) FW update has been locked out administratively.
> * (3) Some other error during FW update.
> - * In each case, an unmaskable message is written to the console
Original conveys the message better than the proposed change.
> + * In each case, an unusable message is written to the console
> * for admin diagnosis.
> */
> if (offset == ADD_STATUS_FW_NOT_SUPPORTED ||
> --
> 2.30.0
>
--
~Randy
Powered by blists - more mailing lists