Paths

Table of Contentst

Differential D47907

init_main: Use TUNABLE_INT_FETCH to initialize verbose_sysinit
ClosedPublic
Actions

Authored by zlei on Wed, Dec 4, 12:22 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Dec 6, 1:50 PM

Subscribers

Details

Reviewers

kevans
cperciva

Commits

rGa7cfcb264512: init_main: Use TUNABLE_INT_FETCH to initialize verbose_sysinit

Summary

It is too late to initialize verbose_sysinit via TUNABLE_INT, as
TUNABLE_INT run at the order of SI_SUB_TUNABLES, thus any subsystems
those have order prior or equal to SI_SUB_TUNABLES are not logged.

MFC after: 1 week

Test Plan

Boot with debug.verbose_sysinit enabled.

# dmesg
---<<BOOT>>---
subsystem 6fffff
   madt_register(0)... done.
   apic_init(0)... done.
subsystem 700000
   mp_setmaxid(0)... done.
   tunable_int_init(&__tunable_int_102)... done.
   tunable_int_init(&__tunable_int_104)... done.
   tunable_int_init(&__tunable_int_106)... done.
   tunable_int_init(&__tunable_int_197)... done.
...

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

zlei created this revision.Wed, Dec 4, 12:22 PM

Herald added subscribers: olce, imp. · View Herald TranscriptWed, Dec 4, 12:22 PM

zlei requested review of this revision.Wed, Dec 4, 12:22 PM

zlei updated this revision to Diff 147467.Wed, Dec 4, 12:41 PM

zlei mentioned this in D47904: init_main: Fix logging the subsystem of pre-loaded kernel modules.

zlei added a parent revision: D47904: init_main: Fix logging the subsystem of pre-loaded kernel modules.

zlei added a child revision: D47909: boottrace: Fix tracing the execution of SYSINITs.Wed, Dec 4, 2:22 PM

Heh, whoops-

This revision is now accepted and ready to land.Wed, Dec 4, 3:28 PM

Closed by commit rGa7cfcb264512: init_main: Use TUNABLE_INT_FETCH to initialize verbose_sysinit (authored by zlei). · Explain WhyFri, Dec 6, 1:48 PM

This revision was automatically updated to reflect the committed changes.

zlei added a commit: rGa7cfcb264512: init_main: Use TUNABLE_INT_FETCH to initialize verbose_sysinit.

Revision Contents
Changeset List

Path

Size

sys/

kern/

2 lines

Diff 147464

sys/kern/init_main.c

Loading...