This is the mail archive of the ecos-bugs@sourceware.org mailing list for the eCos project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[Bug 1001116] Instruct lwIP to use special section for buffers andheap.


Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001116

--- Comment #27 from Ilija Kocho <ilijak@siva.com.mk> 2011-01-30 20:11:49 GMT ---
(In reply to comment #26)
> (In reply to comment #25)
> > (In reply to comment #24)
> > > (In reply to comment #22)
> 
> [snip]
> 
> > > > The box will be unchecked until booldata in your HAL (CYGDAT_.*_NAME)
> > > > is not proper section name. However, this was quickly tested only. 
> > > 
> > > I have noticed it, but I allowed to occur it since I have seen similar
> > > behavior in some other CDLs.
> >  
> > Ilija, I have managed this with:
> 
> [snip]
> 
> > But the difficulty to read/understand came back :-) However, it checks
> > that NAME is string and at the least it looks like section name.
> > 
> > > Actually I tried something similar
> > > 
> > >      cdl_option CYGOPT_LWIP_MEM_SECTION {
> > >          ...
> > >          default_value is_active(CYGDAT_LWIP_MEM_SECTION_NAME)
> > >          active_if     CYGDAT_LWIP_MEM_SECTION_NAME
> >  
> > I will try it in CT.
> 
> Ilija, I have tested it and it looks for me like "golden mean".
> 
> > > But I'm not sure if it depends on the order the packages are processed/
> > > applied.  FYI it worked for my test case (select default template then
>               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>               It's important to me and I would stay on it.
> 
> > > add lwip).
> 
> [snip]
> 
> > But I myself have argued for simplicity.
> 
> This is strong argument :-)
> 
> > Can you try it on your target?
>   ^^^^^^^^^^^^^^
>   Please, forget it.
> 
> Well, let's stop on this point of survey. What do you think?

Seems we came to conclusion. As I understand CYGSEM_LWIP_MEM_SECTION_NAME is
not in 2 steps (with CYGHWR_HAL_MEM_SECTION_NAME) but simple as before, only
enclosed in CYGOPT_LWIP_MEM_PLF_SPEC.

Something like this?

--- lwip_net.tcl ---

    cdl_component CYGOPT_LWIP_MEM_PLF_SPEC {
        display         "Platform specific memory setting"
        flavor          none
        no_define  
        cdl_option CYGSEM_LWIP_MEM_SECTION {
            display "Place lwIP buffers and heap in special memory section."
            flavor bool
            default_value 1
            active_if CYGDAT_LWIP_MEM_SECTION_NAME
            description "..."
       }
   }

--- hal...cdl example---

    cdl_option CYGDAT_LWIP_MEM_SECTION_NAME {
        flavor data
        active_if CYGPKG_NET_LWIP
        parent CYGOPT_LWIP_MEM_PLF_SPEC
        default_value { "\".ahb_bss\"" }
        legal_values { "\".ahb_bss\"" "\".ahb_0\"" }
        display "Memory section for lwIP buffers."
        description " ... Sections are provided by LPC17xx platform HAL." 
    }

-- 
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.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]