lwip: explicitly set required netdev events#9553
lwip: explicitly set required netdev events#9553PeterKietzmann merged 1 commit intoRIOT-OS:masterfrom
Conversation
|
Did you test this? |
|
@PeterKietzmann I've tested with |
|
Ah ok. Yes, native works here as well. On samr21-xpro I get the following assertion which might be unrelated. |
|
Let's investigate that for a bit then... |
|
@PeterKietzmann I'm unable to reproduce that on IoT-lab so far. Do you have more information on how to reproduce this? |
|
I'm on your branch an flashed tests/lwip to one samr21-xpro board. Some seconds after startup the above mentioned assertion triggers |
|
BTW: same on master... |
|
Still nothing on my IoT-lab nodes. I would find it very strange if practically changing nothing causes assertion failures in the lwip packet buffer. |
It doesn't relate to your change. The same error persists on current master, without this PR. So we could theoretically merge this one. On the iotlab (with M3 node) I don't get the assertion either. Did you try the samr21-xpro in saclay? |
Yes, node number 1 and 2 :) |
|
Can you try to build in Docker and submit an other exp on samr nodes? |
|
@PeterKietzmann Done, still nothing, works perfectly fine on this side 😢 |
|
Well, obviously this is not an issue with your PR. Seems more toolchain related. Will open an issue :/ |
Contribution description
Similar to #9467, this modifies LwIP to explicitly set the
NETOPT_RX_END_IRQevent for netdev devices.Issues/PRs references
None