testing/drivers: add watchdog notifier cmocka coverage by Zepp-Hanzj · Pull Request #3639 · apache/nuttx-apps

Zepp-Hanzj · 2026-07-14T13:49:51Z

Summary

Add focused cmocka coverage for the watchdog timeout notifier chain and an independent watchdog-only drivertest configuration.

Changes

Add CONFIG_TESTING_DRIVER_TEST_WATCHDOG_ONLY.
When enabled, register only cmocka_driver_watchdog from the drivertest Makefile/CMake rules.
Add notifier tests covering:
- priority ordering;
- duplicate registration suppression;
- repeated timeout delivery;
- unregistering one notifier while preserving the others;
- WATCHDOG_NOTIFIER_ACTION propagation;
- data == NULL propagation;
- concurrent register/timeout/unregister activity.
Guard reset-cause assertions so the notifier-only simulator configuration does not require board reset-cause support.

Validation

The test was run using the companion NuttX simulator configuration, which registers only the watchdog cmocka driver. The final result was:

[==========] tests: 2 test(s) run.
[  PASSED  ] 2 test(s).

The race test repeatedly interleaves timeout delivery with notifier registration and unregistration and asserts that callbacks remain valid and receive a NULL payload.

Additional hardware-path validation was performed in the companion NuttX PR on STM32F407 WWDG hardware. The corresponding UART logs and screenshots are linked there:

Scope

This PR contains only watchdog notifier tests and their test registration/configuration. Unrelated O_RDONLY compatibility changes are intentionally excluded.

Merge order

Merge the companion NuttX core PR first, then enable this test configuration in CI or simulator validation.

Checklist

Makefile and CMake registration paths updated consistently.
Watchdog-only configuration does not register unrelated drivertest applications.
git diff --check passes.
Simulator cmocka notifier and race tests pass.

Zepp-Hanzj · 2026-07-14T14:37:13Z

Updated the companion branch in response to the check results:

Added Signed-off-by trailers to both commits.
Applied cmake-format to testing/drivers/drivertest/CMakeLists.txt.
Fixed nxstyle issues in the notifier race test (blank lines and block-comment formatting).

The test scope and watchdog-only registration behavior are unchanged.

cederom · 2026-07-14T14:49:57Z

 #if !defined(CONFIG_ARCH_ARMV7A) || !defined(CONFIG_ARCH_HAVE_TRUSTZONE)
-    cmocka_unit_test_prestate(drivertest_watchdog_api, &wdg_state)
+    cmocka_unit_test_prestate(drivertest_watchdog_api, &wdg_state),
+#endif


#endif #endif #ifdef CONFIG_WATCHDOG_TIMEOUT_NOTIFIER cmocka_unit_test_prestate(drivertest_watchdog_notifier, &wdg_state),

cederom

Thank you @Zepp-Hanzj :-)

You may want to udpate code style or leave as is if you want.
Please add git commit messages, see https://nuttx.apache.org/docs/latest/contributing/index.html.

cederom · 2026-07-14T21:51:07Z

yeah we can skip this cosmetic part :-) just a git message and we are ready to go :-)

…

-- CeDeROM, SQ7MHZ, http://www.tomek.cedro.info

linguini1 · 2026-07-14T22:13:15Z

If you are using AI tools, please add the Assisted-by: field to your commit messages.

Could you please attach the logs from the tests you ran on sim?

xiaoxiang781216

d

xiaoxiang781216 · 2026-07-15T05:05:08Z

+  unsigned int index;
+
+  test_nb = (FAR struct watchdog_notifier_test_nb_s *)
+            ((FAR char *)nb - offsetof(struct watchdog_notifier_test_nb_s,


call container_of

xiaoxiang781216 · 2026-07-15T05:08:48Z


+#ifdef CONFIG_WATCHDOG_TIMEOUT_NOTIFIER
+
+struct watchdog_notifier_test_nb_s


why not merge into wdg_state_s

xiaoxiang781216 · 2026-07-15T05:11:06Z

+ifeq ($(CONFIG_TESTING_DRIVER_TEST_WATCHDOG_ONLY),y)
+MAINSRC  += drivertest_watchdog.c
+PROGNAME += cmocka_driver_watchdog
+else


why skip other wdog test

Zepp-Hanzj · 2026-07-15T05:52:29Z

Addressed the review comments in the latest push.

Code updates

Merged notifier test state into the existing wdg_state_s.
Replaced manual offsetof pointer arithmetic with container_of().
Removed the extra space before id.
Added Assisted-by: OpenAI Codex to both commits.
Commits now use the requested identity and trailer:
Signed-off-by: hanzhijian <hanzhijian@zepp.com>.

The watchdog-only option intentionally skips the other drivertest applications: its purpose is to produce a minimal simulator image that registers only cmocka_driver_watchdog. The normal configuration path is unchanged and still builds the complete drivertest set.

Simulator test log

Configuration: sim:watchdog-notifier

Configuration: sim:watchdog-notifier
Configuration: CONFIG_WATCHDOG_AUTOMONITOR_BY_CAPTURE=y
Command: cmocka_driver_watchdog

[==========] tests: Running 2 test(s).
[ RUN      ] drivertest_watchdog_notifier
[       OK ] drivertest_watchdog_notifier
[ RUN      ] drivertest_watchdog_notifier_race
[       OK ] drivertest_watchdog_notifier_race
[==========] tests: 2 test(s) run.
[  PASSED  ] 2 test(s).

The race test interleaves timeout delivery with notifier registration and
unregistration from a concurrent pthread. It completed without a fault,
deadlock, or invalid callback payload.

The capture-stop pending-interrupt validation also passed: the callback returned safely after the association was cleared and did not dereference a NULL upper-half.

Add cmocka coverage for notifier priority ordering, duplicate registration, repeated timeout delivery, unregister behavior, action propagation, and NULL callback data. Signed-off-by: hanzhijian <hanzhijian@zepp.com> Assisted-by: OpenAI Codex

Add a focused watchdog-only cmocka registration and exercise notifier ordering, duplicate registration, repeated timeout delivery, unregister behavior, NULL data, and concurrent register/timeout/unregister activity. Signed-off-by: hanzhijian <hanzhijian@zepp.com> Assisted-by: OpenAI Codex

Zepp-Hanzj · 2026-07-15T05:54:25Z

The checkpatch failure was caused by commit messages shorter than the repository minimum of five lines. Both commits now include a descriptive body, the requested Signed-off-by: hanzhijian <hanzhijian@zepp.com>, and Assisted-by: OpenAI Codex. The branch was force-updated; the new commit range passes the local checkpatch message checks (apart from locally unavailable optional codespell/cvt2utf tools).

Include the container_of declaration and exclude unused hardware-only helpers when the watchdog notifier test is built alone. Signed-off-by: hanzhijian <hanzhijian@zepp.com> Assisted-by: OpenAI Codex

Zepp-Hanzj · 2026-07-15T06:11:35Z

The simulator build exposed and is now fixed in commit 764e0a045: include <nuttx/nuttx.h> for container_of() and conditionally exclude hardware-only helpers from the watchdog-only configuration. A local make -j$(nproc) now completes without the previous compile error or unused-helper warnings. Please rerun the simulator test and attach your own raw build/run logs.

Zepp-Hanzj · 2026-07-15T06:15:20Z

Simulator test log
simulator-watchdog-notifier-0715.log
simulator-watchdog-notifier-run-user.log
simulator-watchdog-notifier-build-user.log

Assisted-by: OpenAI Codex

Zepp-Hanzj · 2026-07-15T06:21:12Z

Thanks @cederom. Agreed — the remaining cosmetic formatting discussion can be skipped. The required commit messages are now in place with both Signed-off-by and Assisted-by trailers, and the simulator build/run logs have been attached in the comments above. The latest branch also contains the watchdog-only build fix and local compilation passes. I will leave the PR ready for final CI/review.

Keep the existing drivertest applications under the main test option and register only the watchdog notifier application for the watchdog-only mode. This prevents unrelated cmocka applications from being added to simulator configurations that do not enable the driver test option. Signed-off-by: hanzhijian <hanzhijian@zepp.com> Assisted-by: OpenAI Codex

Zepp-Hanzj · 2026-07-15T07:04:14Z

The failed CI jobs were investigated. The root cause was the CMake condition in testing/drivers/drivertest/CMakeLists.txt: the else() was matching the outer CONFIG_TESTING_DRIVER_TEST condition, so configurations that did not enable the driver test could still register cmocka applications. This caused cmocka.h errors in the sim-02 matrix.

Fixed in commit b4167a1a5:

Register cmocka_driver_watchdog only when CONFIG_TESTING_DRIVER_TEST_WATCHDOG_ONLY=y.
Keep the original drivertest application block under CONFIG_TESTING_DRIVER_TEST && NOT CONFIG_TESTING_DRIVER_TEST_WATCHDOG_ONLY.
Preserve the normal configuration behavior.

cmake-format --check and git diff --check pass locally. The branch update has triggered a new CI run.

github-actions Bot added Size: M Area: Testing labels Jul 14, 2026

Zepp-Hanzj force-pushed the fix/watchdog-notifier-tests branch from 9584062 to 6ec0489 Compare July 14, 2026 14:35

cederom reviewed Jul 14, 2026

View reviewed changes

Comment thread testing/drivers/drivertest/drivertest_watchdog.c Outdated

Zepp-Hanzj force-pushed the fix/watchdog-notifier-tests branch from 6ec0489 to cdd0b1f Compare July 14, 2026 14:49

cederom reviewed Jul 14, 2026

View reviewed changes

cederom requested changes Jul 14, 2026

View reviewed changes

cederom requested review from acassis, linguini1, raiden00pl and simbit18 July 14, 2026 15:08

xiaoxiang781216 reviewed Jul 15, 2026

View reviewed changes

Zepp-Hanzj force-pushed the fix/watchdog-notifier-tests branch from cdd0b1f to f1d3688 Compare July 15, 2026 05:51

Zepp-Hanzj added 2 commits July 15, 2026 13:54

testing/drivers: add watchdog notifier tests

9a2faa8

Add cmocka coverage for notifier priority ordering, duplicate registration, repeated timeout delivery, unregister behavior, action propagation, and NULL callback data. Signed-off-by: hanzhijian <hanzhijian@zepp.com> Assisted-by: OpenAI Codex

Zepp-Hanzj force-pushed the fix/watchdog-notifier-tests branch from f1d3688 to 8fa68e9 Compare July 15, 2026 05:54

testing/drivers: fix watchdog-only build

764e0a0

Include the container_of declaration and exclude unused hardware-only helpers when the watchdog notifier test is built alone. Signed-off-by: hanzhijian <hanzhijian@zepp.com> Assisted-by: OpenAI Codex

jerpelea previously approved these changes Jul 15, 2026

View reviewed changes

Zepp-Hanzj dismissed jerpelea’s stale review via b4167a1 July 15, 2026 07:04


		#ifdef CONFIG_WATCHDOG_TIMEOUT_NOTIFIER

		struct watchdog_notifier_test_nb_s

Uh oh!

Conversation

Zepp-Hanzj commented Jul 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Changes

Validation

Scope

Merge order

Checklist

Uh oh!

Zepp-Hanzj commented Jul 14, 2026

Uh oh!

Uh oh!

cederom Jul 14, 2026

Choose a reason for hiding this comment

Uh oh!

cederom left a comment

Choose a reason for hiding this comment

Uh oh!

cederom commented Jul 14, 2026 via email

Uh oh!

linguini1 commented Jul 14, 2026

Uh oh!

xiaoxiang781216 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

xiaoxiang781216 Jul 15, 2026

Choose a reason for hiding this comment

Uh oh!

xiaoxiang781216 Jul 15, 2026

Choose a reason for hiding this comment

Uh oh!

xiaoxiang781216 Jul 15, 2026

Choose a reason for hiding this comment

Uh oh!

Zepp-Hanzj commented Jul 15, 2026

Code updates

Simulator test log

Uh oh!

Zepp-Hanzj commented Jul 15, 2026

Uh oh!

Zepp-Hanzj commented Jul 15, 2026

Uh oh!

Zepp-Hanzj commented Jul 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Zepp-Hanzj commented Jul 15, 2026

Uh oh!

Zepp-Hanzj commented Jul 15, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Zepp-Hanzj commented Jul 14, 2026 •

edited

Loading

Zepp-Hanzj commented Jul 15, 2026 •

edited

Loading