-
Enhancement
-
Resolution: Fixed
-
P4
-
hs19
Issue | Fix Version | Assignee | Priority | Status | Resolution | Resolved In Build |
---|---|---|---|---|---|---|
JDK-2193995 | 7 | Y. Ramakrishna | P4 | Closed | Fixed | b96 |
JDK-2197883 | 6u23 | Y. Ramakrishna | P4 | Resolved | Fixed | b01 |
JDK-2199784 | 6u22m | Y. Ramakrishna | P4 | Resolved | Fixed | b01 |
JDK-2197589 | 6u21p | Y. Ramakrishna | P4 | Resolved | Fixed | b03 |
From email thread on openjdk:-
Vladimir Kozlov wrote:
> The file also have LP64 instead of _LP64:
>
> src/share/vm/gc_interface/collectedHeap.cpp:#ifdef LP64
>
Yes, that one is probably not fatal (as far as i can see because
i think LP64 gets defined when building 64-bit and this has been around
a lot longer than COMPLER2; i'll check), but we should fix that too...
thanks Vladimir.
-- ramki
> Vladimir
>
> Y. Srinivas Ramakrishna wrote:
>> 6953483 Typo related to ReduceInitialCardMarks
>>
>> Workaround for the case of CMS: -XX:-ReduceInitialCardMarks.
>>
>> No issue for other collectors (G1, which may otherwise have been
>> vulnerable, has this optimization switched off).
>>
>> Thanks, ChenGuang, for bringing this to our attention.
>> -- ramki
>>
>> Y. Srinivas Ramakrishna wrote:
>>> Looks like I am responsible for that typo (which is still there).
>>> I'll test and fix that issue. Thanks for the fix!
>>>
>>> -- ramki
>>>
>>> ChenGuang Sun wrote:
>>>> Hi,
>>>>
>>>> I'm reading the code and have noticed a minor typo in the function
>>>> "CollectedHeap::pre_initialize()".
>>>>
>>>> I think it should be "#ifdef COMPILER2" instead of "#ifdef
>>>> COMPLER2". I haven't checked whether it has been corrected in the
>>>> latest revision.
>>>>
>>>> Thanks,
>>>> Chen
Vladimir Kozlov wrote:
> The file also have LP64 instead of _LP64:
>
> src/share/vm/gc_interface/collectedHeap.cpp:#ifdef LP64
>
Yes, that one is probably not fatal (as far as i can see because
i think LP64 gets defined when building 64-bit and this has been around
a lot longer than COMPLER2; i'll check), but we should fix that too...
thanks Vladimir.
-- ramki
> Vladimir
>
> Y. Srinivas Ramakrishna wrote:
>> 6953483 Typo related to ReduceInitialCardMarks
>>
>> Workaround for the case of CMS: -XX:-ReduceInitialCardMarks.
>>
>> No issue for other collectors (G1, which may otherwise have been
>> vulnerable, has this optimization switched off).
>>
>> Thanks, ChenGuang, for bringing this to our attention.
>> -- ramki
>>
>> Y. Srinivas Ramakrishna wrote:
>>> Looks like I am responsible for that typo (which is still there).
>>> I'll test and fix that issue. Thanks for the fix!
>>>
>>> -- ramki
>>>
>>> ChenGuang Sun wrote:
>>>> Hi,
>>>>
>>>> I'm reading the code and have noticed a minor typo in the function
>>>> "CollectedHeap::pre_initialize()".
>>>>
>>>> I think it should be "#ifdef COMPILER2" instead of "#ifdef
>>>> COMPLER2". I haven't checked whether it has been corrected in the
>>>> latest revision.
>>>>
>>>> Thanks,
>>>> Chen
- backported by
-
JDK-2197589 collectedHeap.cpp should use #ifdef _LP64 not LP64
-
- Resolved
-
-
JDK-2197883 collectedHeap.cpp should use #ifdef _LP64 not LP64
-
- Resolved
-
-
JDK-2199784 collectedHeap.cpp should use #ifdef _LP64 not LP64
-
- Resolved
-
-
JDK-2193995 collectedHeap.cpp should use #ifdef _LP64 not LP64
-
- Closed
-
- relates to
-
JDK-6578152 fill_region_with_object has usability and safety issues
-
- Resolved
-