This is the mail archive of the
ecos-patches@sourceware.org
mailing list for the eCos project.
[Bug 1001344] Allow nc_test_slave for lwIP to compile for targets with 128KiB RAM.
- From: bugzilla-daemon at bugs dot ecos dot sourceware dot org
- To: ecos-patches at ecos dot sourceware dot org
- Date: Tue, 27 Sep 2011 06:45:29 +0100
- Subject: [Bug 1001344] Allow nc_test_slave for lwIP to compile for targets with 128KiB RAM.
- Auto-submitted: auto-generated
- References: <bug-1001344-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=1001344
--- Comment #3 from Sergei Gavrikov <sergei.gavrikov@gmail.com> 2011-09-27 06:45:27 BST ---
(In reply to comment #2)
> (In reply to comment #1)
> > (In reply to comment #0)
> > > Created an attachment (id=1377)
--> (http://bugs.ecos.sourceware.org/attachment.cgi?id=1377) [details] Reduce
stack usage of
> > > nc_test_slave
>
> >
> > Hi Ilija,
> >
> > I have one **minor** remark regarding attachment 1377 [details]. I
> > would use ECHO_STACK_SIZE in the patch instead proposed name
> > (TMAIN_STACK_SIZE), IMO, such "adjusted" names would look nicer
>
> Thanks for comment Sergei.
>
> I selected stack names to reflect thread names. Regarding your
> proposal I assume that you mean MAIN_STACK_SIZE => ECHO_STACK_SIZE
> rather than TMAIN..., aren't you?
Hmm. I see
@@ -748,7 +750,7 @@
0, // entry parameter
"socket echo test", // Name
&stack[0], // Stack
- STACK_SIZE, // Size
+ TMAIN_STACK_SIZE, // Size
&thread_handle, // Handle
&thread_data // Thread data structure
);
So, I've meant s/TMAIN/ECHO/.
--
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.