* [Qemu-devel] [PATCH] Add dd-style SIGUSR1 progress reporting
@ 2011-04-12 9:34 Jes.Sorensen
2011-04-15 10:34 ` Kevin Wolf
2011-04-27 10:14 ` Kevin Wolf
0 siblings, 2 replies; 3+ messages in thread
From: Jes.Sorensen @ 2011-04-12 9:34 UTC (permalink / raw)
To: kwolf; +Cc: stefanha, qemu-devel
From: Jes Sorensen <Jes.Sorensen@redhat.com>
This introduces support for dd-style progress reporting, if the user
hasn't specified -p to report progress. If sent a SIGUSR1, qemu-img
will report current progress for commands that support progress
reporting.
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
---
qemu-progress.c | 49 ++++++++++++++++++++++++++++++++++++++++++++-----
1 files changed, 44 insertions(+), 5 deletions(-)
diff --git a/qemu-progress.c b/qemu-progress.c
index 656e065..6498161 100644
--- a/qemu-progress.c
+++ b/qemu-progress.c
@@ -26,12 +26,15 @@
#include "osdep.h"
#include "sysemu.h"
#include <stdio.h>
+#include <signal.h>
struct progress_state {
int enabled;
float current;
float last_print;
float min_skip;
+ void (*print)(void);
+ void (*end)(void);
};
static struct progress_state state;
@@ -51,20 +54,56 @@ static void progress_simple_print(void)
static void progress_simple_end(void)
{
- if (state.enabled) {
- printf("\n");
- }
+ printf("\n");
+}
+
+static void progress_simple_init(void)
+{
+ state.print = progress_simple_print;
+ state.end = progress_simple_end;
+}
+
+static void sigusr_print(int signal)
+{
+ printf(" (%3.2f/100%%)\n", state.current);
+}
+
+static void progress_dummy_print(void)
+{
+}
+
+static void progress_dummy_end(void)
+{
+}
+
+static void progress_dummy_init(void)
+{
+ struct sigaction action;
+
+ memset(&action, 0, sizeof(action));
+ sigfillset(&action.sa_mask);
+ action.sa_handler = sigusr_print;
+ action.sa_flags = 0;
+ sigaction(SIGUSR1, &action, NULL);
+
+ state.print = progress_dummy_print;
+ state.end = progress_dummy_end;
}
void qemu_progress_init(int enabled, float min_skip)
{
state.enabled = enabled;
state.min_skip = min_skip;
+ if (enabled) {
+ progress_simple_init();
+ } else {
+ progress_dummy_init();
+ }
}
void qemu_progress_end(void)
{
- progress_simple_end();
+ state.end();
}
void qemu_progress_print(float percent, int max)
@@ -84,6 +123,6 @@ void qemu_progress_print(float percent, int max)
if (current > (state.last_print + state.min_skip) ||
(current == 100) || (current == 0)) {
state.last_print = state.current;
- progress_simple_print();
+ state.print();
}
}
--
1.7.4.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [Qemu-devel] [PATCH] Add dd-style SIGUSR1 progress reporting
2011-04-12 9:34 [Qemu-devel] [PATCH] Add dd-style SIGUSR1 progress reporting Jes.Sorensen
@ 2011-04-15 10:34 ` Kevin Wolf
2011-04-27 10:14 ` Kevin Wolf
1 sibling, 0 replies; 3+ messages in thread
From: Kevin Wolf @ 2011-04-15 10:34 UTC (permalink / raw)
To: Jes.Sorensen; +Cc: stefanha, qemu-devel
Am 12.04.2011 11:34, schrieb Jes.Sorensen@redhat.com:
> From: Jes Sorensen <Jes.Sorensen@redhat.com>
>
> This introduces support for dd-style progress reporting, if the user
> hasn't specified -p to report progress. If sent a SIGUSR1, qemu-img
> will report current progress for commands that support progress
> reporting.
>
> Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Thanks, applied to the block branch.
Kevin
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Qemu-devel] [PATCH] Add dd-style SIGUSR1 progress reporting
2011-04-12 9:34 [Qemu-devel] [PATCH] Add dd-style SIGUSR1 progress reporting Jes.Sorensen
2011-04-15 10:34 ` Kevin Wolf
@ 2011-04-27 10:14 ` Kevin Wolf
1 sibling, 0 replies; 3+ messages in thread
From: Kevin Wolf @ 2011-04-27 10:14 UTC (permalink / raw)
To: Jes.Sorensen; +Cc: stefanha, qemu-devel
Am 12.04.2011 11:34, schrieb Jes.Sorensen@redhat.com:
> From: Jes Sorensen <Jes.Sorensen@redhat.com>
>
> This introduces support for dd-style progress reporting, if the user
> hasn't specified -p to report progress. If sent a SIGUSR1, qemu-img
> will report current progress for commands that support progress
> reporting.
>
> Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
This one seems to break the mingw32 build:
qemu-progress.c: In function 'progress_dummy_init':
qemu-progress.c:78: error: storage size of 'action' isn't known
qemu-progress.c:81: warning: implicit declaration of function 'sigfillset'
qemu-progress.c:81: warning: nested extern declaration of 'sigfillset'
qemu-progress.c:84: warning: implicit declaration of function 'sigaction'
qemu-progress.c:84: warning: nested extern declaration of 'sigaction'
qemu-progress.c:84: error: 'SIGUSR1' undeclared (first use in this function)
qemu-progress.c:84: error: (Each undeclared identifier is reported only once
qemu-progress.c:84: error: for each function it appears in.)
qemu-progress.c:78: warning: unused variable 'action'
Kevin
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2011-04-27 10:11 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-04-12 9:34 [Qemu-devel] [PATCH] Add dd-style SIGUSR1 progress reporting Jes.Sorensen
2011-04-15 10:34 ` Kevin Wolf
2011-04-27 10:14 ` Kevin Wolf
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).