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

Subsystems: arm port, the rest, xen hypervisor arm

STALE1955d

5 messages, 3 authors, 2021-03-24 · open the first message on its own page

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

From: Bhaskar Chowdhury <hidden>
Date: 2021-03-24 13:29:57

s/acrros/across/

Signed-off-by: Bhaskar Chowdhury <redacted>
---
 arch/arm/xen/mm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/xen/mm.c b/arch/arm/xen/mm.c
index 467fa225c3d0..be7c942c74bf 100644
--- a/arch/arm/xen/mm.c
+++ b/arch/arm/xen/mm.c
@@ -105,7 +105,7 @@ 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
+	 * The Linux page may be spanned across multiple Xen page, although
 	 * it's not possible to have a mix of local and foreign Xen page.
 	 * Furthermore, range_straddles_page_boundary is already checking
 	 * if buffer is physically contiguous in the host RAM.
--
2.30.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

From: Stefano Stabellini <sstabellini@kernel.org>
Date: 2021-03-24 18:56:14

On Wed, 24 Mar 2021, Bhaskar Chowdhury wrote:
s/acrros/across/

Signed-off-by: Bhaskar Chowdhury <redacted>
Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>

quoted hunk
---
 arch/arm/xen/mm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/xen/mm.c b/arch/arm/xen/mm.c
index 467fa225c3d0..be7c942c74bf 100644
--- a/arch/arm/xen/mm.c
+++ b/arch/arm/xen/mm.c
@@ -105,7 +105,7 @@ 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
+	 * The Linux page may be spanned across multiple Xen page, although
 	 * it's not possible to have a mix of local and foreign Xen page.
 	 * Furthermore, range_straddles_page_boundary is already checking
 	 * if buffer is physically contiguous in the host RAM.
--
2.30.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

From: Randy Dunlap <hidden>
Date: 2021-03-24 19:00:00

On 3/24/21 11:55 AM, Stefano Stabellini wrote:
On Wed, 24 Mar 2021, Bhaskar Chowdhury wrote:
quoted
s/acrros/across/

Signed-off-by: Bhaskar Chowdhury <redacted>
Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>
Hi,
It seems to me like some of those "page" should be "pages".
quoted
---
 arch/arm/xen/mm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/xen/mm.c b/arch/arm/xen/mm.c
index 467fa225c3d0..be7c942c74bf 100644
--- a/arch/arm/xen/mm.c
+++ b/arch/arm/xen/mm.c
@@ -105,7 +105,7 @@ 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>> +	 * The Linux page may be spanned across multiple Xen page, although
	                                                     pages,
quoted
 	 * it's not possible to have a mix of local and foreign Xen page.
	                                                            pages.
quoted
 	 * Furthermore, range_straddles_page_boundary is already checking
 	 * if buffer is physically contiguous in the host RAM.
--

-- 
~Randy


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

From: Stefano Stabellini <sstabellini@kernel.org>
Date: 2021-03-24 19:12:28

On Wed, 24 Mar 2021, Randy Dunlap wrote:
On 3/24/21 11:55 AM, Stefano Stabellini wrote:
quoted
On Wed, 24 Mar 2021, Bhaskar Chowdhury wrote:
quoted
s/acrros/across/

Signed-off-by: Bhaskar Chowdhury <redacted>
Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>
Hi,
It seems to me like some of those "page" should be "pages".
Yes, good point actually

quoted
quoted
---
 arch/arm/xen/mm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/xen/mm.c b/arch/arm/xen/mm.c
index 467fa225c3d0..be7c942c74bf 100644
--- a/arch/arm/xen/mm.c
+++ b/arch/arm/xen/mm.c
@@ -105,7 +105,7 @@ 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>> +	 * The Linux page may be spanned across multiple Xen page, although
	                                                     pages,
quoted
quoted
 	 * it's not possible to have a mix of local and foreign Xen page.
	                                                            pages.
yes to both

quoted
quoted
 	 * Furthermore, range_straddles_page_boundary is already checking
 	 * if buffer is physically contiguous in the host RAM.
--

-- 
~Randy
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

From: Bhaskar Chowdhury <hidden>
Date: 2021-03-24 23:08:07

On 11:59 Wed 24 Mar 2021, Randy Dunlap wrote:
On 3/24/21 11:55 AM, Stefano Stabellini wrote:
quoted
On Wed, 24 Mar 2021, Bhaskar Chowdhury wrote:
quoted
s/acrros/across/

Signed-off-by: Bhaskar Chowdhury <redacted>
Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>
Hi,
It seems to me like some of those "page" should be "pages".
quoted
quoted
---
 arch/arm/xen/mm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/xen/mm.c b/arch/arm/xen/mm.c
index 467fa225c3d0..be7c942c74bf 100644
--- a/arch/arm/xen/mm.c
+++ b/arch/arm/xen/mm.c
@@ -105,7 +105,7 @@ 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>> +	 * The Linux page may be spanned across multiple Xen page, although
                                                     pages,
quoted
quoted
 	 * it's not possible to have a mix of local and foreign Xen page.
                                                            pages.
quoted
quoted
 	 * Furthermore, range_straddles_page_boundary is already checking
 	 * if buffer is physically contiguous in the host RAM.
--
Thank you....I have sent a V2 now.

~Randy
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help