[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] chardev: add hotplug support.
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH v2] chardev: add hotplug support. |
Date: |
Fri, 12 Oct 2012 19:08:21 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:16.0) Gecko/20121009 Thunderbird/16.0 |
Il 12/10/2012 18:54, Paolo Bonzini ha scritto:
> Il 12/10/2012 14:39, Gerd Hoffmann ha scritto:
>> + chr = qemu_chr_new_from_opts(opts, NULL);
>> + if (chr == NULL) {
>> + qemu_opts_del(opts);
>> + error_setg(&err, "Creating chardev failed\n");
>> + goto exit_err;
>> + }
>> + return 0;
>> +
>
> Since you have to do a v3, please make this part (which is common
> between HMP and QMP) a separate function.
Also please make qemu_chr_new_from_opts use an Error ** at least for the
error that are reported from that function itself (no need to add
propagation to all of qemu-char.c, and it would conflict badly with the
series that does so for qemu-sockets.c).
Paolo
- [Qemu-devel] [PATCH 7/7] chardev: add hotplug support., (continued)
- [Qemu-devel] [PATCH 7/7] chardev: add hotplug support., Gerd Hoffmann, 2012/10/12
- Re: [Qemu-devel] [PATCH 7/7] chardev: add hotplug support., Paolo Bonzini, 2012/10/12
- Re: [Qemu-devel] [PATCH 7/7] chardev: add hotplug support., Gerd Hoffmann, 2012/10/12
- Re: [Qemu-devel] [PATCH 7/7] chardev: add hotplug support., Paolo Bonzini, 2012/10/12
- Re: [Qemu-devel] [PATCH 7/7] chardev: add hotplug support., Gerd Hoffmann, 2012/10/12
- Re: [Qemu-devel] [PATCH 7/7] chardev: add hotplug support., Paolo Bonzini, 2012/10/12
- Re: [Qemu-devel] [PATCH 7/7] chardev: add hotplug support., Gerd Hoffmann, 2012/10/12
- [Qemu-devel] [PATCH v2] chardev: add hotplug support., Gerd Hoffmann, 2012/10/12
- Re: [Qemu-devel] [PATCH v2] chardev: add hotplug support., Gerd Hoffmann, 2012/10/12
- Re: [Qemu-devel] [PATCH v2] chardev: add hotplug support., Paolo Bonzini, 2012/10/12
- Re: [Qemu-devel] [PATCH v2] chardev: add hotplug support.,
Paolo Bonzini <=
- Re: [Qemu-devel] [PATCH v2] chardev: add hotplug support., Lei Li, 2012/10/15
- Re: [Qemu-devel] [PATCH v2] chardev: add hotplug support., Andreas Färber, 2012/10/15
- Re: [Qemu-devel] [PATCH v2] chardev: add hotplug support., Luiz Capitulino, 2012/10/16
- Re: [Qemu-devel] [PATCH v2] chardev: add hotplug support., Eric Blake, 2012/10/15
- Re: [Qemu-devel] [PATCH v2] chardev: add hotplug support., Paolo Bonzini, 2012/10/16
[Qemu-devel] [PATCH 1/7] serial: split serial.c, Gerd Hoffmann, 2012/10/12
[Qemu-devel] [PATCH 3/7] serial: add windows inf file for the pci card to docs, Gerd Hoffmann, 2012/10/12
[Qemu-devel] [PATCH 6/7] usb-serial: only expose device in guest when the chardev is open, Gerd Hoffmann, 2012/10/12
[Qemu-devel] [PATCH 4/7] serial: add 2x + 4x pci variant, Gerd Hoffmann, 2012/10/12
[Qemu-devel] [PATCH 2/7] serial: add pci variant, Gerd Hoffmann, 2012/10/12