Commit 407c3ff6 authored by Hugh Dickins's avatar Hugh Dickins Committed by Greg Kroah-Hartman

kaiser: ENOMEM if kaiser_pagetable_walk() NULL


kaiser_add_user_map() took no notice when kaiser_pagetable_walk() failed.
And avoid its might_sleep() when atomic (though atomic at present unused).
Signed-off-by: default avatarHugh Dickins <hughd@google.com>
Acked-by: default avatarJiri Kosina <jkosina@suse.cz>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 20cbe9a3
...@@ -99,11 +99,11 @@ static pte_t *kaiser_pagetable_walk(unsigned long address, bool is_atomic) ...@@ -99,11 +99,11 @@ static pte_t *kaiser_pagetable_walk(unsigned long address, bool is_atomic)
pgd_t *pgd = native_get_shadow_pgd(pgd_offset_k(address)); pgd_t *pgd = native_get_shadow_pgd(pgd_offset_k(address));
gfp_t gfp = (GFP_KERNEL | __GFP_NOTRACK | __GFP_ZERO); gfp_t gfp = (GFP_KERNEL | __GFP_NOTRACK | __GFP_ZERO);
might_sleep();
if (is_atomic) { if (is_atomic) {
gfp &= ~GFP_KERNEL; gfp &= ~GFP_KERNEL;
gfp |= __GFP_HIGH | __GFP_ATOMIC; gfp |= __GFP_HIGH | __GFP_ATOMIC;
} } else
might_sleep();
if (pgd_none(*pgd)) { if (pgd_none(*pgd)) {
WARN_ONCE(1, "All shadow pgds should have been populated"); WARN_ONCE(1, "All shadow pgds should have been populated");
...@@ -160,13 +160,17 @@ int kaiser_add_user_map(const void *__start_addr, unsigned long size, ...@@ -160,13 +160,17 @@ int kaiser_add_user_map(const void *__start_addr, unsigned long size,
unsigned long end_addr = PAGE_ALIGN(start_addr + size); unsigned long end_addr = PAGE_ALIGN(start_addr + size);
unsigned long target_address; unsigned long target_address;
for (;address < end_addr; address += PAGE_SIZE) { for (; address < end_addr; address += PAGE_SIZE) {
target_address = get_pa_from_mapping(address); target_address = get_pa_from_mapping(address);
if (target_address == -1) { if (target_address == -1) {
ret = -EIO; ret = -EIO;
break; break;
} }
pte = kaiser_pagetable_walk(address, false); pte = kaiser_pagetable_walk(address, false);
if (!pte) {
ret = -ENOMEM;
break;
}
if (pte_none(*pte)) { if (pte_none(*pte)) {
set_pte(pte, __pte(flags | target_address)); set_pte(pte, __pte(flags | target_address));
} else { } else {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment