[ovs-dev] [PATCH] netdev: Disallow creation of netdevs with unsupported options.
Ethan Jackson
ethan at nicira.com
Wed Aug 3 05:17:53 UTC 2011
>> This patch fixes the issue by disallowing the creation of netdevs
>> with unsupported configuration options.
>
> Currently in netdev-vport.c:parse_tunnel_config() we just warn if
> there is an unknown option. I think if we simply returned an error at
> that point it would have the same effect and we wouldn't need to
> duplicate the logic for valid options.
That would work in most cases. It doesn't work in the case where a
tunnel is created with proper configuration options, and then an
invalid configuration option is added. You run into the same problem
because netdev_open() checks if the configuration is equal before
attempting to open the netdev.
I'd be curious what Ben's opinion is. I think the real problem here
may be with the structure of netdev_open(). Perhaps we should always
be opening the new netdev and replacing the old one with it. I'll
have to think about it.
Ethan
More information about the dev
mailing list