[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH] x86: Expose hypervisor's PVH support via xen_caps
On 05/23/2014 11:35 AM, Roger Pau Monné wrote: On 23/05/14 17:32, Boris Ostrovsky wrote:On 05/23/2014 11:20 AM, Andrew Cooper wrote:On 23/05/14 16:08, Boris Ostrovsky wrote:On 05/23/2014 11:00 AM, Andrew Cooper wrote:On 23/05/14 15:55, Boris Ostrovsky wrote:Signed-off-by: Boris Ostrovsky <boris.ostrovsky@xxxxxxxxxx> --- xen/arch/x86/setup.c | 5 +++++ 1 file changed, 5 insertions(+)If the plan is to try and PVH and HVM back into one mode as far as Xen is concerned, doesn't this become redundant?Yes, I was thinking about this but we currently don't have (or, rather, I can't think of) a good way to determine whether we can start a PVH guest. We can grep the log but that doesn't feel like a particularly good solution. One option could be to postpone this patch until 4.5 freezes and see whether we indeed followed up on the plan and if we didn't then integrate it. -borisMy concern here is that if this patch gets accepted, it will have to say forever more as the cap strings are a very public API.Yes, that's true. The problem that we have now is that if we have 'pvh=1' in the config file the guest will fail to start if PVH is not on. Can we, for example, revert (with a warning) to pure PV if that's the case?Won't this option go away once PVH is stable, so the toolstack can detect if the kernel supports PVH and start the guest in this mode by default? (of course falling back to PV if PVH is not supported). Yes, it will, by virtue of hypervisor never having to take this action (reverting to PV) since PVH would be stable and always supported. I just don't know whether ignoring pvh=1 directive on systems where PVH is not supported is acceptable. (Your "of course" seems to indicate that you think it is.) -boris _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |