This is the mail archive of the
ecos-patches@sourceware.org
mailing list for the eCos project.
[Bug 1001116] Instruct lwIP to use special section for buffers and heap.
- From: bugzilla-daemon at bugs dot ecos dot sourceware dot org
- To: ecos-patches at ecos dot sourceware dot org
- Date: Sat, 29 Jan 2011 19:32:24 +0000
- Subject: [Bug 1001116] Instruct lwIP to use special section for buffers and heap.
- Auto-submitted: auto-generated
- References: <bug-1001116-104@http.bugs.ecos.sourceware.org/>
Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001116
--- Comment #14 from Sergei Gavrikov <sergei.gavrikov@gmail.com> 2011-01-29 19:32:20 GMT ---
Ilija, at the least I would stop on this
- lwip_net.cdl:
cdl_option CYGSEM_LWIP_MEM_SECTION {
...
flavor bool
default_value 0
active_if CYGDAT_LWIP_MEM_SECTION_NAME
...
}
- lwipopts.h:
#if CYGSEM_LWIP_MEM_SECTION
#include <cyg/infra/cyg_type.h>
#define MEM_SECTION CYGBLD_ATTRIB_SECTION(CYGDAT_LWIP_MEM_SECTION_NAME)
#else
#define MEM_SECTION
#endif
- mem.c, memp.c: your Attachment 1094
My comments on lwiP config:
- cdl option-semaphore CYGSEM_LWIP_MEM_SECTION is active if other package
provides CYGDAT_LWIP_MEM_SECTION_NAME (string) *and* if user select it.
(It will be "grayed" for the most eCos targets in `configtool' and this
is okay).
- above is quite enough to expand the lwIP package by applying your patch
for lwiP sources and to *close* this bug.
- if you want to force CYGSEM_LWIP_MEM_SECTION, you can distribute one
ECM file to import, for example, `lwip_net.ecm' under your HAL `misc'
directory which will do it (enable the option), and users will import
all proper settings for your target in cases if they need lwIP stack.
My thoughts about HAL config:
It seems to me you can enter some generic CDL component, for example,
component CYG{MEM, or HWR}_HAL_..._SECTION where you can manage also
generic data option (HAL, !LWIP) CYGDAT_HAL_..._SECTION_NAME. Why is
it? I think your idea to utilize all memory can be useful for other
applications too (not only lwIP package). - Right?
Now, back to our distressful CYGDAT_LWIP_NET_SECTION_NAME data option.
This option in fact should be equal CYGDAT_HAL_..._SECTION_NAME data,
if lwIP package has been added (active_if CYGPKG_NET_LWIP).
Just now I would not refer to Bug 1001114 here again if we just want to
get new lwIP option to manage the placement of memory pools, in fact, to
have a way for new definition, MEM_SECTION. Let's apply it and go ahead.
What do you/others think?
--
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.