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

arping: support -i interval option #86

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@booxter

booxter commented May 11, 2017

This option will allow to set time interval to use between packets sent.

It may become handy to work around timing issues in the network. For
example, Linux kernel as of the time of writing by default ignores all
ARP updates that arrive in 1-second interval since previous update for
the same table entry irrespective of whether the previous update was
ignored
. It means that calling apring with -c 3 may still not affect
Linux network peers because every consequent update is ignored because
of the previous ARP update (that was also ignored).

In this scenario, we would now be able to call arping -i 2 to work that
kernel behaviour around.

Linux fix can be found in: https://patchwork.ozlabs.org/patch/760372/

@jsynacek

This comment has been minimized.

Show comment
Hide comment
@jsynacek

jsynacek May 15, 2017

Contributor

LGTM.

Contributor

jsynacek commented May 15, 2017

LGTM.

@okias

This comment has been minimized.

Show comment
Hide comment
@okias

okias Aug 5, 2017

Member

Thank you for your pull request.

Since I just now fixed documentation generation (man,html),
may I ask you to incorporate your original SGML man entry into new XML structure? Should be easy and quick. Also for testing man pages you can use xsltproc, which is piece of cake! :)

After it, nothing prevents for accepting pull request.

Member

okias commented Aug 5, 2017

Thank you for your pull request.

Since I just now fixed documentation generation (man,html),
may I ask you to incorporate your original SGML man entry into new XML structure? Should be easy and quick. Also for testing man pages you can use xsltproc, which is piece of cake! :)

After it, nothing prevents for accepting pull request.

@okias okias requested a review from pavlix Aug 5, 2017

@okias okias added this to the iputils-20171001 release milestone Aug 5, 2017

arping: support -i interval option
This option will allow to set time interval to use between packets sent.

It may become handy to work around timing issues in the network. For
example, Linux kernel as of the time of writing by default ignores all
ARP updates that arrive in 1-second interval since previous update for
the same table entry *irrespective of whether the previous update was
ignored*. It means that calling apring with -c 3 may still not affect
Linux network peers because every consequent update is ignored because
of the previous ARP update (that was also ignored).

In this scenario, we would now be able to call arping -i 2 to work that
kernel behaviour around.

Linux fix can be found in: https://patchwork.ozlabs.org/patch/760372/
@booxter

This comment has been minimized.

Show comment
Hide comment
@booxter

booxter Sep 29, 2017

Updated for xml transition.

booxter commented Sep 29, 2017

Updated for xml transition.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment