[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/3] chardev: add hotplug support.
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 3/3] chardev: add hotplug support. |
Date: |
Fri, 14 Dec 2012 14:45:08 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/17.0 Thunderbird/17.0 |
Il 14/12/2012 14:18, Gerd Hoffmann ha scritto:
> Hi,
>
>> { 'enum': 'ChardevFileMode', 'data':
>> # pty = console under Windows
>> # serial = tty under POSIX
>> [ 'file', 'pipe', 'parport', 'pty', 'serial' ] }
>
> Hmm, why this enum? I'd stay close to -chardev, i.e. specify the type
> by backend name.
Because...
>> { 'enum: 'ChardevFileSource', 'data':
>> [ 'path', 'fd' ] }
>
> I guess I'd just create a new backend type for file descriptor passing
> instead of fitting that into all the existing ones.
... are you passing a file descriptor for a pipe, a file or a
parallel/serial port?
(pty and console have no arguments, I misremembered).
>> { 'union': 'ChardevBackend', 'data': {
>
> This union thing is new, isn't it?
Yeah, a few months old.
> Makes sense to use that indeed.
>
>> 'socket': 'ChardevSocket',
>> 'udp': 'UDPSocketAddress',
>> 'file': 'ChardevFile',
>> 'null': 'ChardevDummy',
>> 'msmouse': 'ChardevDummy',
>> 'braille': 'ChardevDummy',
>> 'stdio': 'ChardevDummy',
>> 'vc': 'ChardevVC',
>
> I doubt we need them all hotpluggable.
True, but:
1) I believe long term it's good to move away from QemuOpts; it's good
to provide a complete interface even if all we're doing for now is
building QemuOpts out of the struct.
2) most of them have no options and trivial anyway;
3) the complicated ones (socket, file, perhaps udp) are also the useful
ones.
Paolo
- [Qemu-devel] [PATCH RESENT 0/3] chardev hotplug patch series, Gerd Hoffmann, 2012/12/14
- [Qemu-devel] [PATCH 2/3] chardev: fix QemuOpts lifecycle, Gerd Hoffmann, 2012/12/14
- [Qemu-devel] [PATCH 1/3] chardev: add error reporting for qemu_chr_new_from_opts, Gerd Hoffmann, 2012/12/14
- [Qemu-devel] [PATCH 3/3] chardev: add hotplug support., Gerd Hoffmann, 2012/12/14
- Re: [Qemu-devel] [PATCH 3/3] chardev: add hotplug support., Paolo Bonzini, 2012/12/14
- Re: [Qemu-devel] [PATCH 3/3] chardev: add hotplug support., Eric Blake, 2012/12/14
- Re: [Qemu-devel] [PATCH 3/3] chardev: add hotplug support., Gerd Hoffmann, 2012/12/14
- Re: [Qemu-devel] [PATCH 3/3] chardev: add hotplug support.,
Paolo Bonzini <=
- Re: [Qemu-devel] [PATCH 3/3] chardev: add hotplug support., Gerd Hoffmann, 2012/12/14
- Re: [Qemu-devel] [PATCH 3/3] chardev: add hotplug support., Gerd Hoffmann, 2012/12/14
- Re: [Qemu-devel] [PATCH 3/3] chardev: add hotplug support., Paolo Bonzini, 2012/12/14