A later change will want to "properly" reboot (i.e. cleanup devsw
bits), so move it there now. A devsw stub is added to boot1 to keep
it linking successfully; libefi already uses the devsw global in
places, but apparently these bits are not used in the final link.
Details
Details
Diff Detail
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 42335 Build 39223: arc lint + arc unit
Event Timeline
stand/efi/libefi/libefi.c | ||
---|---|---|
66 | I think this is a layering violation... devsw is generally unknown to libefi and to most of the other interfaces to boot loader libraries. |
stand/efi/libefi/libefi.c | ||
---|---|---|
66 | We had committed this sin once in devicename.c already; no objection to just moving this back and putting the new command proper into loader/, though |
stand/efi/libefi/libefi.c | ||
---|---|---|
66 | Ah, all that's moving up is knowing that a devsw exists... The struct devsw is widely used itself, so it's less bad than I thought. |