Re: RFC x86/boot/64: BOOT_PGT_SIZE definition for compressed kernel

2020-10-27 Thread Kirill A. Shutemov
On Tue, Oct 27, 2020 at 09:16:17AM -0400, Arvind Sankar wrote: > The #PF handler is already there now with the SEV-ES series, but I agree > it would still complicate things. It's simpler to just increase > BOOT_PGT_SIZE and make it unconditional (i.e. bump it to say 32 or 64 > even if !KASLR).

Re: RFC x86/boot/64: BOOT_PGT_SIZE definition for compressed kernel

2020-10-27 Thread Joerg Roedel
On Tue, Oct 27, 2020 at 03:40:07PM +0300, Kirill A. Shutemov wrote: > BTW, what happens if we underestimate BOOT_PGT_SIZE? Do we overwrite > something? The boot code will print an error and stop the machine when allocating a page-table page fails. I also think that bumping BOOT_PGT_SIZE up to

Re: RFC x86/boot/64: BOOT_PGT_SIZE definition for compressed kernel

2020-10-27 Thread Arvind Sankar
On Tue, Oct 27, 2020 at 03:40:07PM +0300, Kirill A. Shutemov wrote: > On Sat, Oct 24, 2020 at 08:41:58PM -0400, Arvind Sankar wrote: > > Hi, I think the definition of BOOT_PGT_SIZE in > > arch/x86/include/asm/boot.h is insufficient, especially after > > ca0e22d4f011 ("x86/boot/compressed/64:

Re: RFC x86/boot/64: BOOT_PGT_SIZE definition for compressed kernel

2020-10-27 Thread Kirill A. Shutemov
On Sat, Oct 24, 2020 at 08:41:58PM -0400, Arvind Sankar wrote: > Hi, I think the definition of BOOT_PGT_SIZE in > arch/x86/include/asm/boot.h is insufficient, especially after > ca0e22d4f011 ("x86/boot/compressed/64: Always switch to own page table") > > Currently, it allocates 6 pages if KASLR

Re: RFC x86/boot/64: BOOT_PGT_SIZE definition for compressed kernel

2020-10-24 Thread Arvind Sankar
On Sat, Oct 24, 2020 at 08:41:58PM -0400, Arvind Sankar wrote: > Hi, I think the definition of BOOT_PGT_SIZE in > arch/x86/include/asm/boot.h is insufficient, especially after > ca0e22d4f011 ("x86/boot/compressed/64: Always switch to own page table") > > Currently, it allocates 6 pages if KASLR

RFC x86/boot/64: BOOT_PGT_SIZE definition for compressed kernel

2020-10-24 Thread Arvind Sankar
Hi, I think the definition of BOOT_PGT_SIZE in arch/x86/include/asm/boot.h is insufficient, especially after ca0e22d4f011 ("x86/boot/compressed/64: Always switch to own page table") Currently, it allocates 6 pages if KASLR is disabled, and either 17 or 19 pages depending on X86_VERBOSE_BOOTUP