Page MenuHomeFreeBSD

Loader: explain currdev syntax
ClosedPublic

Authored by allanjude on Jul 11 2020, 7:52 PM.

Details

Summary

The loader(8) man page doesn't explain how you correctly format currdev

Diff Detail

Repository
rS FreeBSD src repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

This revision is now accepted and ready to land.Jul 11 2020, 7:54 PM
greg_unrelenting.technology added inline comments.
stand/man/loader.8
486 ↗(On Diff #74334)

huh, what's with the : at the end? I don't remember ever writing it

kevans added inline comments.
stand/man/loader.8
486 ↗(On Diff #74334)

I was under the impression it's to remove ambiguity, because we may literally glue currdev and a path together for,e.g., open(). Omitting it could cause hilarityto ensue, I believe.

stand/man/loader.8
486 ↗(On Diff #74334)

Yeah, because the path ends up being like: zfs:dataset:/boot/kernel/kernel etc
The : at the end is very required.

For ZFS, 'set currdev=zfs:dataset' will return an error, saying it is invalid without the trailing :

This revision was automatically updated to reflect the committed changes.