Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 30 Nov 2011 22:28:37 +0100
From:      Damien Fleuriot <ml@my.gd>
To:        freebsd-hackers@freebsd.org
Subject:   Re: sysctl description spillover and also setting the sysctl ?
Message-ID:  <4ED6A005.60300@my.gd>
In-Reply-To: <20111130184815.GA70174@DataIX.net>
References:  <20111125073630.GC7915@DataIX.net> <201111301152.47002.jhb@freebsd.org> <20111130184815.GA70174@DataIX.net>

next in thread | previous in thread | raw e-mail | index | archive | help


On 11/30/11 7:48 PM, Jason Hellenthal wrote:
> 
> 
> On Wed, Nov 30, 2011 at 11:52:46AM -0500, John Baldwin wrote:
>> On Friday, November 25, 2011 2:36:30 am Jason Hellenthal wrote:
>>>
>>> Found a troubling result of the following and figured someone might want to 
>> take a look.
>>>
>>> Pay close attention to the output and behavior.
>>>
>>> sysctl net.inet.udp.blackhole=0
>>> sysctl net.inet.udp.blackhole
>>> sysctl -d net.inet.udp.blackhole=1
>>> sysctl net.inet.udp.blackhole
>>>
>>>
>>> Is this expected ? should it not just display the description instead of 
>> adjusting ? as well not display the description like it is adjusting the 
>> description too ?
>>
>> Hah, cute.  It should probably fail with an error if you do something like 
>> that, yes.
>>
> 
> Yeah thats what I thought about it to but the more I thought about it, if it just displayed the values changing instead of the description when =N is supplied I think that would be acceptable to. 0 -> 1 in this case. Or possibly sys.oid: 0 -> 1 # <Description> since sysctl.conf(5) also takes comments like that.
> 
> Not really thats something at the top of the list for fixes though. Low fruit. Food for thought.
>


Perhaps you would kindly provide a patch for this ?

I might have a look at it if nobody does, sounds trivial enough that I
might be able to handle it ;)



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4ED6A005.60300>