riot-ci/RIOT
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #13977 from maribu/pm_atomic_access

Browse Source 
sys/pm: Correctly access pm_blocker
This commit is contained in:
Francisco 2020-07-15 16:53:06 +02:00 committed by GitHub
commit 3934f9f09b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
sys/pm_layered/pm.c
View File

@ -36,11 +36,13 @@
/** /**
* @brief Global variable for keeping track of blocked modes * @brief Global variable for keeping track of blocked modes
*/ */
static volatile pm_blocker_t pm_blocker = { .val_u32 = PM_BLOCKER_INITIAL }; static pm_blocker_t pm_blocker = { .val_u32 = PM_BLOCKER_INITIAL };
void pm_set_lowest(void) void pm_set_lowest(void)
{ {
unsigned state = irq_disable();
pm_blocker_t blocker = pm_blocker; pm_blocker_t blocker = pm_blocker;
irq_restore(state);
unsigned mode = PM_NUM_MODES; unsigned mode = PM_NUM_MODES;
while (mode) { while (mode) {
if (blocker.val_u8[mode-1]) { if (blocker.val_u8[mode-1]) {
@ -50,7 +52,7 @@ void pm_set_lowest(void)
} }
/* set lowest mode if blocker is still the same */ /* set lowest mode if blocker is still the same */
unsigned state = irq_disable(); state = irq_disable();
if (blocker.val_u32 == pm_blocker.val_u32) { if (blocker.val_u32 == pm_blocker.val_u32) {
DEBUG("pm: setting mode %u\n", mode); DEBUG("pm: setting mode %u\n", mode);
pm_set(mode); pm_set(mode);
@ -81,7 +83,10 @@ void pm_unblock(unsigned mode)
pm_blocker_t pm_get_blocker(void) pm_blocker_t pm_get_blocker(void)
{ {
return pm_blocker; unsigned state = irq_disable();
pm_blocker_t result = pm_blocker;
irq_restore(state);
return result;
} }
#ifndef PROVIDES_PM_LAYERED_OFF #ifndef PROVIDES_PM_LAYERED_OFF