Fixed one bug in acl_fiber_sem_post() API.

zhengshuxin · zhengshuxin · commit 6485676f8f4a · 2025-01-09T09:57:30.000+08:00
diff --git a/c/src/sync/fiber_sem.c b/c/src/sync/fiber_sem.c
@@ -187,9 +187,12 @@ int acl_fiber_sem_post(ACL_FIBER_SEM *sem)
 	sem->num++;
 
 	if ((ready = FIRST_FIBER(&sem->waiting)) == NULL) {
+#if 0
+		// Don't yield here to avoid never wakeup. --zsx, 2025.1.9
 		if (sem->num >= sem->buf) {
 			acl_fiber_yield();
 		}
+#endif
 		return sem->num;
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -187,9 +187,12 @@ int acl_fiber_sem_post(ACL_FIBER_SEM *sem)`
`187`	`187`	`sem->num++;`
`188`	`188`
`189`	`189`	`if ((ready = FIRST_FIBER(&sem->waiting)) == NULL) {`
	`190`	`+#if 0`
	`191`	`+ // Don't yield here to avoid never wakeup. --zsx, 2025.1.9`
`190`	`192`	`if (sem->num >= sem->buf) {`
`191`	`193`	`acl_fiber_yield();`
`192`	`194`	`}`
	`195`	`+#endif`
`193`	`196`	`return sem->num;`
`194`	`197`	`}`
`195`	`198`