On 16.12.2015 17:56, Bjørn Mork wrote:
>
>
> On December 16, 2015 12:30:08 PM CET, Hannes Frederic Sowa
> <han...@stressinduktion.org> wrote:
>> Signed-off-by: Hannes Frederic Sowa <han...@stressinduktion.org>
>> ---
>> ip/iplink.c | 4 +++-
>> 1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/ip/iplink.c b/ip/iplink.c
>> index f30de86d1858a0..e824082f7d8149 100644
>> --- a/ip/iplink.c
>> +++ b/ip/iplink.c
>> @@ -84,7 +84,7 @@ void iplink_usage(void)
>> fprintf(stderr, " [ state { auto |
>> enable | disable} ] ]\n");
>> fprintf(stderr, " [ master DEVICE ]\n");
>> fprintf(stderr, " [ nomaster ]\n");
>> - fprintf(stderr, " [ addrgenmode { eui64 | none
>> } ]\n");
>> + fprintf(stderr, " [ addrgenmode { eui64 | none
>> | stable_secret }
>> ]\n");
>> fprintf(stderr, " [ protodown { on | off }
>> ]\n");
>> fprintf(stderr, " ip link show [ DEVICE | group GROUP ] [up]
>> [master DEV] [type TYPE]\n");
>>
>> @@ -176,6 +176,8 @@ static int get_addr_gen_mode(const char *mode)
>> return IN6_ADDR_GEN_MODE_EUI64;
>> if (strcasecmp(mode, "none") == 0)
>> return IN6_ADDR_GEN_MODE_NONE;
>> + if (strcasecmp(mode, "stable_secret") == 0)
>> + return IN6_ADDR_GEN_MODE_STABLE_PRIVACY;
>> return -1;
>> }
>>
>
> Ah, didn't notice this before I sent my similar patch. Sorry about that.
>
> But FWIW, I really don't like the renaming you do here.
> If the mode is 'stable_privacy', then it should be known
> by that name here as well.
I am fine with both ways. We can take your patch as you also updated the
man page. :)
The reason why I chose stable secret in the end was to align it with the
already visible proc filename stable_secret and didn't care too much
about the internal variable names.
Thanks,
Hannes
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
