Re: [Qemu-devel] [PATCH v7 02/15] net: use Netdev instead of NetClientOptions in client init

[Markus Armbruster <armbru@redhat.com>]

Eric Blake <eblake@redhat.com> writes:

> From: Kővágó, Zoltán <dirty.ice.hu@gmail.com>

First posted almost a year ago.  I'm glad we're finally ready for this
work.  Zoltán, I'm sorry it has taken us so long.

> This way we no longer need NetClientOptions and can convert Netdev
> into a flat union.

That's work enabled by this patch, not done by it.  We won't actually
get rid of NetClientOptions, only confine it to legacy command line
support.

> Signed-off-by: Kővágó, Zoltán <DirtY.iCE.hu@gmail.com>
> Reviewed-by: Eric Blake <eblake@redhat.com>
> Message-Id: <93ffdfed7054529635e6acb935150d95dc173a12.1441627176.git.DirtY.iCE.hu@gmail.com>
>
> [rework net_client_init1() to pass Netdev by copying from NetdevLegacy,
> rather than merging the two types; rebase to qapi changes]
> Signed-off-by: Eric Blake <eblake@redhat.com>

The patch is mostly mechanical: wherever we replace parameter
NetClientOptions *opts by Netdev *netdev, we replace opts by
netdev->opts, because netdev wraps opts:

    struct Netdev {
        char *id;
        NetClientOptions *opts;
    };

The interesting part is the change to net_client_init1(), which looks
sane.