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
F110491518: D12849.id34502.diff
Wed, Feb 19, 4:22 AM
Unknown Object (File)
Jan 9 2025, 7:45 AM
Unknown Object (File)
Dec 27 2024, 5:14 PM
Unknown Object (File)
Dec 12 2024, 10:08 PM
Unknown Object (File)
Nov 19 2024, 7:30 AM
Unknown Object (File)
Nov 19 2024, 6:06 AM
Unknown Object (File)
Nov 19 2024, 5:46 AM
Unknown Object (File)
Nov 19 2024, 3:07 AM
View All 54 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 12326
Build 12615: 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

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 34502

View Options

sys/boot/efi/libefi/efipart.c

Loading...