loader ptblread() is broken with >512B sectors
ClosedPublic
Actions

Authored by tsoome on Oct 31 2017, 11:48 AM.

Tags

None

Referenced Files

	F148495237: D12847.diff
	Wed, Mar 18, 6:19 AM

	F148495230: D12847.diff
	Wed, Mar 18, 6:19 AM

	F148454169: D12847.id34568.diff
	Wed, Mar 18, 12:12 AM

	F148454157: D12847.id34568.diff
	Wed, Mar 18, 12:12 AM

	Unknown Object (File)
	Mon, Mar 9, 10:38 AM

	Unknown Object (File)
	Thu, Feb 26, 4:30 PM

	Unknown Object (File)
	Wed, Feb 18, 3:06 AM

	Unknown Object (File)
	Dec 25 2025, 1:23 PM

View All Files

Subscribers

None

Details

Reviewers

allanjude
imp

Commits

rS325248: loader ptblread() is broken with >512B sectors

Summary

The loader strategy() function is assuming 512B blocks, so we need to adjust
ptblread() for other sector sizes.

Test Plan

use lsdev -v from loader OK prompt to confirm the partition table.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tsoome created this revision.Oct 31 2017, 11:48 AM

I will test this on my macbookpro NVMe where FreeBSD currently cannot read the partition table.

This looks good to me.... It will certainly be a NOP for 512 byte sectors.

This revision is now accepted and ready to land.Oct 31 2017, 3:10 PM

Reword the comment.

This revision now requires review to proceed.Oct 31 2017, 8:05 PM

Closed by commit rS325248: loader ptblread() is broken with >512B sectors (authored by tsoome). · Explain WhyOct 31 2017, 8:29 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

boot/

common/

disk.c

6 lines

Diff 34568

View Options

head/sys/boot/common/disk.c

loader ptblread() is broken with >512B sectorsClosedPublicActions