Page MenuHomeFreeBSD

boottrace: rc annotations
ClosedPublic

Authored by mhorne on Sep 13 2021, 3:19 PM.
Tags
None
Referenced Files
F108659807: D31930.diff
Mon, Jan 27, 3:29 AM
Unknown Object (File)
Sat, Jan 25, 7:56 PM
Unknown Object (File)
Mon, Jan 20, 7:09 AM
Unknown Object (File)
Sat, Jan 18, 4:32 AM
Unknown Object (File)
Fri, Jan 17, 3:13 PM
Unknown Object (File)
Fri, Jan 17, 1:27 PM
Unknown Object (File)
Fri, Jan 17, 12:13 PM
Unknown Object (File)
Fri, Jan 17, 2:56 AM

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 41481
Build 38370: 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.