Page MenuHomeFreeBSD

Add SI_SUB_TASKQ after SI_SUB_INTR and move taskqueue initialization there for EARLY_AP_STARTUP
ClosedPublic

Authored by cem on Aug 16 2017, 9:32 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jan 1, 4:29 PM
Unknown Object (File)
Nov 12 2024, 9:18 PM
Unknown Object (File)
Oct 17 2024, 9:11 AM
Unknown Object (File)
Oct 12 2024, 12:41 PM
Unknown Object (File)
Oct 2 2024, 2:02 PM
Unknown Object (File)
Oct 2 2024, 11:49 AM
Unknown Object (File)
Sep 26 2024, 8:39 PM
Unknown Object (File)
Sep 23 2024, 11:26 PM
Subscribers

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Aug 16 2017, 9:35 PM

Looks good to me assuming it boots. A few other places that are using SI_SUB_INIT_IF that might be candidates for now using SI_SUB_TASKQ:

compat/linuxkpi/common/src/linux_work.c:SYSINIT(linux_work_init, SI_SUB_INIT_IF, SI_ORDER_THIRD, linux_work_init, NULL);
compat/linuxkpi/common/src/linux_work.c:SYSUNINIT(linux_work_uninit, SI_SUB_INIT_IF, SI_ORDER_THIRD, linux_work_uninit, NULL);
compat/linuxkpi/common/src/linux_tasklet.c:SYSINIT(linux_tasklet, SI_SUB_INIT_IF, SI_ORDER_THIRD, tasklet_subsystem_init, NULL);
compat/linuxkpi/common/src/linux_tasklet.c:SYSUNINIT(linux_tasklet, SI_SUB_INIT_IF, SI_ORDER_THIRD, tasklet_subsystem_uninit, NULL);
sys/gtaskqueue.h:SYSINIT(taskqgroup_##name, SI_SUB_INIT_IF, SI_ORDER_FIRST,    \
sys/gtaskqueue.h:SYSINIT(taskqgroup_##name, SI_SUB_INIT_IF, SI_ORDER_FIRST,    \

I think gtaskqueue.h is trickier. I bet that this would make it simpler though as it could always do the non-EARLY_AP_STARTUP case and remove the #ifdef's as SI_SUB_TASKQ will always be before SI_SUB_SMP.

This revision was automatically updated to reflect the committed changes.