Mirrors_Framework/nuttx
1
0
Fork 0
mirror of https://github.com/apache/nuttx.git synced 2026-09-28 15:54:57 +08:00
Code Issues Packages Projects Releases Wiki Activity

clock_adjtime: use small lock to protect g_adjtime_ppb g_adjtime_wdog

Browse Source 
reason:
We would like to replace the critical section with a small lock.

Signed-off-by: hujun5 <hujun5@xiaomi.com>
This commit is contained in:
hujun5
2025-01-12 22:07:15 +08:00
committed by Xiang Xiao
parent a2d4d74af7
commit 19bca74c7c
1 changed files with 10 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
sched/clock/clock_adjtime.c
+10 -2
View File
@@ -49,6 +49,7 @@
static struct wdog_s g_adjtime_wdog;
static long g_adjtime_ppb;
static spinlock_t g_adjtime_lock = SP_UNLOCKED;
/****************************************************************************
* Private Functions
@@ -58,8 +59,12 @@ static long g_adjtime_ppb;
static void adjtime_wdog_callback(wdparm_t arg)
{
irqstate_t flags;
UNUSED(arg);
flags = spin_lock_irqsave(&g_adjtime_lock);
#ifdef CONFIG_ARCH_HAVE_ADJTIME
up_adjtime(0);
#endif
@@ -69,6 +74,7 @@ static void adjtime_wdog_callback(wdparm_t arg)
#endif
g_adjtime_ppb = 0;
spin_unlock_irqrestore(&g_adjtime_lock, flags);
}
/* Query remaining adjustment in microseconds */
@@ -108,7 +114,8 @@ static int adjtime_start(long long adjust_usec)
ppb = -ppb_limit;
}
flags = enter_critical_section();
flags = spin_lock_irqsave(&g_adjtime_lock);
sched_lock();
/* Set new adjustment */
@@ -134,7 +141,8 @@ static int adjtime_start(long long adjust_usec)
wd_cancel(&g_adjtime_wdog);
}
leave_critical_section(flags);
spin_unlock_irqrestore(&g_adjtime_lock, flags);
sched_unlock();
return ret;
}