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

mqueue: revert refactor of mqueue #13814

Closed

Conversation

fdcavalcanti
Copy link
Contributor

@fdcavalcanti fdcavalcanti commented Oct 3, 2024

Summary

This PR reverts #13413 which refactors mqueue.
The reason for this is that this refactor broke the WiFi CI on both Xtensa and RISC-V Espressif devices.

So far, the following has stopped working across affected SoCs:

  • ifconfig after reboot not responding as it used to
  • connection to AP using wapi throws many errors and is unstable
  • Multiple failures on WiFi performance test using IPERF.

If this affected Espressif devices using those tools, it is possible it affected other devices as well. I'm keeping an eye open on new issues.

Impact

Restores WiFi support on Espressif devices.

Testing

Reverted #13413 on master, run the entire CI and everything works again.

@github-actions github-actions bot added Area: Tooling Area: CI Area: OS Components OS Components issues Size: XL The size of the change in this PR is very large. Consider breaking down the PR into smaller pieces. labels Oct 3, 2024
Copy link

github-actions bot commented Oct 3, 2024

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

Copy link

github-actions bot commented Oct 3, 2024

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@GUIDINGLI GUIDINGLI self-requested a review October 4, 2024 02:03
@GUIDINGLI
Copy link
Contributor

Hold for some time, and I will give you a different way to handle this issue.

@GUIDINGLI
Copy link
Contributor

@fdcavalcanti
Please, provide how to build, how to execute the bin, how to reproduce the issue.

Copy link
Contributor

@GUIDINGLI GUIDINGLI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You must give a Qemu ENV which can reproduce this issue

@fdcavalcanti
Copy link
Contributor Author

Hi @GUIDINGLI. I'll check if QEMU can reproduce this.

Meanwhile, a quick way of viewing the errors:

  • Build setup: ./tools/configure.sh esp32-devkitc:sta_softap
  • Set CONFIG_DEBUG_FEATURES, CONFIG_DEBUG_NET, CONFIG_DEBUG_NET_ERROR, CONFIG_DEBUG_NET_WARN, CONFIG_DEBUG_WIRELESS, CONFIG_DEBUG_WIRELESS_ERROR, CONFIG_DEBUG_WIRELESS_WARN
  • Build and flash.

Run wapi scan wlan0 a couple of times and it stops responding.

@fdcavalcanti
Copy link
Contributor Author

Hold for some time, and I will give you a different way to handle this issue.

What's your plan here?

@GUIDINGLI
Copy link
Contributor

Hold for some time, and I will give you a different way to handle this issue.

What's your plan here?

Sorry, don't have esp32 env current now.
Can reproduce on Qemu? If so, that will be very helpful for this issue

@GUIDINGLI
Copy link
Contributor

@fdcavalcanti

Please see:
#13841

The rootcause is not mq's refactor, but there already has a bug on esp32 time calculation.
One suggestion, if you meet a question, try to fix the bug first time.

@GUIDINGLI
Copy link
Contributor

@fdcavalcanti
You can close this PR now.

@GUIDINGLI GUIDINGLI closed this Oct 6, 2024
@fdcavalcanti
Copy link
Contributor Author

@GUIDINGLI thank you for the fix.
I hope you did not take this personally. Sorry about the confusion around this issue.

@fdcavalcanti
Copy link
Contributor Author

Please let's stand by on this.
Don't know what gave the impression that PR should be merged but the proposed fix did not work and we have WiFi still failing across all devices. Still looking into it.

@xiaoxiang781216
Copy link
Contributor

@GUIDINGLI thank you for the fix.

Sorry, from this reply, I think you verify that #13841 fix your problem.

@GUIDINGLI
Copy link
Contributor

Please let's stand by on this. Don't know what gave the impression that PR should be merged but the proposed fix did not work and we have WiFi still failing across all devices. Still looking into it.

I have reproduce your issue through a real esp32.
compile:
boards/risc-v/esp32c3/esp32c3-generic/configs/wifi
make flash

Run wapi scan wlan0 cmd a couple of times and it stops responding.

Then add patch #13841, and re-compile a re-run.
The problem was gone.

If you still have questions, you can deep into the program to find the rootcause.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: CI Area: OS Components OS Components issues Area: Tooling Size: XL The size of the change in this PR is very large. Consider breaking down the PR into smaller pieces.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants