Skip to content

Release candidate for mbed-os-5.11.2 #9370

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

Merged
merged 135 commits into from
Jan 16, 2019
Merged

Release candidate for mbed-os-5.11.2 #9370

merged 135 commits into from
Jan 16, 2019

Conversation

adbridge
Copy link
Contributor

No description provided.

Olli-Pekka Puolitaival and others added 30 commits January 14, 2019 13:06
Issue was seen with below example
EventQueue q1;
EventQueue q2;

void main() {
while( true ) {
q1.chain( &q2 ); // Chain q2 to q1
q1.chain( NULL ); // Remove chain from q1
//This second step should free the memory from the chained q2 event.
}
}

Memory allocated from q1 slab was freed for q2, which will result in
memory leak.
When accessing non-secure ram and flash r1 was actually used by the calling function.
Change to a callee saved register.
Use r3 instead of r4

Co-Authored-By: orenc17 <[email protected]>
Space needed between 2 string literals
    [Error] fopen.cpp@204,97: [Pe2486]: user-defined literal operator not found
`size` is part of std library since c++17, and results in build
errors for IAR 8.x.
https://en.cppreference.com/w/cpp/iterator/size
For binary compatibility with ARMC5, use the hard ABI variant whenever
we have FP hardware - this is ARMC5's default behaviour, which we do not
override.

Cortex-M4F was already using hard; this brings M7F and M33F into line.
Replace Mutex with PlatformMutex which supports RTOS-absent
Configure RTS pin outout to active low when RTS is disabled. This allows the peer to be able
to send data if its CTS is still enabled.
This PR addresses the issue of #8903.
A new boolean variable 'debug' is added to turn off the display of debugging message (by default). It can be turned on when needed.

Description
1.	Add new boolean parameter ‘debug’ in function ‘RTWInterface’ in file "RTWInterface.h".
2.	Add conditional statement to enable/disable variable ‘GlobalDebugEnable’ in file "RTWInterface.h" to control whether debugging message should be displayed.
Change the use of the local static function `mbedtls_zeroize()` to
the Mbed TLS platform function `mbedtls_platform_zeroize()`.
In case of unusing RTOS, IRQ handler executes "while(1)" and it causes a program freeze.
Therefore, I revised this processing. Also I added the heap setting processing and set the align to 8 byte.
In case of unusing RTOS, IRQ handler executes "while(1)" and it causes a program freeze.
Therefore, I revised this processing.
In case of unusing RTOS, there is no processing against IRQ handler and it causes a linker error.
Therefore, I added this processing with WEAK attribute. Also I added cmain.S file at cmsis/TARGET_CORTEX_A folder.
@0xc0170
Copy link
Contributor

0xc0170 commented Jan 14, 2019

My local test run

pytest tools\test
============================= test session starts =============================
platform win32 -- Python 2.7.15, pytest-3.3.2, py-1.5.2, pluggy-0.6.0
rootdir: C:\Code\mbed-os, inifile:
plugins: hypothesis-3.44.16
collected 162 items

tools\test\detect_targets_test.py .....                                  [  3%]
tools\test\build_api\build_api_test.py ...FF..                           [  7%]
tools\test\config\config_test.py ....................................... [ 31%]
.                                                                        [ 32%]
tools\test\memap\memap_test.py ........................                  [ 46%]
tools\test\memap\parse_test.py .....                                     [ 50%]
tools\test\run_icetea\run_icetea_test.py .......                         [ 54%]
tools\test\run_icetea\run_icetea_unit_test.py ..............             [ 62%]
tools\test\spm\test_generate_partition_code.py ......................... [ 78%]
....................                                                     [ 90%]
tools\test\targets\target_test.py ...                                    [ 92%]
tools\test\toolchains\api_test.py .........                              [ 98%]
tools\test\toolchains\arm_support_test.py ...                            [100%]

Updated hypothesis and others pip install -U pytest pylint hypothesis mock coverage coveralls and got different test results:

pytest tools\test
============================= test session starts =============================
platform win32 -- Python 2.7.15, pytest-4.1.1, py-1.5.2, pluggy-0.8.1
hypothesis profile 'default' -> database=DirectoryBasedExampleDatabase('C:\\Code\\mbed-os\\.hypothesis\\examples')
rootdir: C:\Code\mbed-os, inifile:
plugins: hypothesis-4.0.0
collected 162 items

tools\test\detect_targets_test.py .....                                  [  3%]
tools\test\build_api\build_api_test.py ...FF..                           [  7%]
tools\test\config\config_test.py ....................................... [ 31%]
.                                                                        [ 32%]
tools\test\memap\memap_test.py ........................                  [ 46%]
tools\test\memap\parse_test.py .....                                     [ 50%]
tools\test\run_icetea\run_icetea_test.py .......                         [ 54%]
tools\test\run_icetea\run_icetea_unit_test.py ..............             [ 62%]
tools\test\spm\test_generate_partition_code.py ......................... [ 78%]
....................                                                     [ 90%]
tools\test\targets\target_test.py ...                                    [ 92%]
tools\test\toolchains\api_test.py ...F....F                              [ 98%]
tools\test\toolchains\arm_support_test.py FFF                            [100%]

cc @ARMmbed/mbed-os-tools

@cmonr
Copy link
Contributor

cmonr commented Jan 14, 2019

Note: Fix is now in master (#9371), but we're still waiting on a fix for Cloud Client.

Update to v4.x breaks tools, will need to be addressed separately.
@0xc0170
Copy link
Contributor

0xc0170 commented Jan 15, 2019

Travis fix added to 5.11.2

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 15, 2019

CI started (while we might expect one or two fixes)

@mbed-ci
Copy link

mbed-ci commented Jan 15, 2019

Test run: SUCCESS

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

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 15, 2019

Note, @ARMmbed/mbed-os-storage @JanneKiiskila The 5.11.2 RC is ready , waiting for your +1 that this is good to go

@JanneKiiskila
Copy link
Contributor

JanneKiiskila commented Jan 15, 2019

Ok from our point of view. We have one more issue for urgent fix, but that's probably not going to make this one anyway. (edit: So happy to be wrong!)

@SeppoTakalo
Copy link
Contributor

SeppoTakalo commented Jan 15, 2019

https://jira.arm.com/browse/ONME-4127

MTB_WISE_1530 and MTB_USI_WM_BN_BM_22 Hardfaults when running Mbed OS socket tests. However needs more studying.

This issue seems to be same for all WICED based WiFi boards when running against Arm compiler.
GCC works.

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 15, 2019

We are waiting for @ARMmbed/mbed-os-storage to confirm (one patch still might come)

@adbridge
Copy link
Contributor Author

adbridge commented Jan 15, 2019

https://jira.arm.com/browse/ONME-4127
This issue seems to be same for all WICED based WiFi boards when running against Arm compiler.
GCC works.

As there is a working compiler we would not consider this a blocker for the 5.11.2 release. It can be added to the known issues for the release. @SeppoTakalo could you please drop me an email with a quick summary of the issue that is suitable for putting in the release note as a known issue ?

MBRBlockDevice: When partitioning, clear the rest of first erase unit
@cmonr
Copy link
Contributor

cmonr commented Jan 16, 2019

#9384 merged, and cherry-picked into release-candidate.

@cmonr
Copy link
Contributor

cmonr commented Jan 16, 2019

CI started

@cmonr
Copy link
Contributor

cmonr commented Jan 16, 2019

Note: Travis CI issue is will be resolved with this PR: #9391

Running CI to make sure cherry-picked PR passes with RC, and to give @jannehekkala and team more time to test.

@JanneKiiskila
Copy link
Contributor

Test PRs on-going.

@mbed-ci
Copy link

mbed-ci commented Jan 16, 2019

Test run: SUCCESS

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

@JanneKiiskila
Copy link
Contributor

JanneKiiskila commented Jan 16, 2019

Nucleo F429ZI ethernet doesn't work after firmware update, it can't connect for some reason. @teetak01 has already raised a on bug on this.

2019-01-16T06:41:09.017Z | D1 <<< DutThread: Error occurred : MbedCloudClient::ConnectDnsResolvingFailed
2019-01-16T06:41:09.017Z | D1 <<< DutThread: Error code : 12
2019-01-16T06:41:09.017Z | D1 <<< DutThread: Error details : DnsResolvingFailed

Error ref: ONME-4124. @SeppoTakalo - can we get some runtime for this?

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 16, 2019

Travis fix in

@adbridge
Copy link
Contributor Author

ci started

@mbed-ci
Copy link

mbed-ci commented Jan 16, 2019

Test run: SUCCESS

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

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 16, 2019

5.11.2 is ready 🎉

@cmonr cmonr merged commit a8f0c33 into mbed-os-5.11 Jan 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.