RFR(T) 8233363: Clarify the DumpSharedSpaces condition in InstanceKlass::verify_on
coleen.phillimore at oracle.com
coleen.phillimore at oracle.com
Fri Nov 1 17:27:46 UTC 2019
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