Skip to content

Commit 51d544c

Browse files
author
Maoni0
committed
logging changes
1 parent db9114d commit 51d544c

File tree

2 files changed

+12
-12
lines changed

2 files changed

+12
-12
lines changed

src/coreclr/gc/gc.cpp

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -7086,12 +7086,12 @@ void gc_heap::gc_thread_function ()
70867086
}
70877087
#endif //DYNAMIC_HEAP_COUNT
70887088
uint32_t wait_result = gc_heap::ee_suspend_event.Wait(wait_on_time_out_p ? wait_time : INFINITE, FALSE);
7089-
dprintf (6666, ("waiting for ee_suspend_event done res %d (timeout %d, %I64d ms since last suspend end)(should_change_heap_count is %d) (gradual_decommit_in_progress_p %d)",
7089+
dprintf (9999, ("waiting for ee_suspend_event done res %d (timeout %d, %I64d ms since last suspend end)(should_change_heap_count is %d) (gradual_decommit_in_progress_p %d)",
70907090
wait_result, wait_time, ((GetHighPrecisionTimeStamp() - last_suspended_end_time) / 1000),
70917091
dynamic_heap_count_data.should_change_heap_count, gradual_decommit_in_progress_p));
70927092
if (wait_result == WAIT_TIMEOUT)
70937093
{
7094-
dprintf (6666, ("timeout!"));
7094+
dprintf (9999, ("timeout!"));
70957095
#ifdef DYNAMIC_HEAP_COUNT
70967096
if (dynamic_heap_count_data.should_change_heap_count)
70977097
{
@@ -7129,7 +7129,7 @@ void gc_heap::gc_thread_function ()
71297129
//gc_heap::ee_suspend_event.Set ();
71307130
}
71317131

7132-
dprintf (6666, ("now checking if we need to wait for idle"));
7132+
dprintf (9999, ("now checking if we need to wait for idle"));
71337133

71347134
// wait till the threads that should have gone idle at least reached the place where they are about to wait on the idle event.
71357135
if ((gc_heap::dynamic_adaptation_mode == dynamic_adaptation_to_application_sizes) &&
@@ -7183,16 +7183,16 @@ void gc_heap::gc_thread_function ()
71837183
}
71847184
else
71857185
{
7186-
dprintf (6666, ("GC thread %d waiting_for_gc_start(%d)(gc%Id)", heap_number, n_heaps, VolatileLoadWithoutBarrier(&settings.gc_index)));
7186+
dprintf (9999, ("GC thread %d waiting_for_gc_start(%d)(gc%Id)", heap_number, n_heaps, VolatileLoadWithoutBarrier(&settings.gc_index)));
71877187
gc_start_event.Wait(INFINITE, FALSE);
71887188
#ifdef DYNAMIC_HEAP_COUNT
7189-
dprintf (6666, ("GC thread %d waiting_done_gc_start(%d-%d)(i: %d)(gc%Id)",
7189+
dprintf (9999, ("GC thread %d waiting_done_gc_start(%d-%d)(i: %d)(gc%Id)",
71907190
heap_number, n_heaps, dynamic_heap_count_data.new_n_heaps, dynamic_heap_count_data.init_only_p, VolatileLoadWithoutBarrier (&settings.gc_index)));
71917191

71927192
if ((gc_heap::dynamic_adaptation_mode == dynamic_adaptation_to_application_sizes) &&
71937193
(dynamic_heap_count_data.new_n_heaps != n_heaps))
71947194
{
7195-
dprintf (6666, ("changing heap count on non h0 thread!"));
7195+
dprintf (9999, ("changing heap count on non h0 thread!"));
71967196
// The reason why we need to do this is -
71977197
// + for threads that were participating, we need them to do work for change_heap_count
71987198
// + for threads that were not participating but will need to participate, we need to make sure they are woken now instead of
@@ -7219,7 +7219,7 @@ void gc_heap::gc_thread_function ()
72197219
dprintf (9999, ("GC thread %d wait_on_idle(%d < %d)(gc%Id), total idle %d", heap_number, old_n_heaps, new_n_heaps,
72207220
VolatileLoadWithoutBarrier (&settings.gc_index), VolatileLoadWithoutBarrier (&dynamic_heap_count_data.idle_thread_count)));
72217221
gc_idle_thread_event.Wait (INFINITE, FALSE);
7222-
dprintf (6666, ("GC thread %d waking_from_idle(%d)(gc%Id) after doing change", heap_number, n_heaps, VolatileLoadWithoutBarrier (&settings.gc_index)));
7222+
dprintf (9999, ("GC thread %d waking_from_idle(%d)(gc%Id) after doing change", heap_number, n_heaps, VolatileLoadWithoutBarrier (&settings.gc_index)));
72237223
}
72247224
}
72257225
else
@@ -25746,7 +25746,7 @@ bool gc_heap::prepare_to_change_heap_count (int new_n_heaps)
2574625746

2574725747
bool gc_heap::change_heap_count (int new_n_heaps)
2574825748
{
25749-
dprintf (6666, ("BEG heap%d changing %d->%d", heap_number, n_heaps, new_n_heaps));
25749+
dprintf (9999, ("BEG heap%d changing %d->%d", heap_number, n_heaps, new_n_heaps));
2575025750

2575125751
// use this variable for clarity - n_heaps will change during the transition
2575225752
int old_n_heaps = n_heaps;
@@ -25762,7 +25762,7 @@ bool gc_heap::change_heap_count (int new_n_heaps)
2576225762
#endif //BACKGROUND_GC
2576325763

2576425764
dynamic_heap_count_data.init_only_p = false;
25765-
dprintf (6666, ("in change h%d resetting gc_start, update bgc join to %d heaps", heap_number, new_n_heaps));
25765+
dprintf (9999, ("in change h%d resetting gc_start, update bgc join to %d heaps", heap_number, new_n_heaps));
2576625766
gc_start_event.Reset();
2576725767
gc_t_join.restart ();
2576825768
}
@@ -26069,7 +26069,7 @@ bool gc_heap::change_heap_count (int new_n_heaps)
2606926069
}
2607026070
}
2607126071

26072-
dprintf (6666, ("END heap%d changed %d->%d", heap_number, n_heaps, new_n_heaps));
26072+
dprintf (9999, ("END heap%d changed %d->%d", heap_number, n_heaps, new_n_heaps));
2607326073
return true;
2607426074
}
2607526075

@@ -50162,7 +50162,7 @@ void gc_heap::do_post_gc()
5016250162
#endif //DYNAMIC_HEAP_COUNT
5016350163

5016450164
#if defined(TRACE_GC) && defined(SIMPLE_DPRINTF)
50165-
//flush_gc_log (false);
50165+
flush_gc_log (false);
5016650166
#endif //TRACE_GC && SIMPLE_DPRINTF
5016750167

5016850168
// Now record the gc info.

src/coreclr/gc/gcpriv.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -157,7 +157,7 @@ inline void FATAL_GC_ERROR()
157157
#if defined(USE_REGIONS) && defined(MULTIPLE_HEAPS)
158158
// can only change heap count with regions
159159
#define DYNAMIC_HEAP_COUNT
160-
#define STRESS_DYNAMIC_HEAP_COUNT
160+
//#define STRESS_DYNAMIC_HEAP_COUNT
161161
#endif //USE_REGIONS && MULTIPLE_HEAPS
162162

163163
#ifdef USE_REGIONS

0 commit comments

Comments
 (0)