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

add readme #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

add readme #1

wants to merge 1 commit into from

Conversation

mwarning
Copy link

@mwarning mwarning commented Mar 11, 2023

A small readme.

@mwarning
Copy link
Author

A small explanation what on and off link prefix means is still missing.

Copy link
Member

@jplitza jplitza left a comment

Choose a reason for hiding this comment

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

Good idea, thanks!

Two remarks:

  • Please always use addresses and prefixes from 2001:db8::/32 ("db8", as in "debate") for documentation purposes
  • The only difference between -p and -a is the L-flag! The A-flag is always set:

    uradvd/uradvd.c

    Lines 500 to 503 in 9b0da60

    uint8_t flags = ND_OPT_PI_FLAG_AUTO;
    if (G.prefixes_onlink[i])
    flags |= ND_OPT_PI_FLAG_ONLINK;

    The L-flag means that the receiver is supposed to install a "connected route" for the prefix, while its absence tells the receiver to install a route via the announcing router for the prefix (if it doesn't already have a connected route for it). IMO it doesn't make much sense to set the A-flag without the L-flag, but oh well.

@mwarning
Copy link
Author

Updated. Not sure if the explanation is slightly overkill. :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants