* [PATCH 1/4] vfio/type1: fix a couple of spelling mistakes
2021-03-26 8:35 [PATCH 0/4] vfio: fix a couple of spelling mistakes detected by codespell tool Zhen Lei
@ 2021-03-26 8:35 ` Zhen Lei
2021-03-26 9:20 ` Auger Eric
2021-03-26 8:35 ` [PATCH 2/4] vfio/mdev: Fix spelling mistake "interal" -> "internal" Zhen Lei
` (3 subsequent siblings)
4 siblings, 1 reply; 10+ messages in thread
From: Zhen Lei @ 2021-03-26 8:35 UTC (permalink / raw)
To: Alex Williamson, Cornelia Huck, kvm, Kirti Wankhede, Eric Auger,
linux-kernel
Cc: Zhen Lei
There are several spelling mistakes, as follows:
userpsace ==> userspace
Accouting ==> Accounting
exlude ==> exclude
Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
---
drivers/vfio/vfio_iommu_type1.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/vfio/vfio_iommu_type1.c b/drivers/vfio/vfio_iommu_type1.c
index be444407664af74..21cf1d123036c82 100644
--- a/drivers/vfio/vfio_iommu_type1.c
+++ b/drivers/vfio/vfio_iommu_type1.c
@@ -16,7 +16,7 @@
* IOMMU to support the IOMMU API and have few to no restrictions around
* the IOVA range that can be mapped. The Type1 IOMMU is currently
* optimized for relatively static mappings of a userspace process with
- * userpsace pages pinned into memory. We also assume devices and IOMMU
+ * userspace pages pinned into memory. We also assume devices and IOMMU
* domains are PCI based as the IOMMU API is still centered around a
* device/bus interface rather than a group interface.
*/
@@ -871,7 +871,7 @@ static int vfio_iommu_type1_pin_pages(void *iommu_data,
/*
* If iommu capable domain exist in the container then all pages are
- * already pinned and accounted. Accouting should be done if there is no
+ * already pinned and accounted. Accounting should be done if there is no
* iommu capable domain in the container.
*/
do_accounting = !IS_IOMMU_CAP_DOMAIN_IN_CONTAINER(iommu);
@@ -2171,7 +2171,7 @@ static int vfio_iommu_resv_exclude(struct list_head *iova,
continue;
/*
* Insert a new node if current node overlaps with the
- * reserve region to exlude that from valid iova range.
+ * reserve region to exclude that from valid iova range.
* Note that, new node is inserted before the current
* node and finally the current node is deleted keeping
* the list updated and sorted.
--
1.8.3
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 1/4] vfio/type1: fix a couple of spelling mistakes
2021-03-26 8:35 ` [PATCH 1/4] vfio/type1: fix a couple of spelling mistakes Zhen Lei
@ 2021-03-26 9:20 ` Auger Eric
0 siblings, 0 replies; 10+ messages in thread
From: Auger Eric @ 2021-03-26 9:20 UTC (permalink / raw)
To: Zhen Lei, Alex Williamson, Cornelia Huck, kvm, Kirti Wankhede,
linux-kernel
Hi,
On 3/26/21 9:35 AM, Zhen Lei wrote:
> There are several spelling mistakes, as follows:
> userpsace ==> userspace
> Accouting ==> Accounting
> exlude ==> exclude
Reviewed-by: Eric Auger <eric.auger@redhat.com>
Thanks
Eric
>
> Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
> ---
> drivers/vfio/vfio_iommu_type1.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/vfio/vfio_iommu_type1.c b/drivers/vfio/vfio_iommu_type1.c
> index be444407664af74..21cf1d123036c82 100644
> --- a/drivers/vfio/vfio_iommu_type1.c
> +++ b/drivers/vfio/vfio_iommu_type1.c
> @@ -16,7 +16,7 @@
> * IOMMU to support the IOMMU API and have few to no restrictions around
> * the IOVA range that can be mapped. The Type1 IOMMU is currently
> * optimized for relatively static mappings of a userspace process with
> - * userpsace pages pinned into memory. We also assume devices and IOMMU
> + * userspace pages pinned into memory. We also assume devices and IOMMU
> * domains are PCI based as the IOMMU API is still centered around a
> * device/bus interface rather than a group interface.
> */
> @@ -871,7 +871,7 @@ static int vfio_iommu_type1_pin_pages(void *iommu_data,
>
> /*
> * If iommu capable domain exist in the container then all pages are
> - * already pinned and accounted. Accouting should be done if there is no
> + * already pinned and accounted. Accounting should be done if there is no
> * iommu capable domain in the container.
> */
> do_accounting = !IS_IOMMU_CAP_DOMAIN_IN_CONTAINER(iommu);
> @@ -2171,7 +2171,7 @@ static int vfio_iommu_resv_exclude(struct list_head *iova,
> continue;
> /*
> * Insert a new node if current node overlaps with the
> - * reserve region to exlude that from valid iova range.
> + * reserve region to exclude that from valid iova range.
> * Note that, new node is inserted before the current
> * node and finally the current node is deleted keeping
> * the list updated and sorted.
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 2/4] vfio/mdev: Fix spelling mistake "interal" -> "internal"
2021-03-26 8:35 [PATCH 0/4] vfio: fix a couple of spelling mistakes detected by codespell tool Zhen Lei
2021-03-26 8:35 ` [PATCH 1/4] vfio/type1: fix a couple of spelling mistakes Zhen Lei
@ 2021-03-26 8:35 ` Zhen Lei
2021-03-26 9:19 ` Auger Eric
2021-03-26 8:35 ` [PATCH 3/4] vfio/pci: fix a couple of spelling mistakes Zhen Lei
` (2 subsequent siblings)
4 siblings, 1 reply; 10+ messages in thread
From: Zhen Lei @ 2021-03-26 8:35 UTC (permalink / raw)
To: Alex Williamson, Cornelia Huck, kvm, Kirti Wankhede, Eric Auger,
linux-kernel
Cc: Zhen Lei
There is a spelling mistake in a comment, fix it.
Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
---
drivers/vfio/mdev/mdev_private.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/vfio/mdev/mdev_private.h b/drivers/vfio/mdev/mdev_private.h
index 7d922950caaf3c1..4d62b76c473409d 100644
--- a/drivers/vfio/mdev/mdev_private.h
+++ b/drivers/vfio/mdev/mdev_private.h
@@ -1,6 +1,6 @@
/* SPDX-License-Identifier: GPL-2.0-only */
/*
- * Mediated device interal definitions
+ * Mediated device internal definitions
*
* Copyright (c) 2016, NVIDIA CORPORATION. All rights reserved.
* Author: Neo Jia <cjia@nvidia.com>
--
1.8.3
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 2/4] vfio/mdev: Fix spelling mistake "interal" -> "internal"
2021-03-26 8:35 ` [PATCH 2/4] vfio/mdev: Fix spelling mistake "interal" -> "internal" Zhen Lei
@ 2021-03-26 9:19 ` Auger Eric
0 siblings, 0 replies; 10+ messages in thread
From: Auger Eric @ 2021-03-26 9:19 UTC (permalink / raw)
To: Zhen Lei, Alex Williamson, Cornelia Huck, kvm, Kirti Wankhede,
linux-kernel
On 3/26/21 9:35 AM, Zhen Lei wrote:
> There is a spelling mistake in a comment, fix it.
>
> Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
Reviewed-by: Eric Auger <eric.auger@redhat.com>
> ---
> drivers/vfio/mdev/mdev_private.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/vfio/mdev/mdev_private.h b/drivers/vfio/mdev/mdev_private.h
> index 7d922950caaf3c1..4d62b76c473409d 100644
> --- a/drivers/vfio/mdev/mdev_private.h
> +++ b/drivers/vfio/mdev/mdev_private.h
> @@ -1,6 +1,6 @@
> /* SPDX-License-Identifier: GPL-2.0-only */
> /*
> - * Mediated device interal definitions
> + * Mediated device internal definitions
> *
> * Copyright (c) 2016, NVIDIA CORPORATION. All rights reserved.
> * Author: Neo Jia <cjia@nvidia.com>
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 3/4] vfio/pci: fix a couple of spelling mistakes
2021-03-26 8:35 [PATCH 0/4] vfio: fix a couple of spelling mistakes detected by codespell tool Zhen Lei
2021-03-26 8:35 ` [PATCH 1/4] vfio/type1: fix a couple of spelling mistakes Zhen Lei
2021-03-26 8:35 ` [PATCH 2/4] vfio/mdev: Fix spelling mistake "interal" -> "internal" Zhen Lei
@ 2021-03-26 8:35 ` Zhen Lei
2021-03-26 9:18 ` Auger Eric
2021-03-26 8:35 ` [PATCH 4/4] vfio/platform: Fix spelling mistake "registe" -> "register" Zhen Lei
2021-04-06 19:37 ` [PATCH 0/4] vfio: fix a couple of spelling mistakes detected by codespell tool Alex Williamson
4 siblings, 1 reply; 10+ messages in thread
From: Zhen Lei @ 2021-03-26 8:35 UTC (permalink / raw)
To: Alex Williamson, Cornelia Huck, kvm, Kirti Wankhede, Eric Auger,
linux-kernel
Cc: Zhen Lei
There are several spelling mistakes, as follows:
permision ==> permission
thru ==> through
presense ==> presence
Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
---
drivers/vfio/pci/vfio_pci.c | 2 +-
drivers/vfio/pci/vfio_pci_config.c | 2 +-
drivers/vfio/pci/vfio_pci_nvlink2.c | 4 ++--
3 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/vfio/pci/vfio_pci.c b/drivers/vfio/pci/vfio_pci.c
index 65e7e6b44578c29..d2ab8b5bc8a86fe 100644
--- a/drivers/vfio/pci/vfio_pci.c
+++ b/drivers/vfio/pci/vfio_pci.c
@@ -2409,7 +2409,7 @@ static int __init vfio_pci_init(void)
{
int ret;
- /* Allocate shared config space permision data used by all devices */
+ /* Allocate shared config space permission data used by all devices */
ret = vfio_pci_init_perm_bits();
if (ret)
return ret;
diff --git a/drivers/vfio/pci/vfio_pci_config.c b/drivers/vfio/pci/vfio_pci_config.c
index a402adee8a21558..d57f037f65b85d4 100644
--- a/drivers/vfio/pci/vfio_pci_config.c
+++ b/drivers/vfio/pci/vfio_pci_config.c
@@ -101,7 +101,7 @@
/*
* Read/Write Permission Bits - one bit for each bit in capability
* Any field can be read if it exists, but what is read depends on
- * whether the field is 'virtualized', or just pass thru to the
+ * whether the field is 'virtualized', or just pass through to the
* hardware. Any virtualized field is also virtualized for writes.
* Writes are only permitted if they have a 1 bit here.
*/
diff --git a/drivers/vfio/pci/vfio_pci_nvlink2.c b/drivers/vfio/pci/vfio_pci_nvlink2.c
index 9adcf6a8f888575..f276624fec79f68 100644
--- a/drivers/vfio/pci/vfio_pci_nvlink2.c
+++ b/drivers/vfio/pci/vfio_pci_nvlink2.c
@@ -219,7 +219,7 @@ int vfio_pci_nvdia_v100_nvlink2_init(struct vfio_pci_device *vdev)
unsigned long events = VFIO_GROUP_NOTIFY_SET_KVM;
/*
- * PCI config space does not tell us about NVLink presense but
+ * PCI config space does not tell us about NVLink presence but
* platform does, use this.
*/
npu_dev = pnv_pci_get_npu_dev(vdev->pdev, 0);
@@ -402,7 +402,7 @@ int vfio_pci_ibm_npu2_init(struct vfio_pci_device *vdev)
u32 link_speed = 0xff;
/*
- * PCI config space does not tell us about NVLink presense but
+ * PCI config space does not tell us about NVLink presence but
* platform does, use this.
*/
if (!pnv_pci_get_gpu_dev(vdev->pdev))
--
1.8.3
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 3/4] vfio/pci: fix a couple of spelling mistakes
2021-03-26 8:35 ` [PATCH 3/4] vfio/pci: fix a couple of spelling mistakes Zhen Lei
@ 2021-03-26 9:18 ` Auger Eric
0 siblings, 0 replies; 10+ messages in thread
From: Auger Eric @ 2021-03-26 9:18 UTC (permalink / raw)
To: Zhen Lei, Alex Williamson, Cornelia Huck, kvm, Kirti Wankhede,
linux-kernel
On 3/26/21 9:35 AM, Zhen Lei wrote:
> There are several spelling mistakes, as follows:
> permision ==> permission
> thru ==> through
> presense ==> presence
>
> Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
Reviewed-by: Eric Auger <eric.auger@redhat.com>
Eric
> ---
> drivers/vfio/pci/vfio_pci.c | 2 +-
> drivers/vfio/pci/vfio_pci_config.c | 2 +-
> drivers/vfio/pci/vfio_pci_nvlink2.c | 4 ++--
> 3 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/vfio/pci/vfio_pci.c b/drivers/vfio/pci/vfio_pci.c
> index 65e7e6b44578c29..d2ab8b5bc8a86fe 100644
> --- a/drivers/vfio/pci/vfio_pci.c
> +++ b/drivers/vfio/pci/vfio_pci.c
> @@ -2409,7 +2409,7 @@ static int __init vfio_pci_init(void)
> {
> int ret;
>
> - /* Allocate shared config space permision data used by all devices */
> + /* Allocate shared config space permission data used by all devices */
> ret = vfio_pci_init_perm_bits();
> if (ret)
> return ret;
> diff --git a/drivers/vfio/pci/vfio_pci_config.c b/drivers/vfio/pci/vfio_pci_config.c
> index a402adee8a21558..d57f037f65b85d4 100644
> --- a/drivers/vfio/pci/vfio_pci_config.c
> +++ b/drivers/vfio/pci/vfio_pci_config.c
> @@ -101,7 +101,7 @@
> /*
> * Read/Write Permission Bits - one bit for each bit in capability
> * Any field can be read if it exists, but what is read depends on
> - * whether the field is 'virtualized', or just pass thru to the
> + * whether the field is 'virtualized', or just pass through to the
> * hardware. Any virtualized field is also virtualized for writes.
> * Writes are only permitted if they have a 1 bit here.
> */
> diff --git a/drivers/vfio/pci/vfio_pci_nvlink2.c b/drivers/vfio/pci/vfio_pci_nvlink2.c
> index 9adcf6a8f888575..f276624fec79f68 100644
> --- a/drivers/vfio/pci/vfio_pci_nvlink2.c
> +++ b/drivers/vfio/pci/vfio_pci_nvlink2.c
> @@ -219,7 +219,7 @@ int vfio_pci_nvdia_v100_nvlink2_init(struct vfio_pci_device *vdev)
> unsigned long events = VFIO_GROUP_NOTIFY_SET_KVM;
>
> /*
> - * PCI config space does not tell us about NVLink presense but
> + * PCI config space does not tell us about NVLink presence but
> * platform does, use this.
> */
> npu_dev = pnv_pci_get_npu_dev(vdev->pdev, 0);
> @@ -402,7 +402,7 @@ int vfio_pci_ibm_npu2_init(struct vfio_pci_device *vdev)
> u32 link_speed = 0xff;
>
> /*
> - * PCI config space does not tell us about NVLink presense but
> + * PCI config space does not tell us about NVLink presence but
> * platform does, use this.
> */
> if (!pnv_pci_get_gpu_dev(vdev->pdev))
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 4/4] vfio/platform: Fix spelling mistake "registe" -> "register"
2021-03-26 8:35 [PATCH 0/4] vfio: fix a couple of spelling mistakes detected by codespell tool Zhen Lei
` (2 preceding siblings ...)
2021-03-26 8:35 ` [PATCH 3/4] vfio/pci: fix a couple of spelling mistakes Zhen Lei
@ 2021-03-26 8:35 ` Zhen Lei
2021-03-26 9:17 ` Auger Eric
2021-04-06 19:37 ` [PATCH 0/4] vfio: fix a couple of spelling mistakes detected by codespell tool Alex Williamson
4 siblings, 1 reply; 10+ messages in thread
From: Zhen Lei @ 2021-03-26 8:35 UTC (permalink / raw)
To: Alex Williamson, Cornelia Huck, kvm, Kirti Wankhede, Eric Auger,
linux-kernel
Cc: Zhen Lei
There is a spelling mistake in a comment, fix it.
Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
---
drivers/vfio/platform/reset/vfio_platform_calxedaxgmac.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/vfio/platform/reset/vfio_platform_calxedaxgmac.c b/drivers/vfio/platform/reset/vfio_platform_calxedaxgmac.c
index 09a9453b75c5592..63cc7f0b2e4a437 100644
--- a/drivers/vfio/platform/reset/vfio_platform_calxedaxgmac.c
+++ b/drivers/vfio/platform/reset/vfio_platform_calxedaxgmac.c
@@ -26,7 +26,7 @@
#define XGMAC_DMA_CONTROL 0x00000f18 /* Ctrl (Operational Mode) */
#define XGMAC_DMA_INTR_ENA 0x00000f1c /* Interrupt Enable */
-/* DMA Control registe defines */
+/* DMA Control register defines */
#define DMA_CONTROL_ST 0x00002000 /* Start/Stop Transmission */
#define DMA_CONTROL_SR 0x00000002 /* Start/Stop Receive */
--
1.8.3
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 4/4] vfio/platform: Fix spelling mistake "registe" -> "register"
2021-03-26 8:35 ` [PATCH 4/4] vfio/platform: Fix spelling mistake "registe" -> "register" Zhen Lei
@ 2021-03-26 9:17 ` Auger Eric
0 siblings, 0 replies; 10+ messages in thread
From: Auger Eric @ 2021-03-26 9:17 UTC (permalink / raw)
To: Zhen Lei, Alex Williamson, Cornelia Huck, kvm, Kirti Wankhede,
linux-kernel
Hi,
On 3/26/21 9:35 AM, Zhen Lei wrote:
> There is a spelling mistake in a comment, fix it.
>
> Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
Acked-by: Eric Auger <eric.auger@redhat.com>
Thanks
Eric
> ---
> drivers/vfio/platform/reset/vfio_platform_calxedaxgmac.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/vfio/platform/reset/vfio_platform_calxedaxgmac.c b/drivers/vfio/platform/reset/vfio_platform_calxedaxgmac.c
> index 09a9453b75c5592..63cc7f0b2e4a437 100644
> --- a/drivers/vfio/platform/reset/vfio_platform_calxedaxgmac.c
> +++ b/drivers/vfio/platform/reset/vfio_platform_calxedaxgmac.c
> @@ -26,7 +26,7 @@
> #define XGMAC_DMA_CONTROL 0x00000f18 /* Ctrl (Operational Mode) */
> #define XGMAC_DMA_INTR_ENA 0x00000f1c /* Interrupt Enable */
>
> -/* DMA Control registe defines */
> +/* DMA Control register defines */
> #define DMA_CONTROL_ST 0x00002000 /* Start/Stop Transmission */
> #define DMA_CONTROL_SR 0x00000002 /* Start/Stop Receive */
>
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 0/4] vfio: fix a couple of spelling mistakes detected by codespell tool
2021-03-26 8:35 [PATCH 0/4] vfio: fix a couple of spelling mistakes detected by codespell tool Zhen Lei
` (3 preceding siblings ...)
2021-03-26 8:35 ` [PATCH 4/4] vfio/platform: Fix spelling mistake "registe" -> "register" Zhen Lei
@ 2021-04-06 19:37 ` Alex Williamson
4 siblings, 0 replies; 10+ messages in thread
From: Alex Williamson @ 2021-04-06 19:37 UTC (permalink / raw)
To: Zhen Lei; +Cc: Cornelia Huck, kvm, Kirti Wankhede, Eric Auger, linux-kernel
On Fri, 26 Mar 2021 16:35:24 +0800
Zhen Lei <thunder.leizhen@huawei.com> wrote:
> This detection and correction covers the entire driver/vfio directory.
>
> Zhen Lei (4):
> vfio/type1: fix a couple of spelling mistakes
> vfio/mdev: Fix spelling mistake "interal" -> "internal"
> vfio/pci: fix a couple of spelling mistakes
> vfio/platform: Fix spelling mistake "registe" -> "register"
>
> drivers/vfio/mdev/mdev_private.h | 2 +-
> drivers/vfio/pci/vfio_pci.c | 2 +-
> drivers/vfio/pci/vfio_pci_config.c | 2 +-
> drivers/vfio/pci/vfio_pci_nvlink2.c | 4 ++--
> drivers/vfio/platform/reset/vfio_platform_calxedaxgmac.c | 2 +-
> drivers/vfio/vfio_iommu_type1.c | 6 +++---
> 6 files changed, 9 insertions(+), 9 deletions(-)
>
Applied to vfio next branch for v5.13. Thanks,
Alex
^ permalink raw reply [flat|nested] 10+ messages in thread