[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 20/20] chardev: qom-ify
From: |
Marc-André Lureau |
Subject: |
Re: [Qemu-devel] [PATCH v2 20/20] chardev: qom-ify |
Date: |
Thu, 12 Jan 2017 10:19:03 -0500 (EST) |
Hi
----- Original Message -----
> On 01/10/2017 11:48 AM, Marc-André Lureau wrote:
> > Turn Chardev into Object.
> >
> > qemu_chr_alloc() is replaced by the qemu_chardev_new() constructor. It
> > will call qemu_char_open() to open/intialize the chardev with the
> > ChardevCommon *backend settings.
> >
>
> > -ChardevReturn *qmp_chardev_add(const char *id, ChardevBackend *backend,
> > - Error **errp)
> > +static const ChardevClass *char_get_class(const char *driver, Error
> > **errp)
> > +{
>
> Your followup "[PATCH 16/40] char: get rid of CharDriver" hoists this up
> earlier in the file. Can you minimize churn by just declaring it earlier
> to begin with?
Unless Paolo want me to respin the series, I think we will have to do with it ;)
thanks
- [Qemu-devel] [PATCH v2 14/20] char: rename TCPChardev and NetChardev, (continued)
- [Qemu-devel] [PATCH v2 14/20] char: rename TCPChardev and NetChardev, Marc-André Lureau, 2017/01/10
- [Qemu-devel] [PATCH v2 15/20] spice-char: improve error reporting, Marc-André Lureau, 2017/01/10
- [Qemu-devel] [PATCH v2 11/20] char: allocate CharDriverState as a single object, Marc-André Lureau, 2017/01/10
- [Qemu-devel] [PATCH v2 18/20] baum: use a common prefix for chr callbacks, Marc-André Lureau, 2017/01/10
- [Qemu-devel] [PATCH v2 17/20] gtk: overwrite the console.c char driver, Marc-André Lureau, 2017/01/10
- [Qemu-devel] [PATCH v2 16/20] char: use error_report(), Marc-André Lureau, 2017/01/10
- [Qemu-devel] [PATCH v2 19/20] vc: use a common prefix for chr callbacks, Marc-André Lureau, 2017/01/10
- [Qemu-devel] [PATCH v2 13/20] char: rename CharDriverState Chardev, Marc-André Lureau, 2017/01/10
- [Qemu-devel] [PATCH v2 20/20] chardev: qom-ify, Marc-André Lureau, 2017/01/10
- Re: [Qemu-devel] [PATCH v2 00/20] chardev: qom-ify, Paolo Bonzini, 2017/01/24