In my latest commit, I've pulled in the taskq implementation that we've been 
using in delphix-os (via libcmdutils), directly into libzfs (with slight 
modifications).

It might be possible to pull this out into a completely separate (and new) 
library, but I'm not sure that's actually useful, since there's some 
outstanding concerns w.r.t. out-of-memory handling, and based on @gwr's comment.

For now, I've left symbols with the `utaskq` prefix, but if we all agree that 
it's OK to add this implementation directly to libzfs, and *not* use 
libfakekernel's taskq implementation, I think it might be worthwhile to change 
this prefix to something more libzfs specific. E.g. rename `utaskq.c` to 
`libzfs_taskq.c`, rename `utaskq_create` to `libzfs_taskq_create`, etc.

Folks, please let me know what y'all think.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openzfs/openzfs/pull/536#issuecomment-367104236
------------------------------------------
openzfs-developer
Archives: 
https://openzfs.topicbox.com/groups/developer/discussions/T22334a01fda83bfd-Mfe9d6d8d710d921225b928cb
Powered by Topicbox: https://topicbox.com

Reply via email to