Re: [PATCH 17/17] PVH xen: PVH dom0 creation....

[Mukesh Rathor <mukesh.rathor@oracle.com>]

On Wed, 24 Apr 2013 10:28:35 +0100
"Jan Beulich" <JBeulich@suse.com> wrote:

> >>> On 23.04.13 at 23:26, Mukesh Rathor <mukesh.rathor@oracle.com>
> >>> wrote:
> > +    /* If the e820 ended under 4GB, we must map the remaining
> > space upto 4GB */
> > +    if ( end < GB(4) )
> > +    {
> > +        start_pfn = PFN_UP(end);
> > +        end_pfn = (GB(4)) >> PAGE_SHIFT;
> > +        nump = end_pfn - start_pfn;
> > +        rc = domctl_memory_mapping(d, start_pfn, start_pfn, nump,
> > 1);
> > +        BUG_ON(rc);
> > +    }
> 
> That's necessary, but not sufficient. Or did I overlook MMIO ranges
> getting added somewhere else for Dom0, when they sit above the
> highest E820 covered address?

construct_dom0() adds the entire range:

    /* DOM0 is permitted full I/O capabilities. */
    rc |= ioports_permit_access(dom0, 0, 0xFFFF);
    rc |= iomem_permit_access(dom0, 0UL, ~0UL);

Mukesh