k_sleep() function misbehaving for greater values of kernel ticks

[Aditya Xavier]

Description:

Facing an issue with k_sleep() function misbehaving after few months due to the internal counter ticks crossed uint32_max values. Simulated the same by hardcoding the values given below on.

Setup:

1. NCS SDK version: v2.0.0 / Zephyr SDK v3.0.0 / Zephyr SDK v3.2.0

2. nRF52832 DK

Modifications:

1. Hardcode the values below in file https://github.com/zephyrproject-rtos/zephyr/blob/main/drivers/timer/nrf_rtc_timer.c#L42 to

static volatile uint32_t overflow_cnt = 23839;

static volatile uint64_t anchor = 399966126479;

static uint64_t last_count = 399966126479;

2. Hardcode the curr_tick in file https://github.com/zephyrproject-rtos/zephyr/blob/main/kernel/timeout.c#L15 to

static uint64_t curr_tick = UINT32_MAX + 10000;

Sample application is attached.

8080.hello_world.zip

Expected output:

--------------------------- Hello World! nrf52dk_nrf52832

to be printed every 1minute due to k_sleep(K_MINUTES(1)); but is taking more than 10mins to trigger.

Regards,

Raymond

[Sigurd]

Raymond Sebastian said:

Any update on this issue from your side.??

Yes, got some feedback on this now. From the developer:  Setting startup values is not enough to set system timer infrastructure in the state that is seen as >32bit time. I took different path and used RTC feature which allows to trigger overflow and move COUNTER to 0xFFFFF0. I implemented a function which triggers overflow and updates driver state to move in time: https://github.com/zephyrproject-rtos/zephyr/pull/52287

I used this function and modified hello_world to move in time beyond 32 bits then i tried to sleep for a minute and used RTC2 to check if thread waken up after 60 seconds. It seems that it works correctly.

Below is the branch with this changes. Maybe you can try this approach to trigger the error scenario.

Branch: https://github.com/nordic-krch/zephyr/tree/rtc_beyond32_test

Hello_world modification: https://github.com/nordic-krch/zephyr/commit/06ea5e8fb586935a76861e6315363b7c710b7f9b