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

add xHCI host dead troubleshooting #577

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

XMRHRX
Copy link

@XMRHRX XMRHRX commented Jul 6, 2024

Hello, I want to contribute my solution about a problem trouble me a few months.

When STM32F freeze and not responding to Linux kernel, so kernel think it dead, reconnect USB won't make kernel re-check it.
I find the solution that we can use bind & unbind to make kernel re-check it.

@XMRHRX XMRHRX requested a review from a team as a code owner July 6, 2024 14:22
@BartMassey
Copy link
Member

This sounds more like a Linux kernel bug than something the user should have to deal with? Maybe file a bug report there? (Yes, I know that reporting bugs on the Linux kernel is an adventure. But the host controller really shouldn't "get stuck" here, I think, which is what it looks like is happening offhand.)

@XMRHRX
Copy link
Author

XMRHRX commented Jul 8, 2024

Well, I think I need time to check these,
but I feel it less seem a bug, it feel more like this is just how it currently work, which if not get device response just remove it and never check it.
Maybe other "full implement" device like mouse, will make some request when re-connect, so kernel mark it connected again.(just a guess, I just dive into hardware recently)
Should I close this merge, or just keep it here till I have time to check it?

@BartMassey
Copy link
Member

Can leave it open for now. The messages look to me like the Linux host controller has gotten lost, but I could be wrong.

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