RE: [PATCH] Fix incorrect compound page flags store

["yaoaili [么爱利]" <yaoaili@kingsoft.com>]

Thanks all the same!
And where can I get latest mmotm?

-----Original Message-----
From: HORIGUCHI NAOYA(堀口　直也) [mailto:naoya.horiguchi@nec.com] 
Sent: Tuesday, September 8, 2020 3:02 PM
To: yaoaili126@163.com
Cc: linux-mm@kvack.org; YANGFENG1 [杨峰] <YANGFENG1@kingsoft.com>; yaoaili [么爱利] <yaoaili@kingsoft.com>; willy@infradead.org
Subject: Re: [PATCH] Fix incorrect compound page flags store

On Mon, Sep 07, 2020 at 08:44:42PM -0700, yaoaili126@163.com wrote:
> From: Aili Yao <yaoaili@kingsoft.com>
> 
> PageHuge(p) branch will never be true,but for compound page we need to set page_flags to correct value.
> 
> Reviewed-by: Matthew Wilcox (Oracle) <willy@infradead.org>
> Signed-off-by: Yang Feng < yangfeng1@kingsoft.com>
> Signed-off-by: Aili Yao <yaoaili@kingsoft.com>

I found that this PageHuge() check is removed and no long exists in the latest mmotm, so we don't have worry about it.
Sorry for missing it in my previous review.

Thanks,
Naoya Horiguchi

> ---
>  mm/memory-failure.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/mm/memory-failure.c b/mm/memory-failure.c index 
> f1aa6433f404..e6995976b11d 100644
> --- a/mm/memory-failure.c
> +++ b/mm/memory-failure.c
> @@ -1382,7 +1382,7 @@ int memory_failure(unsigned long pfn, int flags)
>  	 * page_remove_rmap() in try_to_unmap_one(). So to determine page status
>  	 * correctly, we save a copy of the page flags at this time.
>  	 */
> -	if (PageHuge(p))
> +	if (PageCompound(p))
>  		page_flags = hpage->flags;
>  	else
>  		page_flags = p->flags;
> --
> 2.18.4
> 
>