[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 5/8] viridian: use viridian_map/unmap_guest_page() for reference tsc page



On Mon, Jan 07, 2019 at 03:35:41PM +0000, Paul Durrant wrote:
[...]
> > > >                                const struct viridian_page *vp);
> > > > -void viridian_map_guest_page(struct vcpu *v, struct viridian_page
> > *vp);
> > > > +void viridian_map_guest_page(struct domain *d, struct viridian_page
> > > *vp);
> > >
> > > Since you modify this anyway, can you constify struct domain?
> > >
> > 
> > Yes, I guess that should be do-able now that the viridian_page is in a
> > separate allocation unit.
> 
> Alas, no. The call to get_page_from_gfn() takes a non-const struct domain 
> pointer, and I'm not opening that can of worms. Does your R-b still stand?

Yes.

Wei.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.