tests: benchmarks: multicore: Remote GDF switching #20625

nordic-pikr · 2025-02-26T12:12:49Z

Add test cases for fast counter, pwm, spim, uarte instances where global domain frequency is changed from remote application. Use solution for synchronization of host and remote core like in idle_pwm_loopback case.

test_low_level: PR-1912

NordicBuilder · 2025-02-26T12:16:50Z

CI Information

^{To view the history of this post, clich the 'edited' button above}
Build number: 11

Inputs:

Sources:

sdk-nrf: PR head: c769ef86f95c08596a4c5aaa52b3ace89286e36d

more details

sdk-nrf:

PR head: c769ef86f95c08596a4c5aaa52b3ace89286e36d
merge base: 4dfdc18cd66f89855dfa516a750a09cf219cfc97
target head (main): 4dfdc18cd66f89855dfa516a750a09cf219cfc97
Diff

Github labels

Enabled	Name	Description
	ci-disabled	Disable the ci execution
	ci-all-test	Run all of ci, no test spec filtering will be done
	ci-force-downstream	Force execution of downstream even if twister fails
	ci-run-twister	Force run twister
	ci-run-zephyr-twister	Force run zephyr twister

List of changed files detected by CI (20)

tests
│  ├── benchmarks
│  │  ├── multicore
│  │  │  ├── common
│  │  │  │  ├── remote_gdf_switching
│  │  │  │  │  ├── CMakeLists.txt
│  │  │  │  │  ├── boards
│  │  │  │  │  │  │ nrf54h20dk_nrf54h20_cpurad.overlay
│  │  │  │  │  ├── prj.conf
│  │  │  │  │  ├── src
│  │  │  │  │  │  │ main.c
│  │  │  ├── idle_counter
│  │  │  │  ├── Kconfig
│  │  │  │  ├── prj.conf
│  │  │  │  ├── remote
│  │  │  │  │  ├── Kconfig
│  │  │  │  │  │ prj.conf
│  │  │  │  ├── src
│  │  │  │  │  │ main.c
│  │  │  │  ├── sysbuild.cmake
│  │  │  │  │ testcase.yaml
│  │  │  ├── idle_pwm_loopback
│  │  │  │  ├── Kconfig.sysbuild
│  │  │  │  ├── sysbuild.cmake
│  │  │  │  │ testcase.yaml
│  │  │  ├── idle_spim_loopback
│  │  │  │  ├── Kconfig.sysbuild
│  │  │  │  ├── sysbuild.cmake
│  │  │  │  │ testcase.yaml
│  │  │  ├── idle_uarte
│  │  │  │  ├── Kconfig.sysbuild
│  │  │  │  ├── sysbuild.cmake
│  │  │  │  │ testcase.yaml

Outputs:

Toolchain

Version: aedb4c0245
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:aedb4c0245_bece0367df

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

◻️ Toolchain - Skipped: existing toolchain is used
✅ Build twister
- sdk-nrf test count: 143
✅ Integration tests

Disabled integration tests

- desktop52_verification
- doc-internal
- test-fw-nrfconnect-apps
- test-fw-nrfconnect-ble_mesh
- test-fw-nrfconnect-ble_samples
- test-fw-nrfconnect-boot
- test-fw-nrfconnect-chip
- test-fw-nrfconnect-fem
- test-fw-nrfconnect-nfc
- test-fw-nrfconnect-nrf-iot_libmodem-nrf
- test-fw-nrfconnect-nrf-iot_lwm2m
- test-fw-nrfconnect-nrf-iot_mosh
- test-fw-nrfconnect-nrf-iot_positioning
- test-fw-nrfconnect-nrf-iot_samples
- test-fw-nrfconnect-nrf-iot_serial_lte_modem
- test-fw-nrfconnect-nrf-iot_thingy91
- test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
- test-fw-nrfconnect-nrf_crypto
- test-fw-nrfconnect-proprietary_esb
- test-fw-nrfconnect-ps
- test-fw-nrfconnect-rpc
- test-fw-nrfconnect-rs
- test-fw-nrfconnect-tfm
- test-fw-nrfconnect-thread
- test-fw-nrfconnect-zigbee
- test-low-level
- test-sdk-audio
- test-sdk-dfu
- test-sdk-find-my
- test-sdk-mcuboot
- test-sdk-pmic-samples
- test-sdk-sidewalk
- test-sdk-wifi
- test-secdom-samples-public

Note: This message is automatically posted and updated by the CI

nordic-segl · 2025-02-26T12:40:45Z

tests/benchmarks/multicore/idle_pwm_loopback/Kconfig.sysbuild

@@ -1,11 +1,10 @@
 #
-# Copyright (c) 2024 Nordic Semiconductor ASA


I believe that copyright year is a year when a new file was added.
Thus, any change to the existing file shall not change the year.
Am I correct?

github-actions · 2025-02-27T12:21:22Z

You can find the documentation preview for this PR here.

nordicjm · 2025-02-27T12:44:46Z

tests/benchmarks/multicore/idle_counter/sysbuild.cmake

-# Add remote project
-ExternalZephyrProject_Add(
+if(SB_CONFIG_SOC_NRF54H20)
+


Suggested change

nordicjm · 2025-02-27T12:45:07Z

tests/benchmarks/multicore/idle_pwm_loopback/sysbuild.cmake

-endif()
+if(SB_CONFIG_SOC_NRF54H20)
+  if(SB_CONFIG_REMOTE_GLOBAL_DOMAIN_CLOCK_FREQUENCY_SWITCHING)
+      set(REMOTE_SOURCE_DIR ${ZEPHYR_NRF_MODULE_DIR}/tests/benchmarks/multicore/common/remote_gdf_switching)


2 space cmake indent seems have gone to 4 here

nordicjm · 2025-02-27T12:46:55Z

tests/benchmarks/multicore/idle_uarte/testcase.yaml

+      fixture: gpio_loopback
+      pytest_root:
+        - "${CUSTOM_ROOT_TEST_DIR}/test_measure_power_consumption.py::test_measure_and_data_dump_power_consumption_gated_uarte"
+


Suggested change

Add test cases for fast counter, pwm, spim, uarte instances where global domain frequency is changed from remote application. Use solution for synchronization of host and remote core like in idle_pwm_loopback case. Signed-off-by: Piotr Krzyzanowski <piotr.krzyzanowski@nordicsemi.no>

sonarqubecloud · 2025-02-28T14:45:24Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Feb 26, 2025

nordic-pikr force-pushed the remote_gdf branch from 63a72ba to cd63074 Compare February 26, 2025 12:26

nordic-pikr marked this pull request as ready for review February 26, 2025 12:35

nordic-pikr requested review from a team as code owners February 26, 2025 12:35

nordic-piks approved these changes Feb 26, 2025

View reviewed changes

nordic-segl reviewed Feb 26, 2025

View reviewed changes

nordic-pikr force-pushed the remote_gdf branch 3 times, most recently from 6003a4f to cd6c2ba Compare February 26, 2025 15:13

nordic-segl approved these changes Feb 26, 2025

View reviewed changes

nordic-pikr marked this pull request as draft February 27, 2025 08:55

nordic-pikr force-pushed the remote_gdf branch 3 times, most recently from 30e7669 to ea0c4c6 Compare February 27, 2025 12:02

nordicjm reviewed Feb 27, 2025

View reviewed changes

nordic-pikr force-pushed the remote_gdf branch from ea0c4c6 to c769ef8 Compare February 28, 2025 14:39

nordic-pikr marked this pull request as ready for review February 28, 2025 14:50

nordic-pikr requested a review from nordicjm February 28, 2025 15:26

nordic-segl approved these changes Mar 3, 2025

View reviewed changes

nordicjm approved these changes Mar 3, 2025

View reviewed changes

nordic-piks merged commit 40b35ca into nrfconnect:main Mar 3, 2025
14 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

tests: benchmarks: multicore: Remote GDF switching #20625

tests: benchmarks: multicore: Remote GDF switching #20625

nordic-pikr commented Feb 26, 2025 •

edited

Loading

NordicBuilder commented Feb 26, 2025 •

edited

Loading

nordic-segl Feb 26, 2025

github-actions bot commented Feb 27, 2025

nordicjm Feb 27, 2025

nordicjm Feb 27, 2025

nordicjm Feb 27, 2025

sonarqubecloud bot commented Feb 28, 2025

		@@ -1,11 +1,10 @@
		#
		# Copyright (c) 2024 Nordic Semiconductor ASA

tests: benchmarks: multicore: Remote GDF switching #20625

tests: benchmarks: multicore: Remote GDF switching #20625

Conversation

nordic-pikr commented Feb 26, 2025 • edited Loading

NordicBuilder commented Feb 26, 2025 • edited Loading

CI Information

Inputs:

Sources:

Github labels

Outputs:

Toolchain

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

nordic-segl Feb 26, 2025

Choose a reason for hiding this comment

github-actions bot commented Feb 27, 2025

nordicjm Feb 27, 2025

Choose a reason for hiding this comment

nordicjm Feb 27, 2025

Choose a reason for hiding this comment

nordicjm Feb 27, 2025

Choose a reason for hiding this comment

sonarqubecloud bot commented Feb 28, 2025

Quality Gate passed

nordic-pikr commented Feb 26, 2025 •

edited

Loading

NordicBuilder commented Feb 26, 2025 •

edited

Loading