[ovs-dev] [PATCH] bridge: New appctl command iface/set-admin-state.

Ethan Jackson ethan at nicira.com
Fri Aug 10 18:38:25 UTC 2012


> I kind of feel like it's a wart that should be confined to
> netdev-dummy.

That's fine, I could think of arguments either way and made a
judgement call. I'll move the appctl command to netdev-dummy.

Ethan

>
> s/ , /, / here, twice:
>> +    if (!strcasecmp(argv[argc - 1] , "up")) {
>> +        up = true;
>> +    } else if ( !strcasecmp(argv[argc - 1] , "down")) {
>> +        up = false;



More information about the dev mailing list