linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] docs:mm: fix spelling mistakes in heterogeneous memory management page
@ 2024-09-08 16:19 Dennis Lam
  2024-09-09  9:15 ` Lorenzo Stoakes
  2024-09-10 21:15 ` Jonathan Corbet
  0 siblings, 2 replies; 3+ messages in thread
From: Dennis Lam @ 2024-09-08 16:19 UTC (permalink / raw)
  To: jglisse, corbet; +Cc: linux-mm, linux-doc, linux-kernel, Dennis Lam

Signed-off-by: Dennis Lam <dennis.lamerice@gmail.com>
---
 Documentation/mm/hmm.rst | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/Documentation/mm/hmm.rst b/Documentation/mm/hmm.rst
index 0595098a74d9..f6d53c37a2ca 100644
--- a/Documentation/mm/hmm.rst
+++ b/Documentation/mm/hmm.rst
@@ -66,7 +66,7 @@ combinatorial explosion in the library entry points.
 Finally, with the advance of high level language constructs (in C++ but in
 other languages too) it is now possible for the compiler to leverage GPUs and
 other devices without programmer knowledge. Some compiler identified patterns
-are only do-able with a shared address space. It is also more reasonable to use
+are only doable with a shared address space. It is also more reasonable to use
 a shared address space for all other patterns.
 
 
@@ -267,7 +267,7 @@ functions are designed to make drivers easier to write and to centralize common
 code across drivers.
 
 Before migrating pages to device private memory, special device private
-``struct page`` need to be created. These will be used as special "swap"
+``struct page`` needs to be created. These will be used as special "swap"
 page table entries so that a CPU process will fault if it tries to access
 a page that has been migrated to device private memory.
 
@@ -322,7 +322,7 @@ between device driver specific code and shared common code:
    The ``invalidate_range_start()`` callback is passed a
    ``struct mmu_notifier_range`` with the ``event`` field set to
    ``MMU_NOTIFY_MIGRATE`` and the ``owner`` field set to
-   the ``args->pgmap_owner`` field passed to migrate_vma_setup(). This is
+   the ``args->pgmap_owner`` field passed to migrate_vma_setup(). This
    allows the device driver to skip the invalidation callback and only
    invalidate device private MMU mappings that are actually migrating.
    This is explained more in the next section.
@@ -405,7 +405,7 @@ can be used to make a memory range inaccessible from userspace.
 
 This replaces all mappings for pages in the given range with special swap
 entries. Any attempt to access the swap entry results in a fault which is
-resovled by replacing the entry with the original mapping. A driver gets
+resolved by replacing the entry with the original mapping. A driver gets
 notified that the mapping has been changed by MMU notifiers, after which point
 it will no longer have exclusive access to the page. Exclusive access is
 guaranteed to last until the driver drops the page lock and page reference, at
@@ -431,7 +431,7 @@ Same decision was made for memory cgroup. Device memory pages are accounted
 against same memory cgroup a regular page would be accounted to. This does
 simplify migration to and from device memory. This also means that migration
 back from device memory to regular memory cannot fail because it would
-go above memory cgroup limit. We might revisit this choice latter on once we
+go above memory cgroup limit. We might revisit this choice later on once we
 get more experience in how device memory is used and its impact on memory
 resource control.
 
-- 
2.46.0



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] docs:mm: fix spelling mistakes in heterogeneous memory management page
  2024-09-08 16:19 [PATCH] docs:mm: fix spelling mistakes in heterogeneous memory management page Dennis Lam
@ 2024-09-09  9:15 ` Lorenzo Stoakes
  2024-09-10 21:15 ` Jonathan Corbet
  1 sibling, 0 replies; 3+ messages in thread
From: Lorenzo Stoakes @ 2024-09-09  9:15 UTC (permalink / raw)
  To: Dennis Lam; +Cc: jglisse, corbet, linux-mm, linux-doc, linux-kernel

On Sun, Sep 08, 2024 at 12:19:28PM GMT, Dennis Lam wrote:
> Signed-off-by: Dennis Lam <dennis.lamerice@gmail.com>

LGTM,

Reviewed-by: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>

> ---
>  Documentation/mm/hmm.rst | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/Documentation/mm/hmm.rst b/Documentation/mm/hmm.rst
> index 0595098a74d9..f6d53c37a2ca 100644
> --- a/Documentation/mm/hmm.rst
> +++ b/Documentation/mm/hmm.rst
> @@ -66,7 +66,7 @@ combinatorial explosion in the library entry points.
>  Finally, with the advance of high level language constructs (in C++ but in
>  other languages too) it is now possible for the compiler to leverage GPUs and
>  other devices without programmer knowledge. Some compiler identified patterns
> -are only do-able with a shared address space. It is also more reasonable to use
> +are only doable with a shared address space. It is also more reasonable to use
>  a shared address space for all other patterns.
>
>
> @@ -267,7 +267,7 @@ functions are designed to make drivers easier to write and to centralize common
>  code across drivers.
>
>  Before migrating pages to device private memory, special device private
> -``struct page`` need to be created. These will be used as special "swap"
> +``struct page`` needs to be created. These will be used as special "swap"
>  page table entries so that a CPU process will fault if it tries to access
>  a page that has been migrated to device private memory.
>
> @@ -322,7 +322,7 @@ between device driver specific code and shared common code:
>     The ``invalidate_range_start()`` callback is passed a
>     ``struct mmu_notifier_range`` with the ``event`` field set to
>     ``MMU_NOTIFY_MIGRATE`` and the ``owner`` field set to
> -   the ``args->pgmap_owner`` field passed to migrate_vma_setup(). This is
> +   the ``args->pgmap_owner`` field passed to migrate_vma_setup(). This
>     allows the device driver to skip the invalidation callback and only
>     invalidate device private MMU mappings that are actually migrating.
>     This is explained more in the next section.
> @@ -405,7 +405,7 @@ can be used to make a memory range inaccessible from userspace.
>
>  This replaces all mappings for pages in the given range with special swap
>  entries. Any attempt to access the swap entry results in a fault which is
> -resovled by replacing the entry with the original mapping. A driver gets
> +resolved by replacing the entry with the original mapping. A driver gets
>  notified that the mapping has been changed by MMU notifiers, after which point
>  it will no longer have exclusive access to the page. Exclusive access is
>  guaranteed to last until the driver drops the page lock and page reference, at
> @@ -431,7 +431,7 @@ Same decision was made for memory cgroup. Device memory pages are accounted
>  against same memory cgroup a regular page would be accounted to. This does
>  simplify migration to and from device memory. This also means that migration
>  back from device memory to regular memory cannot fail because it would
> -go above memory cgroup limit. We might revisit this choice latter on once we
> +go above memory cgroup limit. We might revisit this choice later on once we
>  get more experience in how device memory is used and its impact on memory
>  resource control.
>
> --
> 2.46.0
>


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] docs:mm: fix spelling mistakes in heterogeneous memory management page
  2024-09-08 16:19 [PATCH] docs:mm: fix spelling mistakes in heterogeneous memory management page Dennis Lam
  2024-09-09  9:15 ` Lorenzo Stoakes
@ 2024-09-10 21:15 ` Jonathan Corbet
  1 sibling, 0 replies; 3+ messages in thread
From: Jonathan Corbet @ 2024-09-10 21:15 UTC (permalink / raw)
  To: Dennis Lam, jglisse; +Cc: linux-mm, linux-doc, linux-kernel, Dennis Lam

Dennis Lam <dennis.lamerice@gmail.com> writes:

> Signed-off-by: Dennis Lam <dennis.lamerice@gmail.com>
> ---
>  Documentation/mm/hmm.rst | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)

Applied, thanks.

(Just FYI, the merge window is approaching, so expect the response on
these to slow considerably soon).

jon


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2024-09-10 21:15 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-09-08 16:19 [PATCH] docs:mm: fix spelling mistakes in heterogeneous memory management page Dennis Lam
2024-09-09  9:15 ` Lorenzo Stoakes
2024-09-10 21:15 ` Jonathan Corbet

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox