Re: [PATCH RESEND] x86: consider effective protection attributes in W+X check

From: Ingo Molnar
Date: Tue Feb 20 2018 - 03:10:53 EST



* Jan Beulich <JBeulich@xxxxxxxx> wrote:

> Using just the leaf page table entry flags would cause a false warning
> in case _PAGE_RW is clear or _PAGE_NX is set in a higher level entry.

Under what circumstances did you see false positive warnings?

> Hand through both the current entry's flags as well as the accumulated
> effective value (the latter as pgprotval_t instead of pgprot_t, as it's
> not an actual entry's value).
>
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
> Reviewed-by: Juergen Gross <jgross@xxxxxxxx>
> ---
> arch/x86/mm/dump_pagetables.c | 92 ++++++++++++++++++++++++++----------------
> 1 file changed, 57 insertions(+), 35 deletions(-)

Could you please rebase this on top of latest tip:master, which changed
arch/x86/mm/dump_pagetables.c non-trivially due the dynamic 5 level paging
changes?

Thanks,

Ingo