Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commitf55f520

Browse files
committed
rp2: Refactor to not use pico-sdk alarm pool functions for sleeping.
The best_effort_wfe_or_timeout() and sleep_us() pico-sdk functions use thepico-sdk alarm pool internally, and that has a bug.Some usages inside pico-sdk (notably multicore_lockout_start_blocking())will still end up calling best_effort_wfe_or_timeout(), although usuallywith "end_of_time" as the timeout value so it should avoid any alarm poolrace conditions.This work was funded through GitHub Sponsors.Signed-off-by: Angus Gratton <angus@redyak.com.au>
1 parentc765ca1 commitf55f520

File tree

4 files changed

+38
-10
lines changed

4 files changed

+38
-10
lines changed

‎ports/rp2/mpconfigport.h

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -263,7 +263,7 @@ extern const struct _mp_obj_type_t mod_network_nic_type_wiznet5k;
263263
if ((TIMEOUT_MS) < 0) { \
264264
__wfe(); \
265265
} else { \
266-
best_effort_wfe_or_timeout(make_timeout_time_ms(TIMEOUT_MS)); \
266+
mp_wfe_or_timeout(TIMEOUT_MS); \
267267
} \
268268
} while (0)
269269

@@ -281,6 +281,9 @@ typedef intptr_t mp_off_t;
281281
#defineBINARY_INFO_ID_MP_FROZEN 0x4a99d719
282282
#defineMICROPY_FROZEN_LIST_ITEM(name,file) bi_decl(bi_string(BINARY_INFO_TAG_MICROPYTHON, BINARY_INFO_ID_MP_FROZEN, name))
283283

284+
// Port-specific function to create a wakeup interrupt after timeout_ms and enter WFE
285+
voidmp_wfe_or_timeout(uint32_ttimeout_ms);
286+
284287
externuint32_trosc_random_u32(void);
285288
externvoidlwip_lock_acquire(void);
286289
externvoidlwip_lock_release(void);

‎ports/rp2/mphalport.c

Lines changed: 33 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -198,17 +198,31 @@ mp_uint_t mp_hal_stdout_tx_strn(const char *str, mp_uint_t len) {
198198
returndid_write ?ret :0;
199199
}
200200

201+
voidmp_hal_delay_us(mp_uint_tus) {
202+
// Avoid calling sleep_us() and invoking the alarm pool by splitting long
203+
// sleeps into an optional longer sleep and a shorter busy-wait
204+
uint64_tend=time_us_64()+us;
205+
if (us>1000) {
206+
mp_hal_delay_ms(us /1000);
207+
}
208+
while (time_us_64()<end) {
209+
// Tight loop busy-wait for accurate timing
210+
}
211+
}
212+
201213
voidmp_hal_delay_ms(mp_uint_tms) {
202-
absolute_time_tt=make_timeout_time_ms(ms);
214+
mp_uint_tstart=mp_hal_ticks_ms();
215+
mp_uint_telapsed=0;
203216
do {
204-
mp_event_handle_nowait();
205-
}while (!best_effort_wfe_or_timeout(t));
217+
mp_event_wait_ms(ms-elapsed);
218+
elapsed=mp_hal_ticks_ms()-start;
219+
}while (elapsed<ms);
206220
}
207221

208222
voidmp_hal_time_ns_set_from_rtc(void) {
209223
// Delay at least one RTC clock cycle so it's registers have updated with the most
210224
// recent time settings.
211-
sleep_us(23);
225+
mp_hal_delay_us(23);
212226

213227
// Sample RTC and time_us_64() as close together as possible, so the offset
214228
// calculated for the latter can be as accurate as possible.
@@ -293,3 +307,18 @@ void soft_timer_init(void) {
293307
// have the timer IRQ execute it directly at the correct priority.
294308
NVIC_SetPriority(TIMER_IRQ_0_IRQn+MICROPY_HW_SOFT_TIMER_ALARM_NUM,IRQ_PRI_PENDSV);
295309
}
310+
311+
voidmp_wfe_or_timeout(uint32_ttimeout_ms) {
312+
soft_timer_entry_ttimer;
313+
314+
soft_timer_static_init(&timer,SOFT_TIMER_MODE_ONE_SHOT,0,NULL);
315+
316+
// Note the timer doesn't have an associated callback, it just exists to create a
317+
// hardware interrupt to wake the CPU
318+
soft_timer_insert(&timer,timeout_ms);
319+
320+
__wfe();
321+
322+
// Clean up the timer node if it's not already
323+
soft_timer_remove(&timer);
324+
}

‎ports/rp2/mphalport.h

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -52,10 +52,6 @@ void mp_thread_end_atomic_section(uint32_t);
5252
voidmp_hal_set_interrupt_char(intc);
5353
voidmp_hal_time_ns_set_from_rtc(void);
5454

55-
staticinlinevoidmp_hal_delay_us(mp_uint_tus) {
56-
sleep_us(us);
57-
}
58-
5955
staticinlinevoidmp_hal_delay_us_fast(mp_uint_tus) {
6056
busy_wait_us(us);
6157
}

‎shared/runtime/softtimer.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ void soft_timer_handler(void) {
8989
heap= (soft_timer_entry_t*)mp_pairheap_pop(soft_timer_lt,&heap->pairheap);
9090
if (entry->flags&SOFT_TIMER_FLAG_PY_CALLBACK) {
9191
mp_sched_schedule(entry->py_callback,MP_OBJ_FROM_PTR(entry));
92-
}else {
92+
}elseif (entry->c_callback){
9393
entry->c_callback(entry);
9494
}
9595
if (entry->mode==SOFT_TIMER_MODE_PERIODIC) {

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp