Addressed flakiness on some tests.

Author: VladSaiocUber

src/runtime/gc_test.go

@@ -1217,10 +1217,9 @@ func TestGoroutineLeakGC(t *testing.T) {
 		makeTest(testCase{name: "Cockroach3710"},
 			`main\.Cockroach3710\.func2\.gowrap1 .* \[sync\.RWMutex\.RLock\]`,
 			`main\.\(\*Store_cockroach3710\)\.processRaft\.func1 .* \[sync\.RWMutex\.Lock\]`),
-		makeTest(testCase{name: "Cockroach6181", repetitions: 50},
+		makeTest(testCase{name: "Cockroach6181", repetitions: 20},
 			`main\.testRangeCacheCoalescedRequests_cockroach6181 .* \[sync\.WaitGroup\.Wait\]`,
-			`main\.testRangeCacheCoalescedRequests_cockroach6181\.func1\.1 .* \[sync\.Mutex\.Lock\]`,
-			`main\.testRangeCacheCoalescedRequests_cockroach6181\.func1\.1 .* \[sync\.RWMutex\.Lock\]`,
+			`main\.testRangeCacheCoalescedRequests_cockroach6181\.func1\.1 .* \[sync\.(RW)?Mutex\.Lock\]`,
 			`main\.testRangeCacheCoalescedRequests_cockroach6181\.func1\.1 .* \[sync\.RWMutex\.RLock\]`),
 		makeTest(testCase{name: "Cockroach7504", repetitions: 100},
 			`main\.Cockroach7504\.func2\.1 .* \[sync\.Mutex\.Lock\]`,
@@ -1303,6 +1302,7 @@ func TestGoroutineLeakGC(t *testing.T) {
 		makeTest(testCase{name: "Hugo5379"},
 			`main\.\(\*Page_hugo5379\)\.initContent\.func1\.1 .* \[sync\.Mutex\.Lock\]`,
 			`main\.\(\*Site_hugo5379\)\.renderPages\.gowrap1 .* \[sync\.Mutex\.Lock\]`,
+			`main\.\(\*Page_hugo5379\)\.initContentPlainAndMeta(\.\(\*Page_hugo5379\)\.initContent)?\.func1\.1 .* \[sync\.Mutex\.Lock\]`,
 			`main\.Hugo5379\.func2 .* \[sync\.WaitGroup\.Wait\]`),
 		makeTest(testCase{name: "Istio16224"},
 			`main\.Istio16224\.func2 .* \[sync\.Mutex\.Lock\]`,
@@ -1315,9 +1315,14 @@ func TestGoroutineLeakGC(t *testing.T) {
 		makeTest(testCase{name: "Istio18454"},
 			`main\.\(\*Worker_istio18454\)\.Start\.func1 .* \[chan receive\]`,
 			`main\.\(\*Worker_istio18454\)\.Start\.func1 .* \[chan send\]`),
-		makeTest(testCase{name: "Kubernetes1321"},
-			`main\.NewMux_kubernetes1321\.gowrap1 .* \[chan send\]`,
-			`main\.testMuxWatcherClose_kubernetes1321 .* \[sync\.Mutex\.Lock\]`),
+		// NOTE (vsaioc):
+		// Kubernetes/1321 is excluded due to a race condition in the original program
+		// that may, in very rare cases, lead to nil pointer dereference crashes.
+		// (Reproducible even with regular GC).
+		//
+		// makeTest(testCase{name: "Kubernetes1321"},
+		// 	`main\.NewMux_kubernetes1321\.gowrap1 .* \[chan send\]`,
+		// 	`main\.testMuxWatcherClose_kubernetes1321 .* \[sync\.Mutex\.Lock\]`),
 		makeTest(testCase{name: "Kubernetes5316"},
 			`main\.finishRequest_kubernetes5316\.func1 .* \[chan send\]`),
 		makeTest(testCase{name: "Kubernetes6632"},
@@ -1327,7 +1332,7 @@ func TestGoroutineLeakGC(t *testing.T) {
 			`main\.\(\*statusManager_kubernetes10182\)\.Start\.func1 .* \[sync\.Mutex\.Lock\]`,
 			`main\.Kubernetes10182\.func2\.gowrap2 .* \[chan send\]`,
 			`main\.Kubernetes10182\.func2\.gowrap3 .* \[chan send\]`),
-		makeTest(testCase{name: "Kubernetes11298"},
+		makeTest(testCase{name: "Kubernetes11298", repetitions: 10},
 			`main\.After_kubernetes11298\.func1 .* \[chan receive\]`,
 			`main\.After_kubernetes11298\.func1 .* \[sync\.Cond\.Wait\]`,
 			`main\.Kubernetes11298\.func2 .* \[chan receive\]`),
@@ -1345,11 +1350,9 @@ func TestGoroutineLeakGC(t *testing.T) {
 			`main\.\(\*NamespaceController_kubernetes30872\)\.Run\.func1 .* \[sync\.Mutex\.Lock\]`),
 		makeTest(testCase{name: "Kubernetes38669"},
 			`main\.newCacheWatcher_kubernetes38669\.gowrap1 .* \[chan send\]`),
-		makeTest(testCase{name: "Kubernetes58107"},
-			`main\.\(\*ResourceQuotaController_kubernetes58107\)\.Run\.gowrap1 .* \[sync\.Cond\.Wait\]`,
-			`main\.\(\*ResourceQuotaController_kubernetes58107\)\.Run\.gowrap1 .* \[sync\.RWMutex\.RLock\]`,
-			`main\.\(\*ResourceQuotaController_kubernetes58107\)\.Run\.gowrap2 .* \[sync\.Cond\.Wait\]`,
-			`main\.\(\*ResourceQuotaController_kubernetes58107\)\.Run\.gowrap2 .* \[sync\.RWMutex\.RLock\]`,
+		makeTest(testCase{name: "Kubernetes58107", repetitions: 5},
+			`main\.\(\*ResourceQuotaController_kubernetes58107\)\.Run\.gowrap[12] .* \[sync\.Cond\.Wait\]`,
+			`main\.\(\*ResourceQuotaController_kubernetes58107\)\.Run\.gowrap[12] .* \[sync\.RWMutex\.RLock\]`,
 			`main\.startResourceQuotaController_kubernetes58107\.gowrap2 .* \[sync\.RWMutex\.Lock\]`),
 		makeTest(testCase{name: "Kubernetes62464"},
 			`main\.Kubernetes62464\.func2\.gowrap1 .* \[sync\.RWMutex\.RLock\]`,
@@ -1358,7 +1361,7 @@ func TestGoroutineLeakGC(t *testing.T) {
 			`main\.Kubernetes70277\.func2 .* \[chan receive\]`),
 		makeTest(testCase{name: "Moby4395"},
 			`main\.Go_moby4395\.func1 .* \[chan send\]`),
-		makeTest(testCase{name: "Moby4951"},
+		makeTest(testCase{name: "Moby4951", repetitions: 10},
 			`main\.Moby4951\.func2\.gowrap1 .* \[sync\.Mutex\.Lock\]`,
 			`main\.Moby4951\.func2\.gowrap2 .* \[sync\.Mutex\.Lock\]`),
 		makeTest(testCase{name: "Moby7559"},