RFR(T) 8233363: Clarify the DumpSharedSpaces condition in InstanceKlass::verify_on
Calvin Cheung
calvin.cheung at oracle.com
Fri Nov 1 18:09:52 UTC 2019
Thanks, Coleen!
On 11/1/19 10:27 AM, coleen.phillimore at oracle.com wrote:
> Me too. thanks for fixing this one.
> Coleen
>
> On 11/1/19 1:15 PM, Calvin Cheung wrote:
>> Thanks!
>>
>> On 11/1/19 9:44 AM, Ioi Lam wrote:
>>> Hi Calvin, this looks good and trivial to me.
>>>
>>> Thanks
>>> Ioi
>>>
>>> Sent from my iPad
>>>
>>>> On Nov 1, 2019, at 9:31 AM, Calvin Cheung
>>>> <Calvin.Cheung at oracle.com> wrote:
>>>>
>>>> bug: https://bugs.openjdk.java.net/browse/JDK-8233363
>>>>
>>>> Summary: change DumpSharedSpaces to Arguments::is_dumping_archive().
>>>>
>>>> bash-4.2$ hg diff src/hotspot/share/oops/instanceKlass.cpp
>>>> diff --git a/src/hotspot/share/oops/instanceKlass.cpp
>>>> b/src/hotspot/share/oops/instanceKlass.cpp
>>>> --- a/src/hotspot/share/oops/instanceKlass.cpp
>>>> +++ b/src/hotspot/share/oops/instanceKlass.cpp
>>>> @@ -3626,7 +3626,7 @@
>>>> Array<int>* method_ordering = this->method_ordering();
>>>> int length = method_ordering->length();
>>>> if (JvmtiExport::can_maintain_original_method_order() ||
>>>> - ((UseSharedSpaces || DumpSharedSpaces) && length != 0)) {
>>>> + ((UseSharedSpaces || Arguments::is_dumping_archive()) &&
>>>> length != 0)) {
>>>> guarantee(length == methods()->length(), "invalid method
>>>> ordering length");
>>>> jlong sum = 0;
>>>> for (int j = 0; j < length; j++) {
>>>>
>>>> Ran CDS and AppCDS tests locally on linux-x64.
>>>>
>>>> thanks,
>>>>
>>>> Calvin
>>>>
>
More information about the hotspot-runtime-dev
mailing list