Skip to content
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

Freertos background asserts if IPv6 is enabled #1591

Merged
merged 1 commit into from
Dec 21, 2023

Conversation

peterharperuk
Copy link
Contributor

If LWIP_IPV6=1 a request to update the multicast list occurs in a callback in the low priority interrupt.

This makes an ioctl call into the driver, at the end of which is a call to cyw43_await_background_or_timeout_us (see CYW43_DO_IOCTL_WAIT). It is attempting to delay until there's "some work to do".

For Freertos this fails an assertion as an attempt is made to acquire a semaphore in interrupt context.

Fixes #1590

If LWIP_IPV6=1 a request to update the multicast list occurs in a
callback in the low priority interrupt.

This makes an ioctl call into the driver, at the end of which is a call
to cyw43_await_background_or_timeout_us (see CYW43_DO_IOCTL_WAIT).
It is attempting to delay until there's "some work to do".

For Freertos this fails an assertion as an attempt is made to acquire a
semaphore in interrupt context.

Fixes raspberrypi#1590
@kilograham kilograham merged commit 9f45e3c into raspberrypi:develop Dec 21, 2023
1 check passed
@kilograham kilograham added this to the 1.5.2 milestone Jan 12, 2024
mungewell pushed a commit to mungewell/pico-sdk that referenced this pull request Mar 18, 2024
If LWIP_IPV6=1 a request to update the multicast list occurs in a
callback in the low priority interrupt.

This makes an ioctl call into the driver, at the end of which is a call
to cyw43_await_background_or_timeout_us (see CYW43_DO_IOCTL_WAIT).
It is attempting to delay until there's "some work to do".

For Freertos this fails an assertion as an attempt is made to acquire a
semaphore in interrupt context.

Fixes raspberrypi#1590
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.

2 participants