RFR(xxs): 8185262: Keep default logging for Metaspace OOM short and concise
Thomas Stüfe
thomas.stuefe at gmail.com
Wed Jul 26 17:51:39 UTC 2017
Thanks Coleen!
On Wed, Jul 26, 2017 at 6:11 PM, <coleen.phillimore at oracle.com> wrote:
>
> Hi Thomas,
> version .01 looks good with Ioi's suggestion. I'll sponsor it.
> thanks,
> Coleen
>
> On 7/25/17 1:29 PM, Ioi Lam wrote:
>
>> Sounds good. Thanks Thomas.
>>
>> - Ioi
>>
>>
>> On 7/25/17 10:25 AM, Thomas Stüfe wrote:
>>
>>> Hi Ioi,
>>>
>>> On Tue, Jul 25, 2017 at 7:12 PM, Ioi Lam <ioi.lam at oracle.com <mailto:
>>> ioi.lam at oracle.com>> wrote:
>>>
>>> Hi Thomas,
>>>
>>> how about switching lines 3739 and 3740?
>>>
>>> 3738 if (log.is_debug()) {
>>> 3739 LogStream ls(log.debug());
>>> 3740 if (loader_data->metaspace_or_null() != NULL) {
>>> 3741 loader_data->dump(&ls);
>>> 3742 }
>>> 3743 }
>>>
>>>
>>> sure.
>>>
>>> What kind of tests have you run?
>>>
>>>
>>> Besides my tiny test case causing metaspace oom? gtests on Windows x64,
>>> Linux x64/ppc and AIX, and, on Linux x64, jtreg tests from
>>> hotspot/runtime/Metaspace. All with fastdebug builds.
>>>
>>> I also looked around whether any test parses the log output, but could
>>> not find any code setting gc+metaspace+freelist.
>>>
>>> Thanks, Thomas
>>>
>>> Thanks
>>> - Ioi
>>>
>>>
>>> On 7/25/17 10:07 AM, Thomas Stüfe wrote:
>>>
>>> Hi all,
>>>
>>> may I have please reviews for this very tiny change:
>>>
>>> bug: https://bugs.openjdk.java.net/browse/JDK-8185262
>>> <https://bugs.openjdk.java.net/browse/JDK-8185262>
>>> webrev: http://cr.openjdk.java.net/~stuefe/webrevs/
>>> <http://cr.openjdk.java.net/%7Estuefe/webrevs/>
>>> 8185262-keep-metaspace-oom-log-short-for-default/webrev.00/webrev/
>>>
>>> On Metaspace OOM, we dump all metaspace chunks in use. This
>>> can be very
>>> lengthy. It would be nice if this part of logging were
>>> restricted to a
>>> higher loglevel. In this change, I restrict it to debug.
>>>
>>> Kind Regards, Thomas
>>>
>>>
>>>
>>>
>>
>
More information about the hotspot-dev
mailing list