2021-03-25 03:36:06

by Bhaskar Chowdhury

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

s/acrros/across/

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

Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
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

2021-03-25 03:38:35

by Randy Dunlap

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

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 <[email protected]>

Acked-by: Randy Dunlap <[email protected]>

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

2021-03-25 15:24:28

by Stefano Stabellini

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

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 <[email protected]>
>
> Acked-by: Randy Dunlap <[email protected]>

Acked-by: Stefano Stabellini <[email protected]>


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