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

Pico W crashes when accessing web workflow interface #7770

Closed
RetiredWizard opened this issue Mar 22, 2023 · 5 comments · Fixed by #7779
Closed

Pico W crashes when accessing web workflow interface #7770

RetiredWizard opened this issue Mar 22, 2023 · 5 comments · Fixed by #7779
Labels
bug rp2040 Raspberry Pi RP2040 web workflow
Milestone

Comments

@RetiredWizard
Copy link

RetiredWizard commented Mar 22, 2023

CircuitPython version

Adafruit CircuitPython 8.1.0-alpha.2-16-g60a9c7e5b on 2023-03-22; Raspberry Pi Pico W with rp2040

Code/REPL

No code executed, tested without a code.py

Behavior

During my testing I first dismounted the CIRCUITPY drive and then attempted to access the web workflow URL. on commit 60a9c7e and newer the Pi Pico W would restart as soon as I clicked on the link. If I had an serial console open an OSError: [Errno 11] EAGAIN message was displayed and then the board would restart. Even though the board appeared to restart once the error occurred I was unable to reconnect to the serial port until I power cycled the board.

The build from this commit: 09f84e3 did not seem to have the issue.

Description

No response

Additional information

No response

@tannewt tannewt added rp2040 Raspberry Pi RP2040 web workflow labels Mar 22, 2023
@tannewt tannewt added this to the 8.1.0 milestone Mar 22, 2023
@jepler
Copy link
Member

jepler commented Mar 22, 2023

#7623 introduced a problem that I believe could cause crashes in the web worfklow, but we thought we promptly fixed it in #7632. The ref you are testing at, 60a9c7e, is within the range of commits known to be affected. Can you check whether this problem is still affecting current versions?

@RetiredWizard
Copy link
Author

I am seeing the issue on the latest build from circuitpython.org. I am a bit concerned I'm missing something because the commit that the problem seems to start at doesn't make a lot of sense for this type of issue and I'm surprised that no one else has reported this by now. I did try plugging the pico w into both a Windows 8 PC and a Linux box and got the same result. I also connected the Pico to a usb charger so there was no computer involved and confirmed that web workflow didn't work in that case either.

@jepler
Copy link
Member

jepler commented Mar 22, 2023

Just to eliminate doubt can you show the boot_out.txt from the "latest version" you're using?

@RetiredWizard
Copy link
Author

RetiredWizard commented Mar 22, 2023

Adafruit CircuitPython 8.1.0-beta.0-48-gcfedcd411 on 2023-03-22; Raspberry Pi Pico W with rp2040
Board ID:raspberry_pi_pico_w
UID:E6614103E71F6622
MAC:28:CD:C1:00:43:35

It looks like another one just dropped on S3, it also does the same thing for me:

Auto-reload is on. Simply save files over USB to run them or enter REPL to disable.

Press any key to enter the REPL. Use CTRL-D to reload.

Adafruit CircuitPython 8.1.0-beta.0-50-ge1f16472c on 2023-03-22; Raspberry Pi Pico W with rp2040
>>>
OSError: [Errno 11]

@DavePutz
Copy link
Collaborator

I am also reproducing this on the latest builds (both downloaded and generated locally). The EAGAIN error (Errno 11) is being raised in socketpool_socket_recv_into() due to the change introduced in PR #7623. However, that identical code is working fine in CP 8.0.4.
Trying to see what other CP 8.1 changes may be affecting this.

jepler added a commit to jepler/circuitpython that referenced this issue Mar 22, 2023
This reverts commit 7e6e824.

Fixes adafruit#7770

The change in adafruit#7623 needs to be revered; the raise-site added in adafruit#7632
is the correct one and the one in socketpool needs to be reverted.

This is not affecting 8.0.x because adafruit#7623 was not back-ported to there
before we realized it was not a full fix.

Both adafruit#7770 and adafruit#7606 should be re-tested. I didn't test.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug rp2040 Raspberry Pi RP2040 web workflow
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants