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

Re: [Xen-devel] [PATCH 2/18 V2]: PVH xen: add XENMEM_add_to_physmap_range



>>> On 19.03.13 at 14:40, Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx> wrote:
> There were some patches in the series (I can't recall the names of them
> thought) that had some of the PV hypercalls return -EINVAL (or -ENOSYS
> as I suggested).
> 
> I think it might be better to introduce one patch that would neuter
> the PV/HVM hypercalls that we don't want to support or can't yet
> (for example b/c we haven't fixed the bugs).
> 
> Are you OK if those paths are done in the common code? Meaning say
> for the PHYSDEV_op hypercalls some return -ENOSYS?

If intended to be permanent, then of course.

If intended to be temporary (until fixed), then yes as long as an
accompanying comment makes this clear (and greppable for).

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

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