Re: [PATCH] libxl: support domainReset

[Jim Fehlig <jfehlig@suse.com>]

Ian Campbell wrote:
> On Tue, 2014-08-05 at 08:06 -0600, Jim Fehlig wrote:
>   
>> Ian Campbell wrote:
>>     
>>> On Tue, 2014-08-05 at 10:55 +0200, Olaf Hering wrote:
>>>   
>>>       
>>>> The original report was that 'Reset' does not work from GUI, like
>>>> virt-manager or virsh. I think the expected outcome is like pushing the
>>>> reset button on a physical board. Xen doesnt do it that way, no idea
>>>> about others.
>>>>     
>>>>         
>>> Sounds like you want libxl_domain_reboot then, perhaps with a fallback
>>> on ERROR_NOPARAVIRT for an HVM guest to sending a trigger.
>>>   
>>>       
>> Hrm, I don't think that's right .  It should be a hard reset
>>
>> http://libvirt.org/html/libvirt-libvirt.html#virDomainReset
>>
>> destroy/start seems the correct way to implement this.
>>     
>
> Yes, given that requirement it is. Sorry for the noise.
>
> Would some sort of hard reset API be useful in libxl?
>   

Sure.  I think having an API that emulates a power reset button would be
a nice addition to libxl's domain operations.  The destroy/start
approach incurs a small bit of overhead, which would be avoided with
such an API.  Clients (perhaps incorrectly) implementing their own
notion of reset  would also be avoided.

In the absence of libxl_domain_reset(), do folks think the destroy/start
approach is acceptable?  As Olaf mentioned, it allows "Force Reset" to
work via virt-manager.

Regards,
Jim