[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] Add chardev-send-break monitor command
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH] Add chardev-send-break monitor command |
Date: |
Wed, 7 Jun 2017 03:10:12 -0400 (EDT) |
----- Original Message -----
> From: "Markus Armbruster" <address@hidden>
> To: "Paolo Bonzini" <address@hidden>
> Cc: "Stefan Fritsch" <address@hidden>, address@hidden, "Dr. David Alan
> Gilbert" <address@hidden>,
> "Marc-André Lureau" <address@hidden>
> Sent: Wednesday, June 7, 2017 9:06:53 AM
> Subject: Re: [Qemu-devel] [PATCH] Add chardev-send-break monitor command
>
> Paolo Bonzini <address@hidden> writes:
>
> >> >> Is there an obvious test that we can enhance to add coverage of the new
> >> >> QMP command?
> >> >
> >> > You could have a new test covering hw/char/serial.c, but I wouldn't let
> >> > that hold the patch.
> >>
> >> Holding patches is pretty much the only leverage I have to get tests for
> >> new stuff :)
> >>
> >> Asking for tests that cover all of serial.c wouldn't be fair. But I am
> >> asking for basic test coverage of new QMP commands.
> >
> > I agree, on the other hand it's not exactly a comparable area. I am
> > planning
> > to write a serial qtest for migration as well (which is more complex than
> > this
> > QMP command) so I might as well write the test for this new QMP command
> > myself to get my feet wet...
>
> I'm willing to take a committment from someone I trust in lieu of actual
> tests. Is this one?
Sure, though it looks like Stefan also wrote actual tests so we might get
two birds with a stone.
Paolo
Re: [Qemu-devel] [PATCH] Add chardev-send-break monitor command, Dr. David Alan Gilbert, 2017/06/08