[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: |
Thu, 18 Oct 2012 20:44:49 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:16.0) Gecko/20121016 Thunderbird/16.0.1 |
Il 18/10/2012 12:03, Gerd Hoffmann ha scritto:
> Trying another approach, see attached patch. This adds backend-specific
> qemu commands to add chardevs, with just the parameters needed for the
> specific backend. Starting with file and tty, both accepting a path.
> 'file' is nice for testing, 'tty' very useful when hotplugging serial
> devices on the host. Others can be added as needed. We probably don't
> need all of them. For example hotplugging the 'stdio' chardev doesn't
> make much sense.
>
> Advantage #1: Cleaner API.
> Advantage #2: No legacy syntax headache when qomifying chardevs.
Disadvantage #1: Harder to extend, more code
Disadvantage #2: Different from all other kinds of host devices.
I still prefer the other one.
Paolo