[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH] ocaml/xc: correct shutdown_reason enumeration
On Thu, 2015-11-05 at 11:56 +0000, David Scott wrote: > > On 5 Nov 2015, at 11:39, Simon Rowe <simon.rowe@xxxxxxxxxxxxx> wrote: > > > > As defined by the Xen public header the fifth value of > > shutdown_reason is watchdog. > > Iâve always been a bit suspicious about having both âPoweroffâ and âHaltâ > there. Perhaps there was some confusion between what could be written to > âcontrol/shutdownâ in xenstore and legal arguments to > `xc_domain_shutdown` and `SCHEDOP_shutdown`? > > Anyway youâre clearly right, `Watchdog` is the 5th value. So I think this > is fine. > > Acked-by: David Scott <dave@xxxxxxxxxx> Applied. > > I happen to notice thereâs a type with the same name in âxenopsdâ[1], so > Iâve cc:d xen-api@lists as a heads-up. > > Thanks, > Dave > > [1] https://github.com/xapi-project/xenopsd/blob/7818ab896d9969c5f5462a2f > 0d0ae62703b104b6/xc/domain.ml#L268 > > > > > Signed-off-by: Simon Rowe <simon.rowe@xxxxxxxxxxxxx> > > --- > > tools/ocaml/libs/xc/xenctrl.mlÂÂ|ÂÂÂÂ2 +- > > tools/ocaml/libs/xc/xenctrl.mli |ÂÂÂÂ2 +- > > 2 files changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/tools/ocaml/libs/xc/xenctrl.ml > > b/tools/ocaml/libs/xc/xenctrl.ml > > index b7ba8b7..beb95b8 100644 > > --- a/tools/ocaml/libs/xc/xenctrl.ml > > +++ b/tools/ocaml/libs/xc/xenctrl.ml > > @@ -89,7 +89,7 @@ type compile_info = > > compile_date : string; > > } > > > > -type shutdown_reason = Poweroff | Reboot | Suspend | Crash | Halt > > +type shutdown_reason = Poweroff | Reboot | Suspend | Crash | Watchdog > > > > type domain_create_flag = CDF_HVM | CDF_HAP > > > > diff --git a/tools/ocaml/libs/xc/xenctrl.mli > > b/tools/ocaml/libs/xc/xenctrl.mli > > index bc4af56..8928a2e 100644 > > --- a/tools/ocaml/libs/xc/xenctrl.mli > > +++ b/tools/ocaml/libs/xc/xenctrl.mli > > @@ -61,7 +61,7 @@ type compile_info = { > > Â compile_domain : string; > > Â compile_date : string; > > } > > -type shutdown_reason = Poweroff | Reboot | Suspend | Crash | Halt > > +type shutdown_reason = Poweroff | Reboot | Suspend | Crash | Watchdog > > > > type domain_create_flag = CDF_HVM | CDF_HAP > > _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |