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

Offload ThingSet processing to dedicated thread #103

Open
martinjaeger opened this issue Aug 18, 2020 · 0 comments
Open

Offload ThingSet processing to dedicated thread #103

martinjaeger opened this issue Aug 18, 2020 · 0 comments

Comments

@martinjaeger
Copy link
Member

The processing of ThingSet requests that result in a large amount of returned data (e.g. ?conf and ?rec) requires considerable amount of RAM (around 1kB). Currently, each thread that processes ThingSet messages (serial, CAN at the moment, GSM, LoRa in future extensions) needs to reserve that amount of stack memory.

Suggested improvement: Offload the processing into a dedicated processing thread that is called asynchronously from other threads that send and receive the data via comms interfaces.

martinjaeger added a commit that referenced this issue Aug 18, 2020
Depending on the amount of data nodes, calling ?rec caused a stack overflow.
Larger stack memory solves the issue, for further improvement see also #103.
martinjaeger added a commit that referenced this issue Aug 18, 2020
Depending on the amount of data nodes, calling ?rec caused a stack overflow.
Larger stack memory solves the issue, for further improvement see also #103.
martinjaeger added a commit that referenced this issue Oct 26, 2020
Depending on the amount of data nodes, calling ?rec caused a stack overflow.
Larger stack memory solves the issue, for further improvement see also #103.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant