[Bug 1001344] Allow nc_test_slave for lwIP to compile for targets with 128KiB RAM.

[bugzilla-daemon at bugs dot ecos dot sourceware dot org]

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

Ilija Kocho <ilijak@siva.com.mk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #1759|0                           |1
        is obsolete|                            |

--- Comment #23 from Ilija Kocho <ilijak@siva.com.mk> 2012-05-17 16:53:30 BST ---
Created an attachment (id=1760)
 --> (http://bugs.ecos.sourceware.org/attachment.cgi?id=1760)
nc_test_slave_120512.diff

(In reply to comment #22)
> The minimum stack for Cortex-m does seem quite high - evidently the worst case
> for interrupt nesting can affect things badly.
> 

In general case it might but in this test we only have 2 interrupt sources:
systick (or equivalent) and Ethernet card. If Ethernet card starts nesting
interrupts then it ought to crash.

> However other architectures do consider the minimum to be a, er, more minimal
> minimum (if you see what I mean). So in practice I would add on maybe
> 4*CYGNUM_HAL_STACK_FRAME_SIZE, and hopefully that won't make things too large
> on Kinetis.

This extra stack space isn't a problem for Kinetis. I added it.
By the way I have cleared some compiler warnings (or should I do it in separate
bug?) ChangeLog will be updated accordingly prior to commit.

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