Page MenuHomeFreeBSD
Differential D5134

Correct the memory rman ranges to be to BUS_SPACE_MAXADDR
ClosedPublic
Actions

Authored by jhibbits on Jan 30 2016, 3:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Dec 21, 4:59 PM
Unknown Object (File)
Sat, Nov 30, 4:37 PM
Unknown Object (File)
Nov 27 2024, 7:56 AM
Unknown Object (File)
Nov 21 2024, 6:39 PM
Unknown Object (File)
Oct 11 2024, 11:17 AM
Unknown Object (File)
Oct 4 2024, 5:17 AM
Unknown Object (File)
Oct 3 2024, 10:12 AM
Unknown Object (File)
Oct 1 2024, 2:34 PM
View All 76 Files
Subscribers
andrew
cem
emaste
imp

Details

Reviewers
cem
jhb
Commits
rS296250: Correct the memory rman ranges to be to BUS_SPACE_MAXADDR
Summary

As part of the migration of rman_res_t to be typed to uintmax_t, memory ranges
must be clamped appropriately for the bus, to prevent completely bogus addresses
from being used.

This is extracted from D4544.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jhibbits updated this revision to Diff 12875.Jan 30 2016, 3:53 PM
jhibbits retitled this revision from to Correct the memory rman ranges to be to BUS_SPACE_MAXADDR.
jhibbits updated this object.
jhibbits edited the test plan for this revision. (Show Details)
jhibbits added a reviewer: jhb.
Herald added subscribers: emaste, andrew, imp. · View Herald TranscriptJan 30 2016, 3:53 PM
jhibbits mentioned this in D5255: Replace all resource occurrences of '0UL/~0UL' with '0/~0'..Feb 11 2016, 2:51 PM
cem accepted this revision.Mar 1 2016, 2:56 AM
cem added a reviewer: cem.
cem added a subscriber: cem.

The only concern I had was around the PAE special case. jhibbits assured me it was at the suggestion of kib and jhb. Looks fine to me.

This revision is now accepted and ready to land.Mar 1 2016, 2:56 AM
Closed by commit rS296250: Correct the memory rman ranges to be to BUS_SPACE_MAXADDR (authored by jhibbits). · Explain WhyMar 1 2016, 2:59 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
arm/
arm/
5 lines
arm64/
arm64/
7 lines
mips/
mips/
4 lines
riscv/
riscv/
7 lines
sparc64/
sparc64/
2 lines
x86/
x86/
8 lines

Diff 13913

View Options

head/sys/arm/arm/nexus.c

Loading...
View Options

head/sys/arm64/arm64/nexus.c

Loading...
View Options

head/sys/mips/mips/nexus.c

Loading...
View Options

head/sys/riscv/riscv/nexus.c

Loading...
View Options

head/sys/sparc64/sparc64/nexus.c

Loading...
View Options

head/sys/x86/x86/nexus.c

Loading...