imrcpan/freebsd
1
0 Fork 0
mirror of https://github.com/freebsd/freebsd-src.git synced 2026-10-28 19:34:43 +01:00
Code Issues Packages Projects Releases Wiki Activity

FreeBSD: Sync taskq_cancel_id() returns with Linux

Browse Source 
Couple places in the code depend on 0 returned only if the task was
actually cancelled.  Doing otherwise could lead to extra references
being dropped.  The race could be small, but I believe CI hit it
from time to time.

Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by:	Alexander Motin <[email protected]>
Sponsored by:	iXsystems, Inc.
Closes #16565
This commit is contained in:
Alexander Motin authored and GitHub committed 2024-09-24 16:29:18 -07:00
1 parent ccc420acd5
commit 832f66b218
1 file changed
+2 -2
Download Patch File Download Diff File Expand all files Collapse all files
module/os/freebsd/spl/spl_taskq.c
+2 -2
View File
@@ -357,7 +357,7 @@ taskq_cancel_id(taskq_t *tq, taskqid_t tid)
taskq_ent_t *ent;
if ((ent = taskq_lookup(tid)) == NULL)
return (0);
return (ENOENT);
if (ent->tqent_type == NORMAL_TASK) {
rc = taskqueue_cancel(tq->tq_queue, &ent->tqent_task, &pend);
@@ -380,7 +380,7 @@ taskq_cancel_id(taskq_t *tq, taskqid_t tid)
}
/* Free the extra reference we added with taskq_lookup. */
taskq_free(ent);
return (rc);
return (pend ? 0 : ENOENT);
}
static void