stan/jdk-24
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

8246405: Add GCLogPrecious functionality to log and report debug errors

Browse Source 
Reviewed-by: pliden, eosterlund
This commit is contained in:
Stefan Karlsson 2020-06-05 07:56:07 +02:00
parent aee74901f7
commit 1c27ce30b4
2 changed files with 53 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
src/hotspot/share/gc/shared/gcLogPrecious.cpp
View File

@ -49,6 +49,8 @@ void GCLogPrecious::vwrite_inner(LogTargetHandle log, const char* format, va_lis
// Log it to UL
log.print("%s", _temp->base());
// Leave _temp buffer to be used by vwrite_and_debug
}
void GCLogPrecious::vwrite(LogTargetHandle log, const char* format, va_list args) {
@ -56,6 +58,24 @@ void GCLogPrecious::vwrite(LogTargetHandle log, const char* format, va_list args
vwrite_inner(log, format, args);
}
void GCLogPrecious::vwrite_and_debug(LogTargetHandle log,
const char* format,
va_list args
DEBUG_ONLY(COMMA const char* file)
DEBUG_ONLY(COMMA int line)) {
DEBUG_ONLY(const char* debug_message;)
{
MutexLocker locker(_lock, Mutex::_no_safepoint_check_flag);
vwrite_inner(log, format, args);
DEBUG_ONLY(debug_message = strdup(_temp->base()));
}
// report error outside lock scope, since report_vm_error will call print_on_error
DEBUG_ONLY(report_vm_error(file, line, debug_message);)
DEBUG_ONLY(BREAKPOINT;)
}
void GCLogPrecious::print_on_error(outputStream* st) {
if (_lines != NULL) {
MutexLocker locker(_lock, Mutex::_no_safepoint_check_flag);

37
src/hotspot/share/gc/shared/gcLogPrecious.hpp
View File

@ -27,6 +27,7 @@
#include "utilities/globalDefinitions.hpp"
#include "logging/logHandle.hpp"
#include "memory/allocation.hpp"
#include "utilities/debug.hpp"
class Mutex;
class stringStream;
@ -34,8 +35,10 @@ class stringStream;
// Log lines to both unified logging and save them to a buffer.
// The lines will be printed when hs_err files are created.
#define log_level_p(level, ...) \
GCLogPreciousHandle(LogTargetHandle::create<LogLevel::level, LOG_TAGS(__VA_ARGS__)>())
#define log_level_p(level, ...) \
GCLogPreciousHandle( \
LogTargetHandle::create<LogLevel::level, LOG_TAGS(__VA_ARGS__)>() \
DEBUG_ONLY(COMMA __FILE__ COMMA __LINE__))
#define log_info_p(...) log_level_p(Info, __VA_ARGS__).write
#define log_debug_p(...) log_level_p(Debug, __VA_ARGS__).write
@ -43,6 +46,11 @@ class stringStream;
#define log_warning_p(...) log_level_p(Warning, __VA_ARGS__).write
#define log_error_p(...) log_level_p(Error, __VA_ARGS__).write
// ... and report error in debug builds
#define log_error_pd(...) \
DEBUG_ONLY(TOUCH_ASSERT_POISON;) \
log_level_p(Error, __VA_ARGS__).write_and_debug
class GCLogPrecious : public AllStatic {
private:
// Saved precious lines
@ -63,14 +71,28 @@ public:
const char* format,
va_list args) ATTRIBUTE_PRINTF(2, 0);
static void vwrite_and_debug(LogTargetHandle log,
const char* format,
va_list args
DEBUG_ONLY(COMMA const char* file)
DEBUG_ONLY(COMMA int line)) ATTRIBUTE_PRINTF(2, 0);
static void print_on_error(outputStream* st);
};
class GCLogPreciousHandle {
LogTargetHandle _log;
DEBUG_ONLY(const char* _file);
DEBUG_ONLY(int _line);
public:
GCLogPreciousHandle(LogTargetHandle log) : _log(log) {}
public:
GCLogPreciousHandle(LogTargetHandle log
DEBUG_ONLY(COMMA const char* file)
DEBUG_ONLY(COMMA int line)) :
_log(log)
DEBUG_ONLY(COMMA _file(file))
DEBUG_ONLY(COMMA _line(line))
{}
void write(const char* format, ...) ATTRIBUTE_PRINTF(2, 3) {
va_list args;
@ -78,6 +100,13 @@ public:
GCLogPrecious::vwrite(_log, format, args);
va_end(args);
}
void write_and_debug(const char* format, ...) ATTRIBUTE_PRINTF(2, 3) {
va_list args;
va_start(args, format);
GCLogPrecious::vwrite_and_debug(_log, format, args DEBUG_ONLY(COMMA _file COMMA _line));
va_end(args);
}
};
#endif // SHARE_GC_SHARED_GCLOGPRECIOUS_HPP