Skip to content

Commit

Permalink
pkg/report: drop "ALT: KMSAN origin" titles for uninit reports
Browse files Browse the repository at this point in the history
It was initially proposed in
#1575 that KMSAN reports with
the same origin should be clustered together using an alt title.

This however turns out to be too aggressive: certain KMSAN reports have
their uninitialized values originating from common functions - this
leads to too many KMSAN reports being glued together. Because KMSAN
reports can be also clustered with KASAN reports or other kernel panics,
ultimately seemingly unrelated crashes are considered similar just
because they share their top frames with two KMSAN reports that, in
turn, share the same origin.

The resulting issues on the dashboard look confusing to the users, they
are hard to find and require manual untangling, which probably outweighs
the benefits of having KMSAN issues with exactly the same origin
clustered together.

For other types of KMSAN reports (infoleaks and use-after-frees) the alt
titles are preserved. First, there are fewer of those on the dashboard.
Second, they are rarely grouped together with non-KASAN reports and
are less likely to cause a lot of mess.
  • Loading branch information
ramosian-glider authored and a-nogikh committed Aug 31, 2023
1 parent 4ad3538 commit 696ea0d
Show file tree
Hide file tree
Showing 21 changed files with 0 additions and 21 deletions.
1 change: 0 additions & 1 deletion pkg/report/linux.go
Original file line number Diff line number Diff line change
Expand Up @@ -1374,7 +1374,6 @@ var linuxOopses = append([]*oops{
fmt: "KMSAN: %[1]v in %[3]v",
alt: []string{
"bad-access in %[3]v",
"KMSAN origin in %[4]v",
},
stack: &stackFmt{
parts: []*regexp.Regexp{
Expand Down
1 change: 0 additions & 1 deletion pkg/report/testdata/linux/report/626
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
TITLE: KMSAN: uninit-value in prepare_task_switch
ALT: KMSAN origin in step_into
ALT: bad-access in prepare_task_switch

[ 567.476354][ T1] =====================================================
Expand Down
1 change: 0 additions & 1 deletion pkg/report/testdata/linux/report/632
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
TITLE: KMSAN: uninit-value in __perf_event_task_sched_in
ALT: KMSAN origin in corrupted
ALT: bad-access in __perf_event_task_sched_in
CORRUPTED: Y

Expand Down
1 change: 0 additions & 1 deletion pkg/report/testdata/linux/report/633
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
TITLE: KMSAN: uninit-value in prepend_path
ALT: KMSAN origin in corrupted
ALT: bad-access in prepend_path
CORRUPTED: Y

Expand Down
1 change: 0 additions & 1 deletion pkg/report/testdata/linux/report/634
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
TITLE: KMSAN: kernel-infoleak in urandom_read_nowarn
ALT: KMSAN origin in crng_reseed
ALT: bad-access in urandom_read_nowarn

[ 600.161674][ T9046] =====================================================
Expand Down
1 change: 0 additions & 1 deletion pkg/report/testdata/linux/report/635
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
TITLE: KMSAN: uninit-value in sctp_epaddr_lookup_transport
ALT: KMSAN origin in sctp_rcv
ALT: bad-access in sctp_epaddr_lookup_transport

[ 701.063465][ C1] =====================================================
Expand Down
1 change: 0 additions & 1 deletion pkg/report/testdata/linux/report/636
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
TITLE: KMSAN: uninit-value in ppp_send_frame
ALT: KMSAN origin in ppp_write
ALT: bad-access in ppp_send_frame

[ 87.733076][ T3479] =====================================================
Expand Down
1 change: 0 additions & 1 deletion pkg/report/testdata/linux/report/670
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
TITLE: KMSAN: uninit-value in native_apic_mem_write
ALT: KMSAN origin in inet_reqsk_alloc
ALT: bad-access in native_apic_mem_write

[ 663.629383][ C1] =====================================================
Expand Down
1 change: 0 additions & 1 deletion pkg/report/testdata/linux/report/671
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
TITLE: KMSAN: uninit-value in flush_to_ldisc
ALT: KMSAN origin in __pskb_copy_fclone
ALT: bad-access in flush_to_ldisc

[ 1725.922949][ T52] =====================================================
Expand Down
1 change: 0 additions & 1 deletion pkg/report/testdata/linux/report/672
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
TITLE: KMSAN: uninit-value in ext4_evict_inode
ALT: KMSAN origin in ext4_alloc_inode
ALT: bad-access in ext4_evict_inode

[ 345.516988][ T3516] =====================================================
Expand Down
1 change: 0 additions & 1 deletion pkg/report/testdata/linux/report/673
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
TITLE: KMSAN: uninit-value in nilfs_bmap_lookup_at_level
ALT: KMSAN origin in nilfs_alloc_inode
ALT: bad-access in nilfs_bmap_lookup_at_level

[ 231.715107][ T3795] =====================================================
Expand Down
1 change: 0 additions & 1 deletion pkg/report/testdata/linux/report/674
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
TITLE: KMSAN: uninit-value in ntfs_iget5
ALT: KMSAN origin in ntfs_alloc_inode
ALT: bad-access in ntfs_iget5

[ 493.519926][ T7865] =====================================================
Expand Down
1 change: 0 additions & 1 deletion pkg/report/testdata/linux/report/681
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
TITLE: KMSAN: uninit-value in net_tx_action
ALT: KMSAN origin in rtnetlink_rcv
ALT: bad-access in net_tx_action

[ 142.141483][ C0] =====================================================
Expand Down
1 change: 0 additions & 1 deletion pkg/report/testdata/linux/report/682
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
TITLE: KMSAN: uninit-value in ieee80211_rx_list
ALT: KMSAN origin in nfnetlink_rcv
ALT: bad-access in ieee80211_rx_list

[ 338.587187][ C0] BUG: KMSAN: uninit-value in ieee80211_rx_list+0x1839/0x5860
Expand Down
1 change: 0 additions & 1 deletion pkg/report/testdata/linux/report/689
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
TITLE: KMSAN: kernel-infoleak in kernfs_fop_read_iter
ALT: KMSAN origin in dev_set_name
ALT: bad-access in kernfs_fop_read_iter

[ 160.663319][ T5029] =====================================================
Expand Down
1 change: 0 additions & 1 deletion pkg/report/testdata/linux/report/690
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
TITLE: KMSAN: kernel-infoleak in __skb_datagram_iter
ALT: KMSAN origin in pfkey_add
ALT: bad-access in __skb_datagram_iter

[ 2104.495854][ T4311] =====================================================
Expand Down
1 change: 0 additions & 1 deletion pkg/report/testdata/linux/report/699
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
TITLE: KMSAN: uninit-value in aes_encrypt
ALT: KMSAN origin in ext4_write_begin
ALT: bad-access in aes_encrypt

[ 311.903743][ T5388] =====================================================
Expand Down
1 change: 0 additions & 1 deletion pkg/report/testdata/linux/report/700
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
TITLE: KMSAN: uninit-value in virtqueue_add
ALT: KMSAN origin in ext4_da_write_begin
ALT: bad-access in virtqueue_add

[ 897.203644][ T1083] =====================================================
Expand Down
1 change: 0 additions & 1 deletion pkg/report/testdata/linux/report/701
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
TITLE: KMSAN: uninit-value in btrfs_bin_search
ALT: KMSAN origin in alloc_extent_buffer
ALT: bad-access in btrfs_bin_search

[ 343.493742][ T5647] =====================================================
Expand Down
1 change: 0 additions & 1 deletion pkg/report/testdata/linux/report/702
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
TITLE: KMSAN: uninit-value in post_read_mst_fixup
ALT: KMSAN origin in map_mft_record
ALT: bad-access in post_read_mst_fixup

[ 355.605345][ T5697] =====================================================
Expand Down
1 change: 0 additions & 1 deletion pkg/report/testdata/linux/report/703
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
TITLE: KMSAN: uninit-value in nilfs_add_checksums_on_logs
ALT: KMSAN origin in block_write_begin
ALT: bad-access in nilfs_add_checksums_on_logs

[ 417.716144][ T7542] =====================================================
Expand Down

0 comments on commit 696ea0d

Please sign in to comment.