Re: [Qemu-devel] [PATCH v2 0/3] char: fix chardev aliases regression

[Paolo Bonzini <pbonzini@redhat.com>]

On 12/06/2017 21:01, Marc-André Lureau wrote:
> Hi Paolo
> 
> On Mon, Jun 12, 2017 at 8:06 PM Paolo Bonzini <pbonzini@redhat.com
> <mailto:pbonzini@redhat.com>> wrote:
> 
> 
> 
>     On 08/06/2017 13:59, Marc-André Lureau wrote:
>     > Hi,
>     >
>     > The patch "char: move CharBackend handling in char-fe unit" broke
>     > chardev aliases. Here is a small series to fix it, and add a simple
>     > unit test to check the alias keep working.
>     >
>     > v2:
>     > - move HAVE_CHARDEV_* in osdep.h (Markus Armbruster)
>     > - add r-b tags from Eric for the patches that didn't change
>     >
>     > Marc-André Lureau (3):
>     >   char: fix alias devices regression
>     >   chardev: don't use alias names in parse_compat()
>     >   test-char: start a /char/serial test
>     >
>     >  include/chardev/char-parallel.h |  5 -----
>     >  include/chardev/char-serial.h   |  8 --------
>     >  include/qemu/osdep.h            | 13 +++++++++++++
>     >  chardev/char.c                  |  4 ++--
>     >  tests/test-char.c               | 30 ++++++++++++++++++++++++++++++
>     >  5 files changed, 45 insertions(+), 15 deletions(-)
>     >
> 
>     Queued, thanks.
> 
> 
> I also sent a pull request with those patches "[PULL0/3] Charpatches" a
> few days ago. I should take the habit to notice it on the PATCH series.

No problem, I would have noticed sooner or later. :)

Paolo