pkg/manager: improve patched crash skip rules

The rules were too generic - there are plenty of functions that have
"stall" as substring, yet they do not relate to rcu stall bugs.

Improve the rules list and add a unit test.

1 files changed, 13 insertions, 0 deletions

diff --git a/pkg/manager/diff_test.go b/pkg/manager/diff_test.go
index 9e27dc288..454a92831 100644
--- a/pkg/manager/diff_test.go
+++ b/pkg/manager/diff_test.go
@@ -107,3 +107,16 @@ func TestModifiedSymbols(t *testing.T) {
 		assert.Equal(t, []string{"function", "function2"}, modifiedSymbols(base, patched))
 	})
 }
+
+func TestSkipDiffRepro(t *testing.T) {
+	for title, skip := range map[string]bool{
+		"no output from test machine":                          true,
+		"SYZFAIL: read failed":                                 true,
+		"lost connection to test machine":                      true,
+		"INFO: rcu detected stall in clone":                    true,
+		"WARNING in arch_install_hw_breakpoint":                false,
+		"KASAN: slab-out-of-bounds Write in __bpf_get_stackid": false,
+	} {
+		assert.Equal(t, skip, skipDiffRepro(title), "title=%q", title)
+	}
+}