-
Notifications
You must be signed in to change notification settings - Fork 7.4k
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
[USB][ABORT][v4.4.2] root_port_handle_events() hits HUB_DRIVER_STATE_ROOT_RECOVERY (IDFGH-8214) #9707
Comments
From @Dazza0 from previous thread.
Answers:
No. It's about 1 out of 20 unplugs (rough guess). Note: I sometimes plug and unplug very rapidly (trying to cause failures, for testing), but I can't remember if that is a prerequisite.
Yes, I'll try this today. Is there a specific line of code to insert this log? |
I finally repro'd it again!! It definitely takes awhile to repro (5+ minutes of trying). I was about to give up! So the issue is: we are in HUB_DRIVER_STATE_ROOT_RECOVERY state, which is not handled by root_port_handle_events() switch statement, when I unplug the device. First things first: I was only able to repro it with
I added logs to
And this logging code to hcd.c
Logs:
|
And here are the full logs
|
bump =) |
@Dazza0 , did you ever get a chance to reason about this? Does |
Another bump! |
bump! Have there been any changes around this? |
bump again |
did we ever get to the bottom of this? |
I'm still hitting this on v5.1.2! Crash when unplugging usb device
|
General issue report
ESP-IDF v4.4.2
ESP32-S3
See original issue: #8748 for more details.
Esp USB stack sometimes crashes (~5% chance) when unplugging a USB device.
The abort happens because we are in HUB_DRIVER_STATE_ROOT_RECOVERY state, which is not handled by
root_port_handle_events()
switch statement.Logs:
The text was updated successfully, but these errors were encountered: