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

Re: [libvirt] [Xen-devel] [PATCH] libxl: support domainReset



On Tue, 2014-08-05 at 09:10 -0600, Jim Fehlig wrote:
> 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.

I think this ought to become pretty easy once Wei's patches to record
the guest cfg in libxl are completed. Wei -- what do you think?

> In the absence of libxl_domain_reset(), do folks think the destroy/start
> approach is acceptable?

FWIW I believe it is.

>   As Olaf mentioned, it allows "Force Reset" to
> work via virt-manager.

Ian.


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