From mboxrd@z Thu Jan  1 00:00:00 1970
From: Jim Fehlig <jfehlig@suse.com>
Subject: Re: [PATCH RFC OSSTEST 7/9] Toolstack: Abstract away
 migration support check.
Date: Mon, 23 Jun 2014 15:28:35 -0600
Message-ID: <53A89C03.7030002@suse.com>
References: <1403018758.25074.5.camel@kazak.uk.xensource.com>	
	<1403018809-25509-7-git-send-email-ian.campbell@citrix.com>
	<1403019217.25074.10.camel@kazak.uk.xensource.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <1403019217.25074.10.camel@kazak.uk.xensource.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Ian Campbell <Ian.Campbell@citrix.com>
Cc: ian.jackson@eu.citrix.com, xen-devel@lists.xen.org
List-Id: xen-devel@lists.xenproject.org

Ian Campbell wrote:
> On Tue, 2014-06-17 at 16:26 +0100, Ian Campbell wrote:
>   
>> Not implemented for libvirt (the check that is)
>>     
>
> Jim,
>
> What would be the right thing to do for this? The intention is for
> osstest to skip migration tests when that feature is unimplemented by
> the toolstack/xen/arch combination.
>
> It looks like the output of "virsh capabilities" indicates migration:
>     <migration_features>
>       <live/>
>       <uri_transports>
>         <uri_transport>tcp</uri_transport>
>       </uri_transports>
>     </migration_features>
> is that the right place to look?
>   

Yes.

> Will this capability be automagically suppressed on ARM until we
> implement it?

Unfortunately, no :-/.  Currently, the libvirt libxl driver will include
<migration_features> and call save, restore, and migrate regardless of arch.

>  FYI xl uses LIBXL_HAVE_NO_SUSPEND_RESUME to figure this
> out...
>   

Ah, interesting.  Thanks for the tip.  I'll need to add similar logic in
the libxl driver.

Regards,
Jim