• Stefan Bader's avatar
    xen: p2m: correctly initialize partial p2m leaf · 8e1b4cf2
    Stefan Bader authored
    After changing the p2m mapping to a tree by
    
      commit 58e05027
        xen: convert p2m to a 3 level tree
    
    and trying to boot a DomU with 615MB of memory, the following crash was
    observed in the dump:
    
    kernel direct mapping tables up to 26f00000 @ 1ec4000-1fff000
    BUG: unable to handle kernel NULL pointer dereference at (null)
    IP: [<c0107397>] xen_set_pte+0x27/0x60
    *pdpt = 0000000000000000 *pde = 0000000000000000
    
    Adding further debug statements showed that when trying to set up
    pfn=0x26700 the returned mapping was invalid.
    
    pfn=0x266ff calling set_pte(0xc1fe77f8, 0x6b3003)
    pfn=0x26700 calling set_pte(0xc1fe7800, 0x3)
    
    Although the last_pfn obtained from the startup info is 0x26700, which
    should in turn not be hit, the additional 8MB which are added as extra
    memory normally seem to be ok. This lead to looking into the initial
    p2m tree construction, which uses the smaller value and assuming that
    there is other code handling the extra memory.
    
    When the p2m tree is set up, the leaves are directly pointed to the
    array which the domain builder set up. But if the mapping is not on a
    boundary that fits into one p2m page, this will result in the last leaf
    being only partially valid. And as the invalid entries are not
    initialized in that case, things go badly wrong.
    
    I am trying to fix that by checking whether the current leaf is a
    complete map and if not, allocate a completely new page and copy only
    the valid pointers there. This may not be the most efficient or elegant
    solution, but at least it seems to allow me booting DomUs with memory
    assignments all over the range.
    
    BugLink: http://bugs.launchpad.net/bugs/686692
    [v2: Redid a bit of commit wording and fixed a compile warning]
    Signed-off-by: default avatarStefan Bader <stefan.bader@canonical.com>
    Signed-off-by: default avatarKonrad Rzeszutek Wilk <konrad.wilk@oracle.com>
    8e1b4cf2
p2m.c 13.2 KB