Skip to content
This repository has been archived by the owner on Jan 21, 2025. It is now read-only.

(fix) TWDT / deadlock when sending chunk response where chunk are lon… #31

Closed
wants to merge 1 commit into from

Conversation

mathieucarbou
Copy link
Owner

…g to compute (like SD Card reading)

Fix from @Levak
Ref: mathieucarbou/ESPAsyncWebServer#165 (comment)

@Levak
Copy link

Levak commented Dec 10, 2024

Well, you were faster than me :)
I was trying to test with the full application with SD, just found few bugs in it (not AsyncTCP, don't worry) :D
For instance, I am getting another Watchdog timeout BUT NO WORRY, this time, I sent 2 parallel SD listing of 700ish files and I see the time of the handler is rising up to 5 seconds before crashing. I would assume it's reading the SD that is that slow, for now.

@mathieucarbou
Copy link
Owner Author

Well, you were faster than me :) I was trying to test with the full application with SD, just found few bugs in it (not AsyncTCP, don't worry) :D For instance, I am getting another Watchdog timeout BUT NO WORRY, this time, I sent 2 parallel SD listing of 700ish files and I see the time of the handler is rising up to 5 seconds before crashing. I would assume it's reading the SD that is that slow, for now.

Ah I was going to leave a message in the existing discussion. I will lock it in order to not spread ourselves every where ;-)

I've opened it to run some perf test and even if SSE is 12% faster, HTTP does not support concurrent requests anymore with the fix. I don't know why yet but spawning 10 workers with autocanon for 20 sec, I only reach 1 req / sec.

Repository owner locked and limited conversation to collaborators Dec 10, 2024
@mathieucarbou
Copy link
Owner Author

Discussion here: mathieucarbou/ESPAsyncWebServer#165

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants