Skip to content

Conversation

JannesVolkens
Copy link
Contributor

@JannesVolkens JannesVolkens commented Feb 26, 2020

Contribution description

After #13383 fixed the eth unicast filtering on stm32 , there were problems (described in #13383 ) when using IP. This PR is a fix for this problem by reactivating the passing of all frames with a multicast address.

Testing procedure

None.

Issues/PRs references

#13383

@benpicco benpicco added Area: drivers Area: Device drivers Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) labels Feb 26, 2020
@kfessel
Copy link
Contributor

kfessel commented Feb 26, 2020

Fixes the issue of not working IPv6: (Neighbor Discovery Protocol) needs Ethernet Group (Muticast) addresses to work.

(maybe this could be part of the commit message)

i testes the patch: compile -> flash -> (device reset (there is an issue) -> ping is working again

could someone please bump murdock for #13470 : I would like to see debugging work again on master

@benpicco benpicco added Reviewed: 3-testing The PR was tested according to the maintainer guidelines CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Feb 26, 2020
@PeterKietzmann
Copy link
Member

I've tested this with Linux<->RIOT on a nucleo-f767z1 running examples/gcoap. In more detail:

  • Flash the application to the board.
  • Empty the Linux neighbor table for the Ethernet interface (to avoid old entries).
  • Ping from Linux or RIOT to the link local address of the other node.
  • Doesn't work. Th nucleo board needs a reset -> @kfessel do you have a fix at hand or would you be so kind to open an issue?
  • After reset, pining works again.

I'm still not happy to enable all multicast traffic by default. @JannesVolkens as discussed offline, please open an issue for that

@PeterKietzmann PeterKietzmann merged commit c78962a into RIOT-OS:master Feb 26, 2020
@kfessel
Copy link
Contributor

kfessel commented Feb 26, 2020

created Issue #13490 to track the nucleo board needs a reset bug

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: drivers Area: Device drivers CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Reviewed: 3-testing The PR was tested according to the maintainer guidelines Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants