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
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 44193
Build 41081: arc lint + arc unit

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.