-
Notifications
You must be signed in to change notification settings - Fork 0
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
Bypass watchdog for long dns resolution #3
Comments
For esp-idf I just added |
I would leave this for later development as it is something that requires quite low level change. |
Came up with a bit of a cleaner solution in PR #5. |
I don't know how to slow down my DNS to trigger the watchdog, but the code looks good and my tests succeeded. Many thanks. Just one tip, I changed
to
because it should be in milliseconds. Is that right? Don't you mind if I wrap it in two functions |
I cannot push to your branch, so test it using mine |
Many thanks @thomas0bernard, I've just merged your PR. |
The DNS resolution that happens in
esp_wireguard_peer_init()
can take long enough on some network setups to trigger the task watchdog timeout panic. I saw that esphome's OTA component seems to have dealt with the same issue, temporarily raising the timeout value for a long operation and bringing it back down after. I'm wondering if this is too much of an edge case to get an "official" fix or if it's worth looking into.See second report here esphome#4256 (comment)
@thomas0bernard this issue for your second report.
The text was updated successfully, but these errors were encountered: