qemu-devel
[Top][All Lists]
Advanced

[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



reply via email to

[Prev in Thread] Current Thread [Next in Thread]