[PATCH 0/4] vfio: fix a couple of spelling mistakes detected by codespell tool

[PATCH 0/4] vfio: fix a couple of spelling mistakes detected by codespell tool

[Zhen Lei]

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(-)

-- 
1.8.3

[PATCH 1/4] vfio/type1: fix a couple of spelling mistakes

[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

[PATCH 2/4] vfio/mdev: Fix spelling mistake "interal" -> "internal"

[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

[PATCH 3/4] vfio/pci: fix a couple of spelling mistakes

[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

[PATCH 4/4] vfio/platform: Fix spelling mistake "registe" -> "register"

[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

Re: [PATCH 4/4] vfio/platform: Fix spelling mistake "registe" -> "register"

[Auger Eric]

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 */
>  
> 

Re: [PATCH 3/4] vfio/pci: fix a couple of spelling mistakes

[Auger Eric]

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))
> 

Re: [PATCH 2/4] vfio/mdev: Fix spelling mistake "interal" -> "internal"

[Auger Eric]

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>
> 

Re: [PATCH 1/4] vfio/type1: fix a couple of spelling mistakes

[Auger Eric]

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.
> 

Re: [PATCH 0/4] vfio: fix a couple of spelling mistakes detected by codespell tool

[Alex Williamson]

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