Merge pull request #4 from rota1001/fix-producer-stopping-condition

jserv · web-flow · commit 63980d40d8c8 · 2025-03-27T13:49:51.000+08:00
Fix incorrect open count check in release function
diff --git a/main.c b/main.c
@@ -428,7 +428,7 @@ static int kxo_open(struct inode *inode, struct file *filp)
 static int kxo_release(struct inode *inode, struct file *filp)
 {
     pr_debug("kxo: %s\n", __func__);
-    if (atomic_dec_and_test(&open_cnt) == 0) {
+    if (atomic_dec_and_test(&open_cnt)) {
         del_timer_sync(&timer);
         flush_workqueue(kxo_workqueue);
         fast_buf_clear();

Original file line number	Diff line number	Diff line change
`@@ -428,7 +428,7 @@ static int kxo_open(struct inode inode, struct file filp)`
`428`	`428`	`static int kxo_release(struct inode inode, struct file filp)`
`429`	`429`	`{`
`430`	`430`	`pr_debug("kxo: %s\n", __func__);`
`431`		`- if (atomic_dec_and_test(&open_cnt) == 0) {`
	`431`	`+ if (atomic_dec_and_test(&open_cnt)) {`
`432`	`432`	`del_timer_sync(&timer);`
`433`	`433`	`flush_workqueue(kxo_workqueue);`
`434`	`434`	`fast_buf_clear();`