[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20201117165319.GA83217@rlk>
Date: Wed, 18 Nov 2020 00:53:19 +0800
From: Hui Su <sh_def@....com>
To: akpm@...ux-foundation.org, linux-mm@...ck.org,
linux-kernel@...r.kernel.org
Cc: sh_def@....com
Subject: Re: [PATCH] mm/lru: simplify is_file_lru() and is_active_lru()
On Wed, Nov 18, 2020 at 12:49:19AM +0800, Hui Su wrote:
> page->lru bit 0 can tell whether the page is
> avtive page or not.
> page->lru bit 1 can tell whether the page is
> file page or not.
>
> Signed-off-by: Hui Su <sh_def@....com>
> ---
> include/linux/mmzone.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/include/linux/mmzone.h b/include/linux/mmzone.h
> index fb3bf696c05e..294369c652d0 100644
> --- a/include/linux/mmzone.h
> +++ b/include/linux/mmzone.h
> @@ -258,12 +258,12 @@ enum lru_list {
>
> static inline bool is_file_lru(enum lru_list lru)
> {
> - return (lru == LRU_INACTIVE_FILE || lru == LRU_ACTIVE_FILE);
> + return lru & LRU_FILE;
> }
>
> static inline bool is_active_lru(enum lru_list lru)
> {
> - return (lru == LRU_ACTIVE_ANON || lru == LRU_ACTIVE_FILE);
> + return lru & LRU_ACTIVE;
> }
>
> #define ANON_AND_FILE 2
> --
> 2.29.0
The commit message is not correct, please ignore this change.
Powered by blists - more mailing lists