Skip to content

Conversation

@wangchdo
Copy link
Contributor

@wangchdo wangchdo commented Jan 20, 2026

Summary

Enable the hrtimer ostest in CI to ensure the hrtimer subsystem is built and exercised in CI.

Impact

This change only enable CI on hritmer module

Testing

ostest passed with hrtimer enabled:

a2g-tc397-5v-tft:nsh

NuttShell (NSH)
nsh>
nsh> uname -a
NuttX 0.0.0 d8dce7c87e Jan 20 2026 09:23:39 tricore a2g-tc397-5v-tft
nsh>
nsh> ostest

(...)

user_main: hrtimer test

End of test memory usage:
VARIABLE  BEFORE   AFTER
======== ======== ========
arena       28e00    28e00
ordblks         6        6
mxordblk    1f890    1f890
uordblks     5578     5578
fordblks    23888    23888

Final memory usage:
VARIABLE  BEFORE   AFTER
======== ======== ========
arena       28e00    28e00
ordblks         1        6
mxordblk    24208    1f890
uordblks     4bf8     5578
fordblks    24208    23888
user_main: Exiting
ostest_main: Exiting with status 0
nsh>
nsh>

rv-virt:smp64

NuttShell (NSH)
nsh> 
nsh> 
nsh> 
nsh> uname -a
NuttX 0.0.0 d8dce7c87e-dirty Jan 20 2026 09:28:44 risc-v rv-virt
nsh> 
nsh> ostest

(...)

user_main: hrtimer test

End of test memory usage:
VARIABLE  BEFORE   AFTER
======== ======== ========
arena     1fc0bc0  1fc0bc0
ordblks         8        6
mxordblk  1f9e758  1f9e758
uordblks    14b10    16310
fordblks  1fac0b0  1faa8b0

Final memory usage:
VARIABLE  BEFORE   AFTER
======== ======== ========
arena     1fc0bc0  1fc0bc0
ordblks         1        6
mxordblk  1fb5ac8  1f9e758
uordblks     b0f8    16310
fordblks  1fb5ac8  1faa8b0
user_main: Exiting
ostest_main: Exiting with status 0
nsh>

@github-actions github-actions bot added Area: OS Components OS Components issues Board: risc-v Size: XS The size of the change in this PR is very small labels Jan 20, 2026
  Enable hrtimer support on the smartl-c906 board so that CI can build
  and exercise the hrtimer subsystem.

Signed-off-by: Chengdong Wang <[email protected]>
@github-actions github-actions bot removed the Area: OS Components OS Components issues label Jan 20, 2026
@wangchdo wangchdo changed the title sched/hrtimer: update callback declaration and enable hrtimer ostest in CI sched/hrtimer: enable hrtimer ostest in CI Jan 20, 2026
@xiaoxiang781216
Copy link
Contributor

@wangchdo please fix the ci warning and error:

====================================================================================
Configuration/Tool: smartl-c906/sotest
2026-01-20 03:07:23
------------------------------------------------------------------------------------
  Cleaning...
/github/workspace/sources/apps/Application.mk:238: target 'signest.c.github.workspace.sources.apps.testing.ostest.o' given more than once in the same rule
/github/workspace/sources/apps/Application.mk:238: target 'signest.c.github.workspace.sources.apps.testing.ostest.o' given more than once in the same rule
  Configuring...
  Building NuttX...
/github/workspace/sources/apps/Application.mk:238: target 'signest.c.github.workspace.sources.apps.testing.ostest.o' given more than once in the same rule
/github/workspace/sources/apps/Application.mk:238: target 'signest.c.github.workspace.sources.apps.testing.ostest.o' given more than once in the same rule
/github/workspace/sources/apps/Application.mk:238: target 'signest.c.github.workspace.sources.apps.testing.ostest.o' given more than once in the same rule
/github/workspace/sources/apps/Application.mk:238: target 'signest.c.github.workspace.sources.apps.testing.ostest.o' given more than once in the same rule
hrtimer.c: In function 'hrtimer_test':
Error: hrtimer.c:219:23: error: passing argument 2 of 'hrtimer_start' from incompatible pointer type [-Wincompatible-pointer-types]
  219 |                       test_hrtimer_callback,
      |                       ^~~~~~~~~~~~~~~~~~~~~
      |                       |
      |                       uint64_t (*)(hrtimer_t *, uint64_t) {aka long unsigned int (*)(struct hrtimer_s *, long unsigned int)}
In file included from hrtimer.c:28:
/github/workspace/sources/nuttx/include/nuttx/hrtimer.h:186:59: note: expected 'hrtimer_entry_t' {aka 'long unsigned int (*)(const struct hrtimer_s *, long unsigned int)'} but argument is of type 'uint64_t (*)(hrtimer_t *, uint64_t)' {aka 'long unsigned int (*)(struct hrtimer_s *, long unsigned int)'}
  186 | int hrtimer_start(FAR hrtimer_t *hrtimer, hrtimer_entry_t func,
      |                                           ~~~~~~~~~~~~~~~~^~~~

@wangchdo
Copy link
Contributor Author

@wangchdo please fix the ci warning and error:

====================================================================================
Configuration/Tool: smartl-c906/sotest
2026-01-20 03:07:23
------------------------------------------------------------------------------------
  Cleaning...
/github/workspace/sources/apps/Application.mk:238: target 'signest.c.github.workspace.sources.apps.testing.ostest.o' given more than once in the same rule
/github/workspace/sources/apps/Application.mk:238: target 'signest.c.github.workspace.sources.apps.testing.ostest.o' given more than once in the same rule
  Configuring...
  Building NuttX...
/github/workspace/sources/apps/Application.mk:238: target 'signest.c.github.workspace.sources.apps.testing.ostest.o' given more than once in the same rule
/github/workspace/sources/apps/Application.mk:238: target 'signest.c.github.workspace.sources.apps.testing.ostest.o' given more than once in the same rule
/github/workspace/sources/apps/Application.mk:238: target 'signest.c.github.workspace.sources.apps.testing.ostest.o' given more than once in the same rule
/github/workspace/sources/apps/Application.mk:238: target 'signest.c.github.workspace.sources.apps.testing.ostest.o' given more than once in the same rule
hrtimer.c: In function 'hrtimer_test':
Error: hrtimer.c:219:23: error: passing argument 2 of 'hrtimer_start' from incompatible pointer type [-Wincompatible-pointer-types]
  219 |                       test_hrtimer_callback,
      |                       ^~~~~~~~~~~~~~~~~~~~~
      |                       |
      |                       uint64_t (*)(hrtimer_t *, uint64_t) {aka long unsigned int (*)(struct hrtimer_s *, long unsigned int)}
In file included from hrtimer.c:28:
/github/workspace/sources/nuttx/include/nuttx/hrtimer.h:186:59: note: expected 'hrtimer_entry_t' {aka 'long unsigned int (*)(const struct hrtimer_s *, long unsigned int)'} but argument is of type 'uint64_t (*)(hrtimer_t *, uint64_t)' {aka 'long unsigned int (*)(struct hrtimer_s *, long unsigned int)'}
  186 | int hrtimer_start(FAR hrtimer_t *hrtimer, hrtimer_entry_t func,
      |                                           ~~~~~~~~~~~~~~~~^~~~

OK, This will be fixed after apache/nuttx-apps#3347 is merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Board: risc-v Size: XS The size of the change in this PR is very small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants