[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] net: Inform the user about deprecated -net opti
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH] net: Inform the user about deprecated -net options |
Date: |
Tue, 15 Dec 2015 09:46:51 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
On 12/15/2015 09:21 AM, Paolo Bonzini wrote:
>
> In fact there isn't much shared code in the is_netdev=0 and is_netdev=1
> cases. Perhaps you could just make a shared function with just
>
> if (net_client_init_fun[opts->type](opts, name, peer, errp) < 0) {
> /* FIXME drop when all init functions store an Error */
> if (errp && !*errp) {
> error_setg(errp, QERR_DEVICE_INIT_FAILED,
> NetClientOptionsKind_lookup[opts->type]);
> }
> return -1;
> }
>
>
> and inline all the rest of net_client_init1, net_visit, net_client_init
> into two functions netdev_add and net_legacy_add. Then -net handling
> (including HMP) can be moved into a separate file which no one looks at.
There's also some pending patches from qapi refactoring and from
Zoltan's work towards audio unification (which spilled over into netdev
refactoring), which may impact things here. For my latest version, see:
http://repo.or.cz/qemu/ericb.git/shortlog/refs/heads/qapi
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH] net: Inform the user about deprecated -net options, Thomas Huth, 2015/12/15
- Re: [Qemu-devel] [PATCH] net: Inform the user about deprecated -net options, Paolo Bonzini, 2015/12/15
- Re: [Qemu-devel] [PATCH] net: Inform the user about deprecated -net options, Thomas Huth, 2015/12/15
- Re: [Qemu-devel] [PATCH] net: Inform the user about deprecated -net options, Paolo Bonzini, 2015/12/15
- Re: [Qemu-devel] [PATCH] net: Inform the user about deprecated -net options,
Eric Blake <=
- Re: [Qemu-devel] [PATCH] net: Inform the user about deprecated -net options, Thomas Huth, 2015/12/15
- Re: [Qemu-devel] [PATCH] net: Inform the user about deprecated -net options, Paolo Bonzini, 2015/12/15
- Re: [Qemu-devel] [PATCH] net: Inform the user about deprecated -net options, Thomas Huth, 2015/12/15
- Re: [Qemu-devel] [PATCH] net: Inform the user about deprecated -net options, Paolo Bonzini, 2015/12/15
- Re: [Qemu-devel] [PATCH] net: Inform the user about deprecated -net options, Thomas Huth, 2015/12/16
- Re: [Qemu-devel] [PATCH] net: Inform the user about deprecated -net options, Paolo Bonzini, 2015/12/16
Re: [Qemu-devel] [PATCH] net: Inform the user about deprecated -net options, Peter Maydell, 2015/12/15