x86: propagate return value of alloc_l1_table()
authorKeir Fraser <keir.fraser@citrix.com>
Mon, 13 Oct 2008 12:15:20 +0000 (13:15 +0100)
committerKeir Fraser <keir.fraser@citrix.com>
Mon, 13 Oct 2008 12:15:20 +0000 (13:15 +0100)
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>
xen/arch/x86/mm.c

index f3082eeedd64fc246acc836fef49503e02f3ef1f..4c8b40a9011c9c751cd6f517c0f981f94d2d7fd9 100644 (file)
@@ -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);