Re: [PATCH 4/6] x86: remove set_memory_x and set_memory_nx

From: Peter Zijlstra
Date: Fri Aug 23 2019 - 10:32:03 EST


On Tue, Aug 13, 2019 at 11:01:44AM +0200, Christoph Hellwig wrote:
> These wrappers don't provide a real benefit over just using
> set_memory_x and set_memory_nx.
>
> Signed-off-by: Christoph Hellwig <hch@xxxxxx>
> ---
> arch/x86/include/asm/set_memory.h | 2 --
> arch/x86/kernel/machine_kexec_32.c | 4 ++--
> arch/x86/mm/init_32.c | 2 +-
> arch/x86/mm/pageattr.c | 16 ----------------
> 4 files changed, 3 insertions(+), 21 deletions(-)
>
> diff --git a/arch/x86/include/asm/set_memory.h b/arch/x86/include/asm/set_memory.h
> index 899ec9ae7cff..fd549c3ebb17 100644
> --- a/arch/x86/include/asm/set_memory.h
> +++ b/arch/x86/include/asm/set_memory.h
> @@ -75,8 +75,6 @@ int set_pages_array_wb(struct page **pages, int addrinarray);
>
> int set_pages_uc(struct page *page, int numpages);
> int set_pages_wb(struct page *page, int numpages);
> -int set_pages_x(struct page *page, int numpages);
> -int set_pages_nx(struct page *page, int numpages);
> int set_pages_ro(struct page *page, int numpages);
> int set_pages_rw(struct page *page, int numpages);

$Subject and patch content don't match up.

Other than that,

Acked-by: Peter Zijlstra (Intel) <peterz@xxxxxxxxxxxxx>

for all x86 patches.