x86/pti/efi: broken conversion from efi to kernel page table
The page table order must be increased for EFI table in order to avoid a bug where NMI tries to change the page table to kernel page table, while efi page table is active. For more disccussion about this bug, see this thread: http://lkml.iu.edu/hypermail/linux/kernel/1801.1/00951.html Signed-off-by: Pavel Tatashin <pasha.tatashin@oracle.com> Reviewed-by: Steven Sistare <steven.sistare@oracle.com> Acked-by: Jiri Kosina <jkosina@suse.cz> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
92e8f20494
commit
1b92c48a2e
3 changed files with 12 additions and 8 deletions
|
@ -27,6 +27,17 @@ static inline void paravirt_release_pud(unsigned long pfn) {}
|
||||||
*/
|
*/
|
||||||
extern gfp_t __userpte_alloc_gfp;
|
extern gfp_t __userpte_alloc_gfp;
|
||||||
|
|
||||||
|
#ifdef CONFIG_PAGE_TABLE_ISOLATION
|
||||||
|
/*
|
||||||
|
* Instead of one PGD, we acquire two PGDs. Being order-1, it is
|
||||||
|
* both 8k in size and 8k-aligned. That lets us just flip bit 12
|
||||||
|
* in a pointer to swap between the two 4k halves.
|
||||||
|
*/
|
||||||
|
#define PGD_ALLOCATION_ORDER 1
|
||||||
|
#else
|
||||||
|
#define PGD_ALLOCATION_ORDER 0
|
||||||
|
#endif
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* Allocate and free page tables.
|
* Allocate and free page tables.
|
||||||
*/
|
*/
|
||||||
|
|
|
@ -345,13 +345,6 @@ static inline void _pgd_free(pgd_t *pgd)
|
||||||
}
|
}
|
||||||
#else
|
#else
|
||||||
|
|
||||||
/*
|
|
||||||
* Instead of one pgd, Kaiser acquires two pgds. Being order-1, it is
|
|
||||||
* both 8k in size and 8k-aligned. That lets us just flip bit 12
|
|
||||||
* in a pointer to swap between the two 4k halves.
|
|
||||||
*/
|
|
||||||
#define PGD_ALLOCATION_ORDER kaiser_enabled
|
|
||||||
|
|
||||||
static inline pgd_t *_pgd_alloc(void)
|
static inline pgd_t *_pgd_alloc(void)
|
||||||
{
|
{
|
||||||
return (pgd_t *)__get_free_pages(PGALLOC_GFP, PGD_ALLOCATION_ORDER);
|
return (pgd_t *)__get_free_pages(PGALLOC_GFP, PGD_ALLOCATION_ORDER);
|
||||||
|
|
|
@ -142,7 +142,7 @@ int __init efi_alloc_page_tables(void)
|
||||||
return 0;
|
return 0;
|
||||||
|
|
||||||
gfp_mask = GFP_KERNEL | __GFP_NOTRACK | __GFP_ZERO;
|
gfp_mask = GFP_KERNEL | __GFP_NOTRACK | __GFP_ZERO;
|
||||||
efi_pgd = (pgd_t *)__get_free_page(gfp_mask);
|
efi_pgd = (pgd_t *)__get_free_pages(gfp_mask, PGD_ALLOCATION_ORDER);
|
||||||
if (!efi_pgd)
|
if (!efi_pgd)
|
||||||
return -ENOMEM;
|
return -ENOMEM;
|
||||||
|
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue