Re: [V8 PATCH 3/8] pvh dom0: Introduce p2m_map_foreign

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

On Thu, 27 Mar 2014 12:29:03 +0000
George Dunlap <george.dunlap@eu.citrix.com> wrote:

> On 03/22/2014 01:39 AM, Mukesh Rathor wrote:
> > @@ -751,16 +750,31 @@ set_mmio_p2m_entry(struct domain *d, unsigned
> > long gfn, mfn_t mfn) set_gpfn_from_mfn(mfn_x(omfn),
> > INVALID_M2P_ENTRY); }
.......
> > -    P2M_DEBUG("set mmio %lx %lx\n", gfn, mfn_x(mfn));
> > -    rc = set_p2m_entry(p2m, gfn, mfn, PAGE_ORDER_4K,
> > p2m_mmio_direct, p2m->default_access);
> > +    P2M_DEBUG("set %d %lx %lx\n", gfn_p2mt, gfn, mfn_x(mfn));
> > +    rc = set_p2m_entry(p2m, gfn, mfn, PAGE_ORDER_4K, gfn_p2mt,
> > +                       p2m->default_access);
> >       gfn_unlock(p2m, gfn, 0);
> >       if ( 0 == rc )
> >           gdprintk(XENLOG_ERR,
> > -            "set_mmio_p2m_entry: set_p2m_entry failed!
> > mfn=%08lx\n",
> > +            "%s: set_p2m_entry failed! mfn=%08lx\n", __func__,
> >               mfn_x(get_gfn_query_unlocked(p2m->domain, gfn, &ot)));
> >       return rc;
> >   }
> >   
> > +/* Set foreign mfn in the given guest's p2m table.
> > + * Returns: True for success. */
> > +static int __attribute__((unused))
> > +set_foreign_p2m_entry(struct domain *d, unsigned long gfn, mfn_t
> > mfn)
> 
> Why "unused"?  It appears that tag remains even at the end of the
> series.

Ooops, meant to undo "unused" in patch 5 where its being called. I
am surprised the compiler didn't complain on that.... 

anyways, will fix it to undo in patch 5 so patch 3 can compile.

thanks
mukesh