Page MenuHomeFreeBSD
Differential D12849

efipart_strategy is using wrong offset with >512B sectors
ClosedPublic
Actions

Authored by tsoome on Oct 31 2017, 11:53 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 10, 3:12 AM
Unknown Object (File)
Mon, Nov 3, 6:09 AM
Unknown Object (File)
Mon, Nov 3, 6:09 AM
Unknown Object (File)
Mon, Nov 3, 6:09 AM
Unknown Object (File)
Mon, Nov 3, 6:08 AM
Unknown Object (File)
Mon, Nov 3, 2:23 AM
Unknown Object (File)
Tue, Oct 28, 4:56 PM
Unknown Object (File)
Tue, Oct 21, 8:37 AM
View All Files
Subscribers
None

Details

Reviewers
allanjude
imp
eric_metricspace.net
Commits
rS325286: efipart_strategy is using wrong offset with >512B sectors
Summary

The strategy() calls are assuming 512B sectors, so we need to adjust the
offset accordingly.

Diff Detail

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

Event Timeline

tsoome created this revision.Oct 31 2017, 11:53 AM
Harbormaster completed remote builds in B12326: Diff 34502.Oct 31 2017, 11:53 AM
imp accepted this revision.Oct 31 2017, 3:11 PM
imp added inline comments.
sys/boot/efi/libefi/efipart.c
880–884

why shift and not offset?

This revision is now accepted and ready to land.Oct 31 2017, 3:11 PM
tsoome updated this revision to Diff 34567.Oct 31 2017, 8:22 PM

offset is better name.

This revision now requires review to proceed.Oct 31 2017, 8:22 PM
imp accepted this revision.Oct 31 2017, 8:29 PM
This revision is now accepted and ready to land.Oct 31 2017, 8:29 PM
Closed by commit rS325286: efipart_strategy is using wrong offset with >512B sectors (authored by tsoome). · Explain WhyNov 1 2017, 6:50 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
sys/
boot/
efi/
libefi/
6 lines

Diff 34567

View Options

sys/boot/efi/libefi/efipart.c

Loading...