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