linux-kernel - Re: [PATCH] mm, docs: fix comments that mention mem_hotplug

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAFqt6zYF3wRRpmhGzK_r9436JbWfVyjnKKp0oWpSZ+3zad=bxw@mail.gmail.com>
Date:   Mon, 20 Jun 2022 07:26:34 +0530
From:   Souptick Joarder <jrdr.linux@...il.com>
To:     Yun-Ze Li <p76091292@...ncku.edu.tw>
Cc:     Andrew Morton <akpm@...ux-foundation.org>,
        Linux-MM <linux-mm@...ck.org>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] mm, docs: fix comments that mention mem_hotplug_end()

On Mon, Jun 20, 2022 at 12:06 AM Yun-Ze Li <p76091292@...ncku.edu.tw> wrote:
>
> Comments that mention mem_hotplug_end() are confusing as there is no
> function called mem_hotplug_end(). Fix them by replacing all the
> occurences of mem_hotplug_end() in the comments with mem_hotplug_done().
>
> Signed-off-by: Yun-Ze Li <p76091292@...ncku.edu.tw>
> ---
>  include/linux/mmzone.h | 4 ++--
>  mm/compaction.c        | 2 +-
>  mm/vmscan.c            | 2 +-
>  3 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/include/linux/mmzone.h b/include/linux/mmzone.h
> index aab70355d64f..7c7947109986 100644
> --- a/include/linux/mmzone.h
> +++ b/include/linux/mmzone.h
> @@ -591,7 +591,7 @@ struct zone {
>          * give them a chance of being in the same cacheline.
>          *
>          * Write access to present_pages at runtime should be protected by
> -        * mem_hotplug_begin/end(). Any reader who can't tolerant drift of
> +        * mem_hotplug_begin/done(). Any reader who can't tolerant drift of
>          * present_pages should get_online_mems() to get a stable value.
>          */
>         atomic_long_t           managed_pages;
> @@ -870,7 +870,7 @@ typedef struct pglist_data {
>         unsigned long nr_reclaim_start; /* nr pages written while throttled
>                                          * when throttling started. */
>         struct task_struct *kswapd;     /* Protected by
> -                                          mem_hotplug_begin/end() */
> +                                          mem_hotplug_begin/done() */
>         int kswapd_order;
>         enum zone_type kswapd_highest_zoneidx;
>
> diff --git a/mm/compaction.c b/mm/compaction.c
> index 65970107b789..b8c1f60c84b9 100644
> --- a/mm/compaction.c
> +++ b/mm/compaction.c
> @@ -3009,7 +3009,7 @@ void kcompactd_run(int nid)
>
>  /*
>   * Called by memory hotplug when all memory in a node is offlined. Caller must
> - * hold mem_hotplug_begin/end().
> + * hold mem_hotplug_begin/done().

Caller must be hold mem_hotplug_begin/done().

>   */
>  void kcompactd_stop(int nid)
>  {
> diff --git a/mm/vmscan.c b/mm/vmscan.c
> index c6918fff06e1..bd6aaf85ee7f 100644
> --- a/mm/vmscan.c
> +++ b/mm/vmscan.c
> @@ -4586,7 +4586,7 @@ void kswapd_run(int nid)
>
>  /*
>   * Called by memory hotplug when all memory in a node is offlined.  Caller must
> - * hold mem_hotplug_begin/end().
> + * hold mem_hotplug_begin/done().

Same here.

>   */
>  void kswapd_stop(int nid)
>  {
> --
> 2.25.1
>
>