New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mail test: fix out of memory problem for NUCLEO_F070RB #5376

Merged
merged 1 commit into from Oct 30, 2017

Conversation

Projects
None yet
6 participants
@maciejbocianski
Member

maciejbocianski commented Oct 25, 2017

Description

This test cause out of memory error ("Operator new[] out of memory") when run on NUCLEO_F070RB with GCC_ARM compiler

Thread stack was changed from 384B to 320B to reduce heap usage

Status

READY

Migrations

NO

@0xc0170 0xc0170 requested review from bulislaw and jeromecoutant Oct 25, 2017

@0xc0170 0xc0170 self-assigned this Oct 25, 2017

@0xc0170 0xc0170 removed their assignment Oct 25, 2017

@maciejbocianski maciejbocianski force-pushed the maciejbocianski:mail_mem_fix branch to adb3dac Oct 26, 2017

@bulislaw

Fine if that passes testing on all boards.

@bulislaw

This comment has been minimized.

Member

bulislaw commented Oct 26, 2017

/morph build

@mbed-ci

This comment has been minimized.

mbed-ci commented Oct 26, 2017

Build : SUCCESS

Build number : 351
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5376/

Triggering tests

/morph test
/morph uvisor-test

@mbed-ci

This comment has been minimized.

@jeromecoutant

This comment has been minimized.

Contributor

jeromecoutant commented Oct 30, 2017

Approved !

| NUCLEO_F070RB-GCC_ARM | NUCLEO_F070RB | tests-mbedmicro-rtos-mbed-mail | OK | 52.67 | default |
| NUCLEO_F072RB-GCC_ARM | NUCLEO_F072RB | tests-mbedmicro-rtos-mbed-mail | OK | 19.84 | default |
| NUCLEO_L073RZ-GCC_ARM | NUCLEO_L073RZ | tests-mbedmicro-rtos-mbed-mail | OK | 23.43 | default |

@theotherjimmy theotherjimmy merged commit 3749f2d into ARMmbed:master Oct 30, 2017

5 checks passed

AWS-CI uVisor Build & Test Success
Details
ci-morph-build build completed
Details
ci-morph-test test completed
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment