LWIP 2.1.2 tcpip thread stack overflow fix.

[tymoteuszblochmobica]

Description

Applied missing LWIP patch to PPP/utils.c
"Major Refactoring & extensions" commited on May 23, 201 by hasnainvirk
According to this patch PPP logger module member "ppp_logit" consumes only 256 byte buffer on stack instead of 1024 like original LWIP code from Savanah repo does.

LWIP 2.1.2 tcpip thread stack is restored to 1200 bytes.

Pull request type

[x ] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

Reviewers

@SeppoTakalo
@mikaleppanen
@kjbracey-arm
@AriParkkila
@kimlep01

Release Notes

[AriParkkila]

It seems that 1600 would be enough e.g. for mbed test -c -m MTB_MTS_DRAGONFLY -n tests-network-interface --app-config=tools/test_configs/CellularInterface.json -vv, but I'm guessing that's still not the worst case..?

[tymoteuszblochmobica]

Im still checking new PPP implementation to verify why more stack is needed. So now 1400 seems ok.

[ciarmcom]

@tymoteuszblochmobica, thank you for your changes.
@mikaleppanen @kjbracey-arm @kimlep01 @AriParkkila @SeppoTakalo @ARMmbed/mbed-os-ipcore @ARMmbed/mbed-os-maintainers please review.

[AriParkkila]

@tymoteuszblochmobica I don't think 1400 is enough, could you check my comment above.

[0xc0170]

@tymoteuszblochmobica I don't think 1400 is enough, could you check my comment above.

@tymoteuszblochmobica Please review

[tymoteuszblochmobica]

Applied missing LWIP patch to PPP/utils.c
"Major Refactoring & extensions" commited on May 23, 201 by hasnainvirk
According to this patch PPP logger module member "ppp_logit" consumes only 256 byte buffer on stack instead of 1024 like original LWIP code from Savanah repo does.

LWIP 2.1.2 tcpip thread stack is restored to 1200 bytes.

[0xc0170]

CI started

[mbed-ci]

Test run: SUCCESS

Summary: 11 of 11 test jobs passed
Build number : 1
Build artifacts

[adbridge]

This is a 5.13 feature