Re: [PATCH] page-flags: Remove unused __[Set|Clear]PagePrivate

[David Hildenbrand]

On 09.10.20 15:59, Miaohe Lin wrote:
> They are unused anymore.

-EINVAL, sentence does not make sense :)

"They are not used anymore."
"They are unused ."

Reviewed-by: David Hildenbrand <david@xxxxxxxxxx>

> 
> Signed-off-by: Miaohe Lin <linmiaohe@xxxxxxxxxx>
> ---
>  include/linux/page-flags.h | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/include/linux/page-flags.h b/include/linux/page-flags.h
> index 4f6ba9379112..50cbf5e931bc 100644
> --- a/include/linux/page-flags.h
> +++ b/include/linux/page-flags.h
> @@ -363,8 +363,7 @@ PAGEFLAG(SwapBacked, swapbacked, PF_NO_TAIL)
>   * for its own purposes.
>   * - PG_private and PG_private_2 cause releasepage() and co to be invoked
>   */
> -PAGEFLAG(Private, private, PF_ANY) __SETPAGEFLAG(Private, private, PF_ANY)
> -	__CLEARPAGEFLAG(Private, private, PF_ANY)
> +PAGEFLAG(Private, private, PF_ANY)
>  PAGEFLAG(Private2, private_2, PF_ANY) TESTSCFLAG(Private2, private_2, PF_ANY)
>  PAGEFLAG(OwnerPriv1, owner_priv_1, PF_ANY)
>  	TESTCLEARFLAG(OwnerPriv1, owner_priv_1, PF_ANY)
> 


-- 
Thanks,

David / dhildenb