Skip to content

Conversation

@HedongGao
Copy link
Contributor

Summary

Nuttx use IFF_NOARP as a packet config, this is illogical and no longer in line with user habits.
Referring to the ARP configuration method of Linux, Nuttx has added the ability to configure ARP on interface.

Impact

Referring to the custom of configuring ARP on Linux interfaces, the 'arp/- app' parameter has been added to the ifconfig command. By default, the interface supports ARP functionality.
When the ARP function of an interface is turned off, the net stack will discard ARP messages received from that interface, and the protocol stack will not send messages to the outside through that interface.

Testing

When compiling nuttx, open the ping tool for functional testing.

Set up a SIM environment, sim and host can ping each other successfully.
After using the command 'ifconfig eth0 -arp', the ping between sim and host failed. The packet capture shows that sim cannot process APR request messages or send ARP request messages.
Then use 'ifconfig eth0 arp' to enable arp fuction for eth0, sim and host can ping each other again.

@github-actions github-actions bot added Area: Networking Effects networking subsystem Size: M The size of the change in this PR is medium labels Dec 24, 2025
@HedongGao
Copy link
Contributor Author

In another PR, modifications were made to the ifconfig tool

@xiaoxiang781216
Copy link
Contributor

@HedongGao please fix:

../nuttx/tools/checkpatch.sh -c -u -m -g 28298b9d5e3e0c132446e57841f1f0634ba0aef1..HEAD
Used config files:
    1: .codespellrc
/home/runner/work/nuttx/nuttx/nuttx/net/arp/arp_input.c:203: recive ==> receive
/home/runner/work/nuttx/nuttx/nuttx/net/arp/arp_out.c:254: forbiden ==> forbidden
Some checks failed. For contributing guidelines, see:
  https://github.com/apache/nuttx/blob/master/CONTRIBUTING.md

ARP can be configured on interface.

Signed-off-by: gaohedong <gaohedong@xiaomi.com>
@HedongGao
Copy link
Contributor Author

@HedongGao please fix:

../nuttx/tools/checkpatch.sh -c -u -m -g 28298b9d5e3e0c132446e57841f1f0634ba0aef1..HEAD
Used config files:
    1: .codespellrc
/home/runner/work/nuttx/nuttx/nuttx/net/arp/arp_input.c:203: recive ==> receive
/home/runner/work/nuttx/nuttx/nuttx/net/arp/arp_out.c:254: forbiden ==> forbidden
Some checks failed. For contributing guidelines, see:
  https://github.com/apache/nuttx/blob/master/CONTRIBUTING.md

DONE

@xiaoxiang781216 xiaoxiang781216 merged commit 23013b3 into apache:master Dec 25, 2025
73 of 76 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Networking Effects networking subsystem Size: M The size of the change in this PR is medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants