Skip to content

Make APRX hear outgoing packets on ax25 ports#81

Open
tvrusso wants to merge 1 commit into
PhirePhly:masterfrom
tvrusso:ax25-outgoing-packets
Open

Make APRX hear outgoing packets on ax25 ports#81
tvrusso wants to merge 1 commit into
PhirePhly:masterfrom
tvrusso:ax25-outgoing-packets

Conversation

@tvrusso
Copy link
Copy Markdown

@tvrusso tvrusso commented May 19, 2026

APRX set the protocol for the ax25 receive socket to ETH_P_AX25. This was done to prevent non-AX25 packets from coming in on the socket. It was put in in commit 395fbdf back in 2007.

The use of ETH_P_AX25 assured that non-AX25 packets were ignored, but it predated other code implemented in commit ff8fd51 just two years later, that filtered out a lot of uninteresting packets.

This commit reinstates use of ETH_P_ALL and expands the filtering to accept outgoing packets.

If the AX.25 port is shared by aprx and other aprs programs (as they are at my station), aprx now sees the packets transmitted by those other programs and gates or digipeats them appropriately. Previously, those packets would only be seen by aprx if they were digipeated by another station and heard on our receive ports.

APRX set the protocol for the ax25 receive socket to ETH_P_AX25.  This was
done to prevent non-AX25 packets from coming in on the socket.  It was
put in in commit 395fbdf back in 2007.

The use of ETH_P_AX25 assured that non-AX25 packets were ignored, but it
predated other code  implemented in commit ff8fd51 just two years later, that
filtered out a lot of uninteresting packets.

This commit reinstates use of ETH_P_ALL and expands the filtering to accept
outgoing packets.

If the AX.25 port is shared by aprx and other aprs programs, aprx now sees
the packets transmitted by those other programs an gates or digipeats them
appropriately.  Previously, those packets would only be seen by aprx if
they were digipeated by another station and heard on our receive ports.
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.

1 participant