Page MenuHomeFreeBSD

boottrace: rc annotations
ClosedPublic

Authored by mhorne on Sep 13 2021, 3:19 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Oct 19, 1:34 AM
Unknown Object (File)
Oct 4 2024, 4:34 PM
Unknown Object (File)
Oct 3 2024, 1:07 PM
Unknown Object (File)
Oct 2 2024, 5:14 AM
Unknown Object (File)
Oct 1 2024, 4:40 PM
Unknown Object (File)
Oct 1 2024, 10:24 AM
Unknown Object (File)
Sep 28 2024, 9:07 AM
Unknown Object (File)
Sep 27 2024, 2:19 PM

Details

Summary

Extend the rc framework to generate boottrace entries around the execution
of each rc script, if enabled.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

0mp added inline comments.
libexec/rc/rc.subr
1446
2221

if we use command -v here we don't have to spawn an extra process I think.

libexec/rc/rc.subr
1414–1416

Now that I think about it, a variable name starting with rc_ would fit better. e.g., rc_boottrace.

2222

Quoting is always a good idea here. Especially if one day we decide that boottrace_cmd is configurable.

0mp added inline comments.
libexec/rc/rc.subr
2223

Last minor thing would be to document that this variable is available. Ideally in rc.subr(8) where rc_* variables like rc_arg and rc_flags are documented.

Also, would be nice to add boottrace(4) to either rc or rc.subr.

This revision is now accepted and ready to land.Feb 18 2022, 9:18 AM
This revision was automatically updated to reflect the committed changes.
libexec/rc/rc.subr
2223

Sure, I've made a note to do this as a follow-up.