netdev - Re: arp(7) description of gc_stale

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <94006afd-5ba5-ffd3-835e-22e2ca87dec3@gmail.com>
Date:   Wed, 14 Apr 2021 12:09:05 +0200
From:   "Alejandro Colomar (man-pages)" <alx.manpages@...il.com>
To:     Adam Liddell <ml+kernel.org@...ddell.com>, mtk.manpages@...il.com
Cc:     linux-man@...r.kernel.org, netdev@...r.kernel.org
Subject: Re: arp(7) description of gc_stale_time

[CC += netdev]

On 4/12/21 12:13 AM, Adam Liddell wrote:
> Hi,
> 
> Any opinion on this?
> 
> Adam
> 
> On Tue, 16 Feb 2021, at 17:00, Adam Liddell wrote:
>> Hi,
>>
>> The arp(7) page's description of gc_stale_time doesn't quite describe
>> the behaviour correctly, at least as I understand it.
>>
>> The current description suggests this is the time interval at which a
>> loop will look for stale entries. However, this field is the threshold
>> for marking an entry dead for removal, based on when it was last used
>> (see net/core/neighbour.c lines 935-942) and whether the table is over
>> gc_thresh1. How often this check is done appears to be determined by
>> base_reachable_time (/2) and the third option gc_interval is not
>> involved in this process as far as I can tell, despite its name.
>>
>> Perhaps a draft alternate description could be something along the lines of:
>>
>> Determines the threshold for removing a cache entry after it was last
>> used and when the cache is larger than gc_thresh1. Defaults to 60
>> seconds.
>>

Hi Adam,

Thanks for the ping.  I don't know, sorry.
I added netdev@ so that maybe they can give an opinion.  In the 
man-pages, that text has been there since we use git, so I don't know 
who wrote it.

Thanks,

Alex


-- 
Alejandro Colomar
Linux man-pages comaintainer; https://www.kernel.org/doc/man-pages/
http://www.alejandro-colomar.es/