diff options
author | Keir Fraser <keir.fraser@citrix.com> | 2008-10-13 13:15:20 +0100 |
---|---|---|
committer | Keir Fraser <keir.fraser@citrix.com> | 2008-10-13 13:15:20 +0100 |
commit | b525c05cf95fa9528cfb83ecc120087881013980 (patch) | |
tree | 61ac2b29d4e0ea0132bf7474849d02ef1a2ec7fc | |
parent | 4f6b83015e2304f3e7560ef33d04de6131df3b0c (diff) | |
download | xen-b525c05cf95fa9528cfb83ecc120087881013980.tar.gz xen-b525c05cf95fa9528cfb83ecc120087881013980.tar.bz2 xen-b525c05cf95fa9528cfb83ecc120087881013980.zip |
x86: propagate return value of alloc_l1_table()
A blatant mistake of mine resulted in the return value of
alloc_l1_table() to be ignored with the preemptable page table update
changes.
Signed-off-by: Jan Beulich <jbeulich@novell.com>
-rw-r--r-- | xen/arch/x86/mm.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/xen/arch/x86/mm.c b/xen/arch/x86/mm.c index f3082eeedd..4c8b40a901 100644 --- a/xen/arch/x86/mm.c +++ b/xen/arch/x86/mm.c @@ -1883,8 +1883,7 @@ static int alloc_page_type(struct page_info *page, unsigned long type, switch ( type & PGT_type_mask ) { case PGT_l1_page_table: - alloc_l1_table(page); - rc = 0; + rc = alloc_l1_table(page); break; case PGT_l2_page_table: rc = alloc_l2_table(page, type, preemptible); |