All of lore.kernel.org
 help / color / mirror / Atom feed
* xentop.c trivial fix
@ 2007-01-07 20:43 Peter
  0 siblings, 0 replies; only message in thread
From: Peter @ 2007-01-07 20:43 UTC (permalink / raw)
  To: xen-devel

Hi.

I run:
xentop --delay=30 --iterations 2 --batch

I get output, 30 second delay, output, 30 second delay, exit.

I want output, 30 seconds delay, output, exit.

The following patch implements that behavior and I think this is likely
the desired behavior anyway.  i.e. just a trivial bug in code.

# diff -U5 xentop.c.orig xentop.c
--- xentop.c.orig       2006-12-29 08:05:13.000000000 +0000
+++ xentop.c    2006-12-29 08:06:40.000000000 +0000
@@ -1053,13 +1053,13 @@
         } else {
                         do {
                                 gettimeofday(&curtime, NULL);
                                 top();
                                 oldtime = curtime;
-                               sleep(delay);
                                 if ((!loop) && !(--iterations))
                                         break;
+                               sleep(delay);
                         } while (1);
         }

         /* Cleanup occurs in cleanup(), so no work to do here. */

Regards, Peter

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2007-01-07 20:43 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-01-07 20:43 xentop.c trivial fix Peter

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.