Re: [PATCH 2/4] powerpc/mm: map shared vmemmap tail pages as read-only

[IBM]

Muchun Song <songmuchun@xxxxxxxxxxxxx> writes:

> PowerPC radix uses an architecture-specific vmemmap population path for
> shared tail mappings, so it does not inherit the read-only protection added
> to the common path.
>
> Protect these mappings as well so illegal writes fault instead of
> corrupting metadata shared by multiple tail pages.
>
> Signed-off-by: Muchun Song <songmuchun@xxxxxxxxxxxxx>
> ---
>  arch/powerpc/mm/book3s64/radix_pgtable.c | 3 +++
>  1 file changed, 3 insertions(+)

Looks good to me. Thanks for taking care of powerpc path as well.
Feel free to add:

Reviewed-by: Ritesh Harjani (IBM) <ritesh.list@xxxxxxxxx>