public inbox for gdb-prs@sourceware.org
help / color / mirror / Atom feed
* [Bug gdb/31237] New: [gdb, clang] FAIL: gdb.gdb/index-file.exp: gdb-index files are identical
@ 2024-01-12 8:13 vries at gcc dot gnu.org
2024-01-12 10:18 ` [Bug gdb/31237] " vries at gcc dot gnu.org
` (8 more replies)
0 siblings, 9 replies; 10+ messages in thread
From: vries at gcc dot gnu.org @ 2024-01-12 8:13 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=31237
Bug ID: 31237
Summary: [gdb, clang] FAIL: gdb.gdb/index-file.exp: gdb-index
files are identical
Product: gdb
Version: HEAD
Status: NEW
Severity: normal
Priority: P2
Component: gdb
Assignee: unassigned at sourceware dot org
Reporter: vries at gcc dot gnu.org
Target Milestone: ---
I build gdb with clang, and ran into:
...
(gdb) PASS: gdb.gdb/index-file.exp: create second dwarf-index files
Executing on host: cmp -s
"/home/vries/gdb/build/gdb/testsuite/outputs/gdb.gdb/index-file/index_1/xgdb.gdb-index"
"/home/vries/gdb/build/gdb/testsuite/outputs/gdb.gdb/index-file/index_2/xgdb.gdb-index"
(timeout = 300)
builtin_spawn -ignore SIGHUP cmp -s
/home/vries/gdb/build/gdb/testsuite/outputs/gdb.gdb/index-file/index_1/xgdb.gdb-index
/home/vries/gdb/build/gdb/testsuite/outputs/gdb.gdb/index-file/index_2/xgdb.gdb-index^M
FAIL: gdb.gdb/index-file.exp: gdb-index files are identical
...
And indeed:
...
$ ls -al index_2/xgdb.gdb-index
-rw-------. 1 vries vries 13582433 12 jan 09:08 index_2/xgdb.gdb-index
$ ls -al index_1/xgdb.gdb-index
-rw-------. 1 vries vries 13582873 12 jan 09:08 index_1/xgdb.gdb-index
...
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Bug gdb/31237] [gdb, clang] FAIL: gdb.gdb/index-file.exp: gdb-index files are identical
2024-01-12 8:13 [Bug gdb/31237] New: [gdb, clang] FAIL: gdb.gdb/index-file.exp: gdb-index files are identical vries at gcc dot gnu.org
@ 2024-01-12 10:18 ` vries at gcc dot gnu.org
2024-01-12 10:19 ` vries at gcc dot gnu.org
` (7 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: vries at gcc dot gnu.org @ 2024-01-12 10:18 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=31237
--- Comment #1 from Tom de Vries <vries at gcc dot gnu.org> ---
(In reply to Tom de Vries from comment #0)
> I build gdb with clang, and ran into:
This was on aarch64-linux.
Now also reproduced on x86_64-linux, with clang 15.0.7, both at -O0 and 02.
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Bug gdb/31237] [gdb, clang] FAIL: gdb.gdb/index-file.exp: gdb-index files are identical
2024-01-12 8:13 [Bug gdb/31237] New: [gdb, clang] FAIL: gdb.gdb/index-file.exp: gdb-index files are identical vries at gcc dot gnu.org
2024-01-12 10:18 ` [Bug gdb/31237] " vries at gcc dot gnu.org
@ 2024-01-12 10:19 ` vries at gcc dot gnu.org
2024-01-12 16:03 ` tromey at sourceware dot org
` (6 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: vries at gcc dot gnu.org @ 2024-01-12 10:19 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=31237
Tom de Vries <vries at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |aburgess at redhat dot com
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Bug gdb/31237] [gdb, clang] FAIL: gdb.gdb/index-file.exp: gdb-index files are identical
2024-01-12 8:13 [Bug gdb/31237] New: [gdb, clang] FAIL: gdb.gdb/index-file.exp: gdb-index files are identical vries at gcc dot gnu.org
2024-01-12 10:18 ` [Bug gdb/31237] " vries at gcc dot gnu.org
2024-01-12 10:19 ` vries at gcc dot gnu.org
@ 2024-01-12 16:03 ` tromey at sourceware dot org
2024-01-13 1:56 ` tromey at sourceware dot org
` (5 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: tromey at sourceware dot org @ 2024-01-12 16:03 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=31237
Tom Tromey <tromey at sourceware dot org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |tromey at sourceware dot org
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Bug gdb/31237] [gdb, clang] FAIL: gdb.gdb/index-file.exp: gdb-index files are identical
2024-01-12 8:13 [Bug gdb/31237] New: [gdb, clang] FAIL: gdb.gdb/index-file.exp: gdb-index files are identical vries at gcc dot gnu.org
` (2 preceding siblings ...)
2024-01-12 16:03 ` tromey at sourceware dot org
@ 2024-01-13 1:56 ` tromey at sourceware dot org
2024-01-13 14:25 ` vries at gcc dot gnu.org
` (4 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: tromey at sourceware dot org @ 2024-01-13 1:56 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=31237
--- Comment #2 from Tom Tromey <tromey at sourceware dot org> ---
I wonder if this could be a race in the DWARF reader.
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Bug gdb/31237] [gdb, clang] FAIL: gdb.gdb/index-file.exp: gdb-index files are identical
2024-01-12 8:13 [Bug gdb/31237] New: [gdb, clang] FAIL: gdb.gdb/index-file.exp: gdb-index files are identical vries at gcc dot gnu.org
` (3 preceding siblings ...)
2024-01-13 1:56 ` tromey at sourceware dot org
@ 2024-01-13 14:25 ` vries at gcc dot gnu.org
2024-01-13 14:54 ` vries at gcc dot gnu.org
` (3 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: vries at gcc dot gnu.org @ 2024-01-13 14:25 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=31237
--- Comment #3 from Tom de Vries <vries at gcc dot gnu.org> ---
(In reply to Tom Tromey from comment #2)
> I wonder if this could be a race in the DWARF reader.
At first glance, that doesn't seem to be the case.
I get consistent results for worker-threads 8, and consistent results for
work-threads 4, just a difference between those 2.
It might be a similar issue as fixed by commit aff250145af ("gdb: generate
gdb-index identically regardless of work thread count") for the symbol table,
but for the address table.
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Bug gdb/31237] [gdb, clang] FAIL: gdb.gdb/index-file.exp: gdb-index files are identical
2024-01-12 8:13 [Bug gdb/31237] New: [gdb, clang] FAIL: gdb.gdb/index-file.exp: gdb-index files are identical vries at gcc dot gnu.org
` (4 preceding siblings ...)
2024-01-13 14:25 ` vries at gcc dot gnu.org
@ 2024-01-13 14:54 ` vries at gcc dot gnu.org
2024-01-16 7:42 ` vries at gcc dot gnu.org
` (2 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: vries at gcc dot gnu.org @ 2024-01-13 14:54 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=31237
--- Comment #4 from Tom de Vries <vries at gcc dot gnu.org> ---
Hmm, there's an actual difference in the address table:
...
$ grep " 122$" 1 | sort -d
000000000043ac80 000000000043ad48 122
00000000006bbeb0 00000000006bd024 122
00000000028d4aa8 00000000028d4aa9 122
00000000028d4b50 00000000028d4b51 122
0000000002b74880 0000000002b74881 122
0000000002b74888 0000000002b74889 122
$ grep " 122$" 2 | sort -d
00000000006bbeb0 00000000006bd024 122
00000000028d4aa8 00000000028d4aa9 122
00000000028d4b50 00000000028d4b51 122
0000000002b74880 0000000002b74881 122
0000000002b74888 0000000002b74889 122
...
The first element of 1 is extra, the rest of the entries is equal.
Let's grep for the starting address of the extra element:
...
$ grep 43ac80 1
000000000043ac80 000000000043ad48 122
000000000043ac80 000000000043ace8 244
000000000043ac80 000000000043b1c4 490
000000000043ac80 000000000043ad48 600
$ grep 43ac80 2
000000000043ac80 000000000043ace8 244
000000000043ac80 000000000043b1c4 490
...
So, for a given address 0x43ac80 there is a number of CUs that match, and the
number is different between the two address tables.
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Bug gdb/31237] [gdb, clang] FAIL: gdb.gdb/index-file.exp: gdb-index files are identical
2024-01-12 8:13 [Bug gdb/31237] New: [gdb, clang] FAIL: gdb.gdb/index-file.exp: gdb-index files are identical vries at gcc dot gnu.org
` (5 preceding siblings ...)
2024-01-13 14:54 ` vries at gcc dot gnu.org
@ 2024-01-16 7:42 ` vries at gcc dot gnu.org
2024-01-16 10:55 ` vries at gcc dot gnu.org
2024-01-16 12:36 ` vries at gcc dot gnu.org
8 siblings, 0 replies; 10+ messages in thread
From: vries at gcc dot gnu.org @ 2024-01-16 7:42 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=31237
--- Comment #5 from Tom de Vries <vries at gcc dot gnu.org> ---
(In reply to Tom de Vries from comment #4)
> So, for a given address 0x43ac80 there is a number of CUs that match, and
> the number is different between the two address tables.
And, the overlap is caused by clang generating:
...
Disassembly of section .text._ZN14frame_info_ptrC2ERKS_:
0000000000000000 <_ZN14frame_info_ptrC2ERKS_>:
...
after which the code is merged by the linker.
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Bug gdb/31237] [gdb, clang] FAIL: gdb.gdb/index-file.exp: gdb-index files are identical
2024-01-12 8:13 [Bug gdb/31237] New: [gdb, clang] FAIL: gdb.gdb/index-file.exp: gdb-index files are identical vries at gcc dot gnu.org
` (6 preceding siblings ...)
2024-01-16 7:42 ` vries at gcc dot gnu.org
@ 2024-01-16 10:55 ` vries at gcc dot gnu.org
2024-01-16 12:36 ` vries at gcc dot gnu.org
8 siblings, 0 replies; 10+ messages in thread
From: vries at gcc dot gnu.org @ 2024-01-16 10:55 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=31237
--- Comment #6 from Tom de Vries <vries at gcc dot gnu.org> ---
The cooked index consists of shards, and each shard has an address map, of type
addrmap_mutable.
The addr_mutable object supports overlapping insertion, but with
insertion-order specific results.
So, by changing the number of worker threads, we change the number of shards,
and consequently the CUs indexed by each shard, and consequently the sequence
of insertions into each per-shard address map.
This is fine for the non-overlapping cases, we'd expect the same collection of
<key,value> pairs, just distributed differently over the shards.
But for the overlapping case, that doesn't work.
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Bug gdb/31237] [gdb, clang] FAIL: gdb.gdb/index-file.exp: gdb-index files are identical
2024-01-12 8:13 [Bug gdb/31237] New: [gdb, clang] FAIL: gdb.gdb/index-file.exp: gdb-index files are identical vries at gcc dot gnu.org
` (7 preceding siblings ...)
2024-01-16 10:55 ` vries at gcc dot gnu.org
@ 2024-01-16 12:36 ` vries at gcc dot gnu.org
8 siblings, 0 replies; 10+ messages in thread
From: vries at gcc dot gnu.org @ 2024-01-16 12:36 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=31237
--- Comment #7 from Tom de Vries <vries at gcc dot gnu.org> ---
Created attachment 15306
--> https://sourceware.org/bugzilla/attachment.cgi?id=15306&action=edit
Demonstrator patch adding "maint find-addr"
To easily demonstrate the problem outside of the scope of index writing:
...
$ gdb -q -batch -iex "maint set worker-thread 4" xgdb -ex "maint find-addr
0x43ac80"
((dwarf2_per_cu_data *) 0xffff30005da0), offset: 551f
((dwarf2_per_cu_data *) 0xffff3000f4c0), offset: feffeb
((dwarf2_per_cu_data *) 0xffff30018750), offset: 1f6b978
((dwarf2_per_cu_data *) 0)
((dwarf2_per_cu_data *) 0)
$ gdb -q -batch -iex "maint set worker-thread 8" xgdb -ex "maint find-addr
0x43ac80"
((dwarf2_per_cu_data *) 0xffff10005da0), offset: 551f
((dwarf2_per_cu_data *) 0xffff1000a810), offset: 7e8be8
((dwarf2_per_cu_data *) 0xffff1000f4c0), offset: feffeb
((dwarf2_per_cu_data *) 0)
((dwarf2_per_cu_data *) 0xffff10018750), offset: 1f6b978
((dwarf2_per_cu_data *) 0xffff1001d580), offset: 251ad16
((dwarf2_per_cu_data *) 0)
((dwarf2_per_cu_data *) 0)
((dwarf2_per_cu_data *) 0)
...
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2024-01-16 12:36 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-01-12 8:13 [Bug gdb/31237] New: [gdb, clang] FAIL: gdb.gdb/index-file.exp: gdb-index files are identical vries at gcc dot gnu.org
2024-01-12 10:18 ` [Bug gdb/31237] " vries at gcc dot gnu.org
2024-01-12 10:19 ` vries at gcc dot gnu.org
2024-01-12 16:03 ` tromey at sourceware dot org
2024-01-13 1:56 ` tromey at sourceware dot org
2024-01-13 14:25 ` vries at gcc dot gnu.org
2024-01-13 14:54 ` vries at gcc dot gnu.org
2024-01-16 7:42 ` vries at gcc dot gnu.org
2024-01-16 10:55 ` vries at gcc dot gnu.org
2024-01-16 12:36 ` vries at gcc dot gnu.org
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).