Re: [PATCH V2] ARM: xen/mm.c: A mundane typo fix

Re: [PATCH V2] ARM: xen/mm.c: A mundane typo fix

[Randy Dunlap]

On 3/24/21 9:35 PM, Bhaskar Chowdhury wrote:
> s/acrros/across/
> 
> Plus some words need prural version...so did it.(page->pages)
> 
> Signed-off-by: Bhaskar Chowdhury <unixbhaskar@gmail.com>

Acked-by: Randy Dunlap <rdunlap@infradead.org>

> ---
>  Changes from V1:
>   Randy's findings incorporated.
> 
>  arch/arm/xen/mm.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/xen/mm.c b/arch/arm/xen/mm.c
> index 467fa225c3d0..8596dd32dcd5 100644
> --- a/arch/arm/xen/mm.c
> +++ b/arch/arm/xen/mm.c
> @@ -105,8 +105,8 @@ bool xen_arch_need_swiotlb(struct device *dev,
>  	 *	- The Linux page refers to foreign memory
>  	 *	- The device doesn't support coherent DMA request
>  	 *
> -	 * The Linux page may be spanned acrros multiple Xen page, although
> -	 * it's not possible to have a mix of local and foreign Xen page.
> +	 * The Linux page may be spanned across multiple Xen pages, although
> +	 * it's not possible to have a mix of local and foreign Xen pages.
>  	 * Furthermore, range_straddles_page_boundary is already checking
>  	 * if buffer is physically contiguous in the host RAM.
>  	 *
> --


-- 
~Randy

[PATCH V2] ARM: xen/mm.c: A mundane typo fix

[Bhaskar Chowdhury]

s/acrros/across/

Plus some words need prural version...so did it.(page->pages)

Signed-off-by: Bhaskar Chowdhury <unixbhaskar@gmail.com>
---
 Changes from V1:
  Randy's findings incorporated.

 arch/arm/xen/mm.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/xen/mm.c b/arch/arm/xen/mm.c
index 467fa225c3d0..8596dd32dcd5 100644
--- a/arch/arm/xen/mm.c
+++ b/arch/arm/xen/mm.c
@@ -105,8 +105,8 @@ bool xen_arch_need_swiotlb(struct device *dev,
 	 *	- The Linux page refers to foreign memory
 	 *	- The device doesn't support coherent DMA request
 	 *
-	 * The Linux page may be spanned acrros multiple Xen page, although
-	 * it's not possible to have a mix of local and foreign Xen page.
+	 * The Linux page may be spanned across multiple Xen pages, although
+	 * it's not possible to have a mix of local and foreign Xen pages.
 	 * Furthermore, range_straddles_page_boundary is already checking
 	 * if buffer is physically contiguous in the host RAM.
 	 *
--
2.30.1

Re: [PATCH V2] ARM: xen/mm.c: A mundane typo fix

[Stefano Stabellini]

On Wed, 24 Mar 2021, Randy Dunlap wrote:
> On 3/24/21 9:35 PM, Bhaskar Chowdhury wrote:
> > s/acrros/across/
> > 
> > Plus some words need prural version...so did it.(page->pages)
> > 
> > Signed-off-by: Bhaskar Chowdhury <unixbhaskar@gmail.com>
> 
> Acked-by: Randy Dunlap <rdunlap@infradead.org>

Acked-by: Stefano Stabellini <sstabellini@kernel.org>


> > ---
> >  Changes from V1:
> >   Randy's findings incorporated.
> > 
> >  arch/arm/xen/mm.c | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/arch/arm/xen/mm.c b/arch/arm/xen/mm.c
> > index 467fa225c3d0..8596dd32dcd5 100644
> > --- a/arch/arm/xen/mm.c
> > +++ b/arch/arm/xen/mm.c
> > @@ -105,8 +105,8 @@ bool xen_arch_need_swiotlb(struct device *dev,
> >  	 *	- The Linux page refers to foreign memory
> >  	 *	- The device doesn't support coherent DMA request
> >  	 *
> > -	 * The Linux page may be spanned acrros multiple Xen page, although
> > -	 * it's not possible to have a mix of local and foreign Xen page.
> > +	 * The Linux page may be spanned across multiple Xen pages, although
> > +	 * it's not possible to have a mix of local and foreign Xen pages.
> >  	 * Furthermore, range_straddles_page_boundary is already checking
> >  	 * if buffer is physically contiguous in the host RAM.
> >  	 *
> > --
> 
> 
> -- 
> ~Randy
>