Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
References: <20210128112028.64831-1-linmiaohe@huawei.com>
In-Reply-To: <20210128112028.64831-1-linmiaohe@huawei.com>
From:   Souptick Joarder <jrdr.linux@gmail.com>
Date:   Fri, 29 Jan 2021 09:20:03 +0530
Message-ID: <CAFqt6zYX5f5Vf_S838XdnjTyLmDsmS+2OS16HpMuzjJjKhS5PQ@mail.gmail.com>
Subject: Re: [PATCH] mm/hugetlb: Fix some comment typos
To:     Miaohe Lin <linmiaohe@huawei.com>
Cc:     Andrew Morton <akpm@linux-foundation.org>,
        Mike Kravetz <mike.kravetz@oracle.com>,
        Linux-MM <linux-mm@kvack.org>, linux-kernel@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

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

Reviewed-by: Souptick Joarder <jrdr.linux@gmail.com>

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