lwIP - A Lightweight TCP/IP stack - Bugs: bug #48953, mdns has rather high stack usage...
You are not allowed to post comments on this tracker with your current authentication level.
bug #48953: mdns has rather high stack usage (sizeof(struct mdns_domain)*2)
Submitter: | Simon Goldschmidt <goldsimon> | ||
Submitted: | Tue 30 Aug 2016 07:29:30 PM UTC | ||
Category: | None | Severity: | 2 - Minor |
Item Group: | Change Request | Status: | None |
Privacy: | Public | Assigned to: | None |
Open/Closed: | Open | Planned Release: | None |
lwIP version: | git head |
Thu 04 Jan 2018 07:07:46 AM UTC, comment #5: |
Simon Goldschmidt <goldsimon>![]() |
Fri 10 Mar 2017 10:13:17 PM UTC, comment #4: mdns_readname_loop should be fixed (one copy avoided), but the bigger problem of allocating 2 mdns_domain structs on the stack remains. For some parts of the code, allocating those two opne after the other should work, but for some, it's not as easy...
|
Simon Goldschmidt <goldsimon>![]() |
Thu 15 Dec 2016 09:48:14 PM UTC, comment #3: 1k might be a little much, but struct mdns_domain (260 bytes) doesn't make it better. I mean if we really need this memory, so be it (rather allocate it on the stack where it's reusable for other code than allocating it seperately), but if we could avoid it it would be better :-)
|
Simon Goldschmidt <goldsimon>![]() |
Mon 19 Sep 2016 05:28:48 AM UTC, comment #2: On my project, in order to get mDNS to work I had to add about 1k extra stack space to both the TCP/IP thread and my Ethernet input thread (the latter probably due to LWIP_TCPIP_CORE_LOCKING_INPUT). |
Daniel Elstner <danielk> |
Wed 31 Aug 2016 08:44:30 AM UTC, comment #1: This is mentioned in the documentation that it uses some stack.
|
Erik Ekman <yarrick>![]() |
Tue 30 Aug 2016 07:29:30 PM UTC, original submission:
mdns_readname_loop() decodes a DNS name (this is kind of duplicate to what dns.c does, already, so might be merged?).
|
Simon Goldschmidt <goldsimon>![]() |
No files currently attached
Depends on the following items: None found
Items that depend on this one: None found
There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.
Follow 2 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2017-03-10 | goldsimon | Severity | 3 - Normal | ![]() |
2 - Minor |
Summary | mdns_readname_loop() allocates rather big array on the stack | ![]() |
mdns has rather high stack usage (sizeof(struct mdns_domain)*2) |
See also bug #52763 and bug #52770 for discussions related to this.