Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change NFD's CS replacement policy to LRU #5

Merged
merged 1 commit into from
Nov 11, 2024
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
23 changes: 10 additions & 13 deletions templates/nfd/nfd.conf.j2
Original file line number Diff line number Diff line change
@@ -1,8 +1,3 @@
general
{

}

log
{
default_level INFO
Expand All @@ -20,7 +15,7 @@ tables
cs_max_packets 262144
{% endif %}

cs_policy priority_fifo
cs_policy lru
cs_unsolicited_policy drop-all

strategy_choice
Expand Down Expand Up @@ -61,13 +56,13 @@ face_system

udp
{
listen yes
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is just for consistency with the other config sections that have this explicitly set (even though it's the default).

port {{udp_port}}
enable_v4 yes
enable_v6 yes

idle_timeout 600
unicast_mtu 1452
keep_alive_interval 25
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is nonfunctional, it was never implemented in NFD. The config file parser still accepts it for some reason but I'd like to drop it at some point.


mcast yes
mcast_group 224.0.23.170
Expand All @@ -76,11 +71,12 @@ face_system
mcast_port_v6 56363
mcast_ad_hoc no

whitelist {
whitelist
{
*
}
blacklist {

blacklist
{
}
}

Expand All @@ -93,11 +89,12 @@ face_system
mcast_group 01:00:5E:00:17:AA
mcast_ad_hoc no

whitelist {
whitelist
{
*
}
blacklist {

blacklist
{
}
}

Expand Down