Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[OIS] CI improvments and stabilization (#25866)
* [OIS] PlatformManagerImpl improvements Improve CHIP stack initialization in PlatformManager - add stack initialization flag. Check initialization stack state in event loop functions. Initialize the CHIP stack in timer starting and event posting. Dispatch all events in queue. Signed-off-by: ATmobica <artur.tynecki@arm.com> * [OIS] Improve unit tests Remove CHIP stack initialization in unit-tests application main. Restore CHIP stack init and shutdown in unit tests context. Skip TestEventLogging due to lack of real-time clock support. Skip TestTLVPacketBufferBackingStore due to LwIP buffers usage in SystemPacketBuffer. Signed-off-by: ATmobica <artur.tynecki@arm.com> * [Fix] Fix LwIP structure initialization in unit-tests Zero-initialization of ip6_addr_t structure in TestInetAddress. That protects against error that may occur when the memory is aligned to multiples of 4 and LWIP_IPV6_SCOPES is enabled. In this case, the ip6_addr_t contains 16 bytes array of address and 1 byte of zone but the structure length is 20. If we don't set the initial value for ip_addr_1 and ip_addr_2 the memcmp returns mismatch. Signed-off-by: ATmobica <artur.tynecki@arm.com> * [Fix] Fix PlatformMgr unit test - event loop stopping Add sleep after stopping event loop task. In OIS platform implementations the event loop thread is self-terminating. We need time to process the stopping event inside event loop. Signed-off-by: ATmobica <artur.tynecki@arm.com> * [OIS] Improve build/run script Update Matter Python builder. Check if FVP process run properly and wait for the connection port log. Validate if FVP starts properly. Wait for the FVP stop. Move Pigweed env activation to a common point. Create supported apps list and use it. Fixing the optimization issue for mcu-driver-hal target in the non-debug build. Signed-off-by: ATmobica <artur.tynecki@arm.com> * [OIS] Improve internal cmake function Use target name from function argument instead of APP_TARGET variable in linker.cmake and sdk.cmake. Add custom command to sign the TF-M non-secure image. Signed-off-by: ATmobica <artur.tynecki@arm.com> * [OIS] Integration test suite improvements Common: Improve commissioning function to return commission error. Check if FVP process run properly and wait for the connection port log. Set verbose for wait_for_output() function. The default value is false - not print waiting log. Fix telnet connection readline function, buffering output date and processes only entire lines (ending with "\n"). Increase wait for lock-app application run timeout. Increase wait for setup QR code timeout. Add "verbose" state setting in the device class. Improve read/write ZCL attributes and send ZCL command functions. Improve controller device - add controllerConfig fixture, the dictionary with settings, remove persistent storage file at the end, change fixture scope to session. Lock test: Apply common changes. Shell test: Change the shell prompt to "Enter command:\r\n". It should be terminated by a newline character. Add waiting for the prompt before sending the shell command. Update file header. Signed-off-by: ATmobica <artur.tynecki@arm.com> * [OIS] Extend RAM size for TF-M app For unit-test applications, 1MB ISRAM0 is not enough. Extend it by adding ISRAM1, then 2MB of memory is available. Change TF-M linker script and AN552 target configuration. Signed-off-by: ATmobica <artur.tynecki@arm.com> * [OIS] Add TF-M support to unit-tests Adapt unit-tests project to TF-M support. Change unit-tests debug Vscode task. Increase application run timeout for unit-tests integration test. Signed-off-by: ATmobica <artur.tynecki@arm.com> * [OIS] Move unit-test validation to pyTest integration tests Create unit-tests Python integration test that checks and control time execution of OIS unit-tests. Change run script to only run single test on FVP. Add testing of unit-test apps to test script and Vscode tasks and CI. Signed-off-by: ATmobica <artur.tynecki@arm.com> * [OIS] Improve LwIP configuration Open IoT SDK version udpate. Move LwIP opts settings to sdk.cmake. As default the config/openiotsdk/lwip/user_lwipopts.h file is used. You can provide your settings with define LWIP_PROJECT_OPTS_DIR with path to custom file. Add CONFIG_CHIP_OPEN_IOT_SDK_LWIP_DEBUG variable to enable LwIP debug logs. It's disabled by default. Add LwIP debug logs option to build script and VScode tasks. Create common LwIP options for examples and unit-tests. Signed-off-by: ATmobica <artur.tynecki@arm.com> * [OIS] TF-M settings improvements Enable TFM fault trace support - application hangs instead of reset in case of fault. Force secure partition isolation to level 1 - this speed up processing which allows slow Github runners to pass the tests. Signed-off-by: Vincent Coubard <vincent.coubard@arm.com> * [OIS] Rework application startup and serial output Add startup library This library setup the C runtime: - Initialize stdio - Start the kernel - Create an run the main thread The main thread setups TFM and the locks used by the libc before entering main. Additionaly, lock support has been added to malloc/free, operator new and delete have been overriden to print errors and sbrk is overriden to print heap segment overallocation. The startup also rework stdio: - Override iotsdk-serial-retarget - Lock stream at the libc level - Buffer bytes on the RX path Add option to set serial baud rate from Cmake level. Decrease serial baudrate for shell examples to 9600. Create ois_logging_init() function and move setting log filter for non-debug mode to it. Co-authored-by: ATmobica <artur.tynecki@arm.com> Signed-off-by: Vincent Coubard <vincent.coubard@arm.com> * [OIS] Fix TDB key-value store string writing Add correct condition for nullptr string argument. Signed-off-by: ATmobica <artur.tynecki@arm.com> * [OIS] Restore testing steps in the CI workflow Adjust timeouts for job and steps. Signed-off-by: ATmobica <artur.tynecki@arm.com> --------- Signed-off-by: ATmobica <artur.tynecki@arm.com> Signed-off-by: Vincent Coubard <vincent.coubard@arm.com> Co-authored-by: Vincent Coubard <vincent.coubard@arm.com>
- Loading branch information
