-
Notifications
You must be signed in to change notification settings - Fork 12
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] L2Cache flush issue for some USB device #3
Comments
This bug leads to fatal kernel crashes:
|
Hey @xypron, I just found there is another v5.10 kernel repo at starfive-tech: https://github.com/starfive-tech/sft-riscv-linux-5.10, it has some more commits and at least one that might help with your issue, at least according to the patch description: |
@marckleinebudde Thanks for noting that. Yes there are currently two StarFive Linux 5.10 repos:
Fu Wei (@tekkamanninja) is planning to bring these together into one common repo that both Freelight-U-SDK and Fedora can point to. |
It looks like the https://github.com/starfive-tech/beagle_kernel_5.10 is older, can we delete it to avoid confusion? |
https://github.com/starfive-tech/beagle_kernel_5.10/tree/Fedora has a commit that is only 12 hours old. It would make sense to make Fedora the main branch. |
@xypron right! The last time I checked was before I went to bed :) |
@marckleinebudde @xypron there is new fedora branches just pushed by @tekkamanninja
We agreed today in StarFive meeting that we need to have just one kernel repo and just one uboot repo that both https://github.com/starfive-tech/freelight-u-sdk/tree/starfive and https://github.com/starfive-tech/beaglev_fedora use. @tekkamanninja is working on this consolidation. |
@MichaelZhuxx This is a Fedora issue and please assign to tekkamanninja. |
I understand this is only expected in the VIC7100. But, this seems pretty consistent happening with this address On my boad plugging in a USB drive I also see something like the below.
This seems related to this kernel patch: I am guessing we are somewhere calling flush where we shouldn't. Do we know which call to flush is actually causing the noise above with usb? It seems like the above patch to flush dcache for every write is to workaround the issue of memory writes not working correctly on the VIC7100. Some links:
So I suspect if we fix up the dcache flush workaround the USB may be more stable. Should this issue be moved to linux instead of fedora? |
Closing this issue in favor of starfive-tech/linux#1 |
Based on fedora image: Fedora-riscv64-vic7100-dev-raw-image-Rawhide-20210419121453.n.0-sda.raw
After system starts, plug in a SuperSpeed USB device from thinkplus, several flush information is pop-up.
this USB flash disk cannot wok normally.
The text was updated successfully, but these errors were encountered: