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

Update module civetweb (bug fixes and increased stack size requirement) #33520

Closed
gudnimg opened this issue Mar 20, 2021 · 1 comment
Closed
Labels
Enhancement Changes/Updates/Additions to existing features

Comments

@gudnimg
Copy link
Contributor

gudnimg commented Mar 20, 2021

Proposal

The current revision in west.yml refers to commit zephyrproject-rtos/civetweb@e6903b8

revision: e6903b80c09d17cd1a8bb32e40080005558aad29

As I understand it, Zephyr is directly using this repo: https://github.com/zephyrproject-rtos/civetweb

I was trying to help with an issue related to Civetweb #33516 and noticed the Zephyr version pulled by West is out of date. One commit of interest: zephyrproject-rtos/civetweb@3565451

The stack size increases a lot and may rule out some boards.


Example:

When compiling sample samples/net/civetweb/http_server the stack size increases by 24576 bytes. For a Nucleo F429ZI board this represents an increase of 12.5%.

Command: west build samples/net/civetweb/http_server -b nucleo_f429zi

Memory region         Used Size  Region Size  %age Used
           FLASH:      121408 B         2 MB      5.79%
             CCM:          0 GB        64 KB      0.00%
            SRAM:       95199 B       192 KB     48.42%
        IDT_LIST:          0 GB         2 KB      0.00%

After zephyrproject-rtos/civetweb@3565451:

Memory region         Used Size  Region Size  %age Used
           FLASH:      123392 B         2 MB      5.88%
             CCM:          0 GB        64 KB      0.00%
            SRAM:      119775 B       192 KB     60.92%
        IDT_LIST:          0 GB         2 KB      0.00%

Release notes: https://github.com/civetweb/civetweb/blob/master/RELEASE_NOTES.md


How to update Zephyr's Civetweb?

To update the revision I found these steps:

  1. Copy the commit number from https://github.com/zephyrproject-rtos/civetweb you want to use as HEAD
  2. Paste it in line
    revision: e6903b80c09d17cd1a8bb32e40080005558aad29
  3. Call west update to update Civetweb locally.
@gudnimg gudnimg added the Enhancement Changes/Updates/Additions to existing features label Mar 20, 2021
@gudnimg gudnimg changed the title Update module civetweb from v1.11 to v1.14 Update module civetweb Mar 20, 2021
@gudnimg gudnimg changed the title Update module civetweb Update module civetweb (bug fixes and increased stack size requirement) Mar 20, 2021
@gmarull
Copy link
Member

gmarull commented Aug 11, 2022

civetweb is no longer part of Zephyr, closing

@gmarull gmarull closed this as completed Aug 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Changes/Updates/Additions to existing features
Projects
None yet
Development

No branches or pull requests

2 participants