From mboxrd@z Thu Jan 1 00:00:00 1970 From: Yang Hongyang Subject: Re: [PATCH 1/2]xl: Add "xl uptime" command Date: Thu, 13 May 2010 18:10:58 +0800 Message-ID: <4BEBD032.5080408@cn.fujitsu.com> References: <4BEBC81A.70109@cn.fujitsu.com> <4BEBCC61.3000703@eu.citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: <4BEBCC61.3000703@eu.citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Vincent Hanquez Cc: "xen-devel@lists.xensource.com" List-Id: xen-devel@lists.xenproject.org Hi Vincent, On 05/13/2010 05:54 PM, Vincent Hanquez wrote: > On 13/05/10 10:36, Yang Hongyang wrote: >> Add "xl uptime" command, a clone of "xm uptime". >> >> Signed-off-by: Yang Hongyang >> >> diff -r d77a88f938c6 tools/libxl/libxl.c >> --- a/tools/libxl/libxl.c Tue May 11 14:05:28 2010 +0100 >> +++ b/tools/libxl/libxl.c Fri May 14 01:25:56 2010 +0800 >> @@ -2629,3 +2629,23 @@ >> >> return rc; >> } >> + >> +unsigned long libxl_get_start_time(struct libxl_ctx *ctx, uint32_t >> domid) >> +{ >> + char *dompath =3D libxl_xs_get_dompath(ctx, domid); >> + unsigned long s_time =3D 0; >> + char *start_time =3D NULL; >> + char *vm_path =3D NULL; >> + >> + vm_path =3D libxl_xs_read(ctx, XBT_NULL, libxl_sprintf(ctx, >> "%s/vm", dompath)); >> + start_time =3D libxl_xs_read(ctx, XBT_NULL, libxl_sprintf(ctx, >> "%s/start_time", vm_path)); >> + if (start_time =3D=3D NULL) { >> + XL_LOG_ERRNOVAL(ctx, XL_LOG_ERROR, -1, >> + "Can't get start time of domain '%d'", domid); >> + return -1; >> + } >> + s_time =3D strtoul(start_time, NULL, 10); >> + >> + return s_time; >> +} >=20 > this function should more than likely return a uint64_t or a uint32_t, > but certainly not an unsigned long. and since it apply to a domain it > should be call libxl_domain_get_start_time. Thank you for your review,I'll fix that. >=20 > I find this command basically useless, since on restore the "start_time= " > is not kept. do we actually need this ? >=20 I think it's sometimes useful. I noticed that just the day before yesterd= ay, somebody asked in the xen-api list about "uptime" command. -------- Original Message -------- Subject: [Xen-API] Want to find out the uptime of individual VM running on Citrix Xen Date: Tue, 11 May 2010 04:00:13 -0600 From: Sane, Abhijeet To: xen-api@lists.xensource.com Hi, I want to know if it is possible to get the uptime of individual VM that are running on the Citrix host using the Citrix Xen python API.. Something on the lines of =E2=80=98xm uptime=E2=80=99 =20 Regards Abhijeet Sane --=20 Regards Yang Hongyang