On 12/06/2017 21:01, Marc-André Lureau wrote: > Hi Paolo > > On Mon, Jun 12, 2017 at 8:06 PM Paolo Bonzini <pbonz...@redhat.com > <mailto:pbonz...@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
