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

samples: sockets: big_http_download: Increase stack size by 128 bytes #23487

Merged

Conversation

pfalcon
Copy link
Contributor

@pfalcon pfalcon commented Mar 16, 2020

Without it, the sample crashes for qemu_x86 with:

[00:00:03.080,000] os: Page fault at address 0x7275632f (error code 0x10)
[00:00:03.080,000] os: Linear address not present in page tables
[00:00:03.080,000] os: PDPTE: Non-present
...

Increase by reasonable, but small amount, to keep watching stack usage
growth trends in Zephyr.

Signed-off-by: Paul Sokolovsky paul.sokolovsky@linaro.org

@pfalcon pfalcon added area: Networking Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc. labels Mar 16, 2020
@zephyrbot zephyrbot added the area: Samples Samples label Mar 16, 2020
@zephyrbot
Copy link
Collaborator

zephyrbot commented Mar 16, 2020

All checks are passing now.

Tip: The bot edits this comment instead of posting a new one, so you can check the comment's history to see earlier messages.

@pfalcon
Copy link
Contributor Author

pfalcon commented Mar 16, 2020

The previous bump was just in the beginning of January: b3ef46e . Not sure who would be to contact on that, so pinging @nashif and @andrewboie regarding the situation that Zephyr's stack usage grows constantly, visible with naked eye.

@pfalcon pfalcon mentioned this pull request Mar 16, 2020
Without it, the sample crashes for qemu_x86 with:

[00:00:03.080,000] <err> os: Page fault at address 0x7275632f
  (error code 0x10)
[00:00:03.080,000] <err> os: Linear address not present in page tables
[00:00:03.080,000] <err> os: PDPTE: Non-present
...

Increase by reasonable, but small amount, to keep watching stack usage
growth trends in Zephyr.

Signed-off-by: Paul Sokolovsky <paul.sokolovsky@linaro.org>
@pfalcon pfalcon force-pushed the big_http_download-bump-stack branch from 7b078a1 to a421b56 Compare March 16, 2020 12:21
@jukkar jukkar merged commit bbb4355 into zephyrproject-rtos:master Mar 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Networking area: Samples Samples Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants