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

Sending big blocks of data triggers async_tcp task watchdog #121

Closed
detmot5 opened this issue Aug 4, 2021 · 8 comments · May be fixed by #122
Closed

Sending big blocks of data triggers async_tcp task watchdog #121

detmot5 opened this issue Aug 4, 2021 · 8 comments · May be fixed by #122
Labels
stale

Comments

@detmot5
Copy link

detmot5 commented Aug 4, 2021

Hey,
When serving larger files (like 250KB) i got problem with triggering wdt in async_tcp task. After disabling it website still failed to load.
After some research I found source of the problem - too small size of _async_queue. Changing it to 256 or 512 solves my issue.

static inline bool _init_async_event_queue(){
    if(!_async_queue){
        _async_queue = xQueueCreate(512, sizeof(lwip_event_packet_t *));
        if(!_async_queue){
            return false;
        }
    }
    return true;
}

It worked for me.
@mkfrey mkfrey mentioned this issue Aug 16, 2021
Open
@altmannmarcelo altmannmarcelo mentioned this issue Sep 2, 2021
Closed
@nietaki
Copy link

nietaki commented Sep 29, 2021

Can confirm this helps for me as well - left a comment on the PR how to use the changed code from a platformio project.

@proddy proddy mentioned this issue Nov 5, 2021
Closed
@proddy
Copy link

proddy commented Nov 6, 2021

And this also fixed my issues when serving large files via ESPAsyncWebServer - so many thanks for posting this @detmot5 !

@proddy proddy mentioned this issue Nov 6, 2021
Closed
@zekageri
Copy link

zekageri commented Nov 8, 2021

Where is _async_queue defined?

@proddy
Copy link

proddy commented Nov 8, 2021

Where is _async_queue defined?

what do you mean? There's only one .cpp file ;)

@zekageri
Copy link

zekageri commented Nov 8, 2021

Oh. I got a tweaked version of the AsyncTCP lib by @yubox-node-org.
There is no _async_queue in there. :D

@stale
Copy link

stale bot commented Mar 30, 2022

[STALE_SET] This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 14 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Mar 30, 2022
@stale
Copy link

stale bot commented Apr 16, 2022

[STALE_DEL] This stale issue has been automatically closed. Thank you for your contributions.

@stale stale bot closed this as completed Apr 16, 2022
@infwingit
Copy link

You helped me a lot , BIG THANKS !!!

FerdinandvHagen added a commit to FerdinandvHagen/AsyncTCP that referenced this issue Nov 27, 2022
@FerdinandvHagen
Increase queue size
8a4b550 
From me-no-dev#121
@peff74 peff74 mentioned this issue May 15, 2023
Open
@tbnobody tbnobody mentioned this issue Apr 2, 2024
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
stale
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Make event queue size adjustable via Kconfig mkfrey/AsyncTCP
5 participants
@nietaki @proddy @zekageri @detmot5 @infwingit