2021-01-28 11:25:42

by Miaohe Lin

[permalink] [raw]
Subject: [PATCH] mm/hugetlb: Fix some comment typos

Fix typos sasitfy to satisfy, reservtion to reservation, hugegpage to
hugepage and uniprocesor to uniprocessor in comments.

Signed-off-by: Miaohe Lin <[email protected]>
---
include/linux/hugetlb.h | 2 +-
mm/hugetlb.c | 6 +++---
2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/include/linux/hugetlb.h b/include/linux/hugetlb.h
index ef5b144b8aac..95a5a239c8f2 100644
--- a/include/linux/hugetlb.h
+++ b/include/linux/hugetlb.h
@@ -37,7 +37,7 @@ struct hugepage_subpool {
struct hstate *hstate;
long min_hpages; /* Minimum huge pages or -1 if no minimum. */
long rsv_hpages; /* Pages reserved against global pool to */
- /* sasitfy minimum size. */
+ /* satisfy minimum size. */
};

struct resv_map {
diff --git a/mm/hugetlb.c b/mm/hugetlb.c
index 53ea65d1c5ab..c42c61c2653e 100644
--- a/mm/hugetlb.c
+++ b/mm/hugetlb.c
@@ -1444,7 +1444,7 @@ static void __free_huge_page(struct page *page)
* reservation. If the page was associated with a subpool, there
* would have been a page reserved in the subpool before allocation
* via hugepage_subpool_get_pages(). Since we are 'restoring' the
- * reservtion, do not call hugepage_subpool_put_pages() as this will
+ * reservation, do not call hugepage_subpool_put_pages() as this will
* remove the reserved page from the subpool.
*/
if (!restore_reserve) {
@@ -3715,7 +3715,7 @@ static unsigned long hugetlb_vm_op_pagesize(struct vm_area_struct *vma)
/*
* We cannot handle pagefaults against hugetlb pages at all. They cause
* handle_mm_fault() to try to instantiate regular-sized pages in the
- * hugegpage VMA. do_page_fault() is supposed to trap this, so BUG is we get
+ * hugepage VMA. do_page_fault() is supposed to trap this, so BUG is we get
* this far.
*/
static vm_fault_t hugetlb_vm_op_fault(struct vm_fault *vmf)
@@ -4513,7 +4513,7 @@ u32 hugetlb_fault_mutex_hash(struct address_space *mapping, pgoff_t idx)
}
#else
/*
- * For uniprocesor systems we always use a single mutex, so just
+ * For uniprocessor systems we always use a single mutex, so just
* return 0 and avoid the hashing overhead.
*/
u32 hugetlb_fault_mutex_hash(struct address_space *mapping, pgoff_t idx)
--
2.19.1


2021-01-29 03:54:40

by Souptick Joarder

[permalink] [raw]
Subject: Re: [PATCH] mm/hugetlb: Fix some comment typos

On Thu, Jan 28, 2021 at 4:50 PM Miaohe Lin <[email protected]> wrote:
>
> Fix typos sasitfy to satisfy, reservtion to reservation, hugegpage to
> hugepage and uniprocesor to uniprocessor in comments.
>
> Signed-off-by: Miaohe Lin <[email protected]>

Reviewed-by: Souptick Joarder <[email protected]>

> ---
> include/linux/hugetlb.h | 2 +-
> mm/hugetlb.c | 6 +++---
> 2 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/include/linux/hugetlb.h b/include/linux/hugetlb.h
> index ef5b144b8aac..95a5a239c8f2 100644
> --- a/include/linux/hugetlb.h
> +++ b/include/linux/hugetlb.h
> @@ -37,7 +37,7 @@ struct hugepage_subpool {
> struct hstate *hstate;
> long min_hpages; /* Minimum huge pages or -1 if no minimum. */
> long rsv_hpages; /* Pages reserved against global pool to */
> - /* sasitfy minimum size. */
> + /* satisfy minimum size. */
> };
>
> struct resv_map {
> diff --git a/mm/hugetlb.c b/mm/hugetlb.c
> index 53ea65d1c5ab..c42c61c2653e 100644
> --- a/mm/hugetlb.c
> +++ b/mm/hugetlb.c
> @@ -1444,7 +1444,7 @@ static void __free_huge_page(struct page *page)
> * reservation. If the page was associated with a subpool, there
> * would have been a page reserved in the subpool before allocation
> * via hugepage_subpool_get_pages(). Since we are 'restoring' the
> - * reservtion, do not call hugepage_subpool_put_pages() as this will
> + * reservation, do not call hugepage_subpool_put_pages() as this will
> * remove the reserved page from the subpool.
> */
> if (!restore_reserve) {
> @@ -3715,7 +3715,7 @@ static unsigned long hugetlb_vm_op_pagesize(struct vm_area_struct *vma)
> /*
> * We cannot handle pagefaults against hugetlb pages at all. They cause
> * handle_mm_fault() to try to instantiate regular-sized pages in the
> - * hugegpage VMA. do_page_fault() is supposed to trap this, so BUG is we get
> + * hugepage VMA. do_page_fault() is supposed to trap this, so BUG is we get
> * this far.
> */
> static vm_fault_t hugetlb_vm_op_fault(struct vm_fault *vmf)
> @@ -4513,7 +4513,7 @@ u32 hugetlb_fault_mutex_hash(struct address_space *mapping, pgoff_t idx)
> }
> #else
> /*
> - * For uniprocesor systems we always use a single mutex, so just
> + * For uniprocessor systems we always use a single mutex, so just
> * return 0 and avoid the hashing overhead.
> */
> u32 hugetlb_fault_mutex_hash(struct address_space *mapping, pgoff_t idx)
> --
> 2.19.1
>
>