Am Samstag, 1. Februar 2014, 19:00:01 schrieb Peter Maydell:
> On 20 January 2014 19:34, Michael Walle <mich...@walle.cc> wrote:
> > Do not use qemu_log().
> >
> > Signed-off-by: Michael Walle <mich...@walle.cc>
> > ---
> >
> > hw/misc/lm32_sys.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/hw/misc/lm32_sys.c b/hw/misc/lm32_sys.c
> > index 8176cdb..6af0cca 100644
> > --- a/hw/misc/lm32_sys.c
> > +++ b/hw/misc/lm32_sys.c
> > @@ -80,7 +80,7 @@ static void sys_write(void *opaque, hwaddr addr,
> >
> > case R_PASSFAIL:
> > s->regs[addr] = value;
> > testname = (char *)s->testname;
> >
> > - qemu_log("TC %-32s %s\n", testname, (value) ? "FAILED" : "OK");
> > + fprintf(stderr, "TC %-32s %s\n", testname, (value) ? "FAILED" :
> > "OK");
> >
> > break;
>
> This looks wrong to me -- devices shouldn't print to stderr, ideally.
lm32_sys is actually no real device. it is just used for unit testing.
-michael
