Re: [Qemu-devel] [PATCH 1/2] Add dd-style SIGUSR1 progress reporting

[Paolo Bonzini]

On 04/28/2011 09:18 AM, Jes Sorensen wrote:
> On 04/27/11 18:14, Markus Armbruster wrote:
> > > +static void progress_simple_init(void)
> > > +{
> > > +    state.print = progress_simple_print;
> > > +    state.end = progress_simple_end;
> > > +}
> > > +
> > > +#ifdef CONFIG_POSIX
> > > +static void sigusr_print(int signal)
> > > +{
> > > +    printf("    (%3.2f/100%%)\n", state.current);
> >
> > printf() is not async-signal-safe.  I don't think you can safely call it
> > in a signal handler.
>
> Grrrr, you're absolutely right! Back to the drawing board!

Let's add our own version of strtol to QEMU. :)

Paolo