Re: lwip 1.3.1 testing

[Sergei Gavrikov <sergei.gavrikov@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 842 bytes --]

On Wed, Aug 26, 2009 at 08:38:42AM +0200, Simon Kallweit wrote:
> Sergei Gavrikov wrote:
>> On Tue, Aug 25, 2009 at 03:08:20PM +0300, Sergei Gavrikov wrote:
>>> On Tue, Aug 25, 2009 at 09:41:13AM +0200, Simon Kallweit wrote:
>>>> Hi
>>>>
>>>> I have done a new lwip test release with the following changes:
>>>>
>>>> * incorporated Sergei's patch (ctrl-c support)
>>>> * added an lwip_eth_simple and lwip_eth_sequential template
>>>>
>>>> The new test release can be downloaded from   
>>>> http://download.westlicht.ch/lwip-20090825.tar.gz

Hi

Simon, ya 2 cents. I found that the eth_tread's stack size is redefined
in sequential.c:101. First, that stack size is defined in the package's
header via CDL option. It seems the redefinition in sequetial.c can be
quite removed, otherwise the CDL value (eth stack size) wont be applied.

Sergei

[-- Attachment #2: sequential.c.patch --]
[-- Type: text/x-diff, Size: 481 bytes --]

diff -r bed5605f7e6d net/lwip_tcpip/current/src/ecos/sequential.c
--- a/net/lwip_tcpip/current/src/ecos/sequential.c	Tue Aug 25 18:32:48 2009 +0300
+++ b/net/lwip_tcpip/current/src/ecos/sequential.c	Wed Aug 26 23:28:54 2009 +0300
@@ -98,8 +98,6 @@
 CYG_HAL_TABLE_BEGIN(__NETDEVTAB__, netdev);
 CYG_HAL_TABLE_END(__NETDEVTAB_END__, netdev);
 
-#define ETH_THREAD_STACK_SIZE CYGNUM_HAL_STACK_SIZE_TYPICAL
-
 static sys_thread_t eth_thread_handle;
 static cyg_sem_t eth_thread_sem;