Re: [PATCHv7 13/14] x86: Enable 5-level paging support

From: Ingo Molnar
Date: Thu Jun 22 2017 - 05:25:14 EST

* Kirill A. Shutemov <kirill.shutemov@xxxxxxxxxxxxxxx> wrote:

> Most of things are in place and we can enable support of 5-level paging.
> The patch makes XEN_PV dependent on !X86_5LEVEL. XEN_PV is not ready to
> work with 5-level paging.

Please make a short comment about that in the Kconfig code as well, instead of a
silent, undocumented 'depends' clause.

> int
> + default 5 if X86_5LEVEL
> default 4 if X86_64
> default 3 if X86_PAE
> default 2
> @@ -1390,6 +1391,10 @@ config X86_PAE
> has the cost of more pagetable lookup overhead, and also
> consumes more pagetable space per process.
> +config X86_5LEVEL
> + bool "Enable 5-level page tables support"
> + depends on X86_64

So since users will be enabling it, this needs a proper help text that explains
what hardware supports it ("future Intel CPUs" will do if models are not public
yet), a short blurb about what it's good for - and a link to the Documentation/
file explaining it all.