Backports
| Issue | Fix Version | Assignee | Priority | Status | Resolution | Resolved In Build |
|---|---|---|---|---|---|---|
| JDK-2216147 | 8 | John Cuthbertson | P4 | Resolved | Fixed | b10 |
| JDK-2219681 | 7u4 | John Cuthbertson | P4 | Closed | Fixed | b01 |
Description
As part of the review comments for 7092245, it was suggested that a more complete solution would be to redeclare HeapRegion::GrainBytes as a size_t. Currently it is declared as an int.
The size_t type is unsigned. So while it is unlikely we will need to be able to represent a heap region size > 2G in the next decade or so, changing the declaration will help type checking.
There are 2 other values associated with heap region size that should probably be changed from int to size_t too: GrainWords and CardsPerRegion.
The size_t type is unsigned. So while it is unlikely we will need to be able to represent a heap region size > 2G in the next decade or so, changing the declaration will help type checking.
There are 2 other values associated with heap region size that should probably be changed from int to size_t too: GrainWords and CardsPerRegion.
Attachments
Issue Links
- backported by
-
JDK-2216147 G1: HeapRegion::GrainBytes, GrainWords, and CardsPerRegion should be size_t
-
- Resolved
-
-
-
- Closed
-
- relates to
-
JDK-8275886 G1: remove obsolete comment in HeapRegion::setup_heap_region_size
-
- Resolved
-