Page MenuHomeFreeBSD

loader: always set media size from partition.
ClosedPublic

Authored by tsoome on Nov 3 2018, 12:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Dec 23, 7:55 PM
Unknown Object (File)
Dec 3 2024, 9:55 AM
Unknown Object (File)
Dec 3 2024, 9:55 AM
Unknown Object (File)
Dec 3 2024, 9:55 AM
Unknown Object (File)
Dec 3 2024, 9:54 AM
Unknown Object (File)
Dec 3 2024, 9:43 AM
Unknown Object (File)
Nov 26 2024, 10:06 PM
Unknown Object (File)
Nov 24 2024, 3:29 PM

Details

Summary

The disk access is validated by using partition table definitions, therefore
we have no need for if statements, just set the disk size.

Of course the partition table itself may be incorrect/inconsistent, but if
so, we are in trouble anyhow.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 20606
Build 20027: arc lint + arc unit

Event Timeline

Seems legit to me. I like the abstraction even if we don't determine the size in multiple places.

juan.molina_club.fr added inline comments.
stand/i386/libi386/biosdisk.c
445

Typo: mulitplication

tsoome added inline comments.
stand/i386/libi386/biosdisk.c
445

Yes, thanks:)

tsoome marked an inline comment as done.

Typo fixed.

This revision was not accepted when it landed; it landed in state Needs Review.Nov 7 2018, 11:14 AM
This revision was automatically updated to reflect the committed changes.