[profiler] Report GC roots outside of heapshots unless the nodefaults option is given.

Author: Alex Rønne Petersen

mono/profiler/log-args.c

@@ -69,6 +69,7 @@ parse_arg (const char *arg, ProfilerConfig *config)
 			//enables new style of default events, IE, nothing.
 		} else {
 			config->enable_mask = PROFLOG_EXCEPTION_EVENTS | PROFLOG_COUNTER_EVENTS;
+			config->always_do_root_report = TRUE;
 			compat_args_parsing = TRUE;
 		}
 	}

mono/profiler/log.c

@@ -1285,7 +1285,8 @@ gc_event (MonoProfiler *profiler, MonoProfilerGCEvent ev, uint32_t generation)
 		if (mono_atomic_load_i32 (&log_profiler.heapshot_requested))
 			log_profiler.do_heap_walk = TRUE;
 
-		if (ENABLED (PROFLOG_GC_ROOT_EVENTS) && log_profiler.do_heap_walk)
+		if (ENABLED (PROFLOG_GC_ROOT_EVENTS) &&
+		    (log_config.always_do_root_report || log_profiler.do_heap_walk))
 			mono_profiler_set_gc_roots_callback (log_profiler.handle, gc_roots);
 
 		/*

mono/profiler/log.h

@@ -512,6 +512,9 @@ typedef struct {
 	// Heapshot frequency in number of collections (for MONO_HEAPSHOT_X_GC). Can be changed at runtime.
 	unsigned int hs_freq_gc;
 
+	// Should root reports be done even outside of heapshots?
+	gboolean always_do_root_report;
+
 	// Whether to do a heapshot on shutdown.
 	gboolean hs_on_shutdown;