Page MenuHomeFreeBSD

reboot: Use posix_spawn instead of system
ClosedPublic

Authored by imp on Feb 13 2024, 4:50 AM.
Tags
None
Referenced Files
F125968390: D43860.id134238.diff
Wed, Aug 13, 6:48 PM
Unknown Object (File)
Sat, Aug 2, 9:03 PM
Unknown Object (File)
Sat, Aug 2, 7:37 PM
Unknown Object (File)
Mon, Jul 28, 6:55 AM
Unknown Object (File)
Wed, Jul 16, 12:00 PM
Unknown Object (File)
Jun 24 2025, 1:00 PM
Unknown Object (File)
Jun 23 2025, 11:53 PM
Unknown Object (File)
Jun 21 2025, 5:53 AM
Subscribers

Details

Summary

Use posix_spawn to avoid having to allocate memory needed for the system
command line.

Sponsored by: Netflix

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 56017
Build 52906: arc lint + arc unit

Event Timeline

imp requested review of this revision.Feb 13 2024, 4:50 AM
imp created this revision.
sbin/reboot/reboot.c
76

Shouldn't there only be one copy?

94–110

Check return value? Especially for EINTR.

104–106

Would seem more natural to flip these two cases?

sbin/reboot/reboot.c
76

Oh, yes.

94–110

Should we look on EINTR? Or just ignore it?

104–106

sure.

This revision is now accepted and ready to land.Feb 15 2024, 11:17 PM
This revision was automatically updated to reflect the committed changes.