Sure, no problems. I was looking for that commit, but using git blame
didn't work, since there the data structure seems to be part of
autogenerated code.
2015-11-05 14:01 GMT+01:00 Markus Armbruster <arm...@redhat.com>:
> Vincenzo Maffione <v.maffi...@gmail.com> writes:
>
>> Reorganization of struct NetClientOptions caused a compilation failure
>> of the netmap backend. This patch fixes the issue by properly accessing the
>> union field.
>>
>> Signed-off-by: Vincenzo Maffione <v.maffi...@gmail.com>
>> ---
>> net/netmap.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/net/netmap.c b/net/netmap.c
>> index 508b829..4197a9c 100644
>> --- a/net/netmap.c
>> +++ b/net/netmap.c
>> @@ -439,7 +439,7 @@ int net_init_netmap(const NetClientOptions *opts,
>> const char *name, NetClientState *peer, Error **errp)
>> {
>> /* FIXME error_setg(errp, ...) on failure */
>> - const NetdevNetmapOptions *netmap_opts = opts->netmap;
>> + const NetdevNetmapOptions *netmap_opts = opts->u.netmap;
>> NetClientState *nc;
>> NetmapPriv me;
>> NetmapState *s;
>
> Broken in commit e4ba22b. Recommend to point to that in the commit
> message. Thanks for cleaning up the mess we made there!
>
> Reviewed-by: Markus Armbruster <arm...@redhat.com>
--
Vincenzo Maffione
