[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 0/3] char: fix chardev aliases regression
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH v2 0/3] char: fix chardev aliases regression |
Date: |
Tue, 13 Jun 2017 12:05:25 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.0 |
On 12/06/2017 21:01, Marc-André Lureau wrote:
> Hi Paolo
>
> On Mon, Jun 12, 2017 at 8:06 PM Paolo Bonzini <address@hidden
> <mailto:address@hidden>> 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