[Bug testsuite/30528] New: [gdb/testsuite] FAIL: gdb.dwarf2/per-bfd-sharing.exp: couldn't remove files in temporary cache dir

[Bug testsuite/30528] New: [gdb/testsuite] FAIL: gdb.dwarf2/per-bfd-sharing.exp: couldn't remove files in temporary cache dir

[vries at gcc dot gnu.org]

https://sourceware.org/bugzilla/show_bug.cgi?id=30528

            Bug ID: 30528
           Summary: [gdb/testsuite] FAIL: gdb.dwarf2/per-bfd-sharing.exp:
                    couldn't remove files in temporary cache dir
           Product: gdb
           Version: 13.1
            Status: NEW
          Severity: normal
          Priority: P2
         Component: testsuite
          Assignee: unassigned at sourceware dot org
          Reporter: vries at gcc dot gnu.org
  Target Milestone: ---

With a gdb 13.1 based package on SLE-11 i586, I run into:
...
(gdb) PASS: gdb.dwarf2/per-bfd-sharing.exp: first=readnow: second=readnow:
third=readnow: load third: print foo
Executing on host: sh -c "rm /tmp/tmp.xQ6KmzbodJ/*.gdb-index"   (timeout = 300)
builtin_spawn sh -c rm /tmp/tmp.xQ6KmzbodJ/*.gdb-index^M
FAIL: gdb.dwarf2/per-bfd-sharing.exp: couldn't remove files in temporary cache
dir
testcase
/usr/src/packages/BUILD/gdb-13.1/gdb/testsuite/gdb.dwarf2/per-bfd-sharing.exp
...

I don't understand what has caused this (part of what makes this difficult is
that there's no error message).

I'm not able to reproduce this.

However, I think that this should not be a FAIL.

A FAIL is for when GDB does something incorrect, which is not necesarily the
case here, so perhaps this should be warning.

However, if the failure to remove the file is due to the file not being there
in the first place, then that is a GDB error.  We just don't explicitly check
for it currently.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug testsuite/30528] [gdb/testsuite] FAIL: gdb.dwarf2/per-bfd-sharing.exp: couldn't remove files in temporary cache dir

[vries at gcc dot gnu.org]

https://sourceware.org/bugzilla/show_bug.cgi?id=30528

--- Comment #1 from Tom de Vries <vries at gcc dot gnu.org> ---
Reproduced again on SLE-11 x86_64:
...
/data/vries/osc/devel/gcc/gdb/tmp-qa-local/pkgs/gdb-testresults.SLE-11.x86_64/gdb-x86_64-suse-linux-m64.sum:
FAILs:
FAIL: gdb.dwarf2/per-bfd-sharing.exp: couldn't remove files in temporary cache
dir
...

I wondered if missing build-id support could be the cause of this, but
elsewhere in the same log we have:
...
PASS: gdb.base/sepdebug.exp: build-id support by binutils
...

Another possibility could be that we're looking at some of the race conditions
in the index cache that were fixed recently.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug testsuite/30528] [gdb/testsuite] FAIL: gdb.dwarf2/per-bfd-sharing.exp: couldn't remove files in temporary cache dir

[vries at gcc dot gnu.org]

https://sourceware.org/bugzilla/show_bug.cgi?id=30528

--- Comment #2 from Tom de Vries <vries at gcc dot gnu.org> ---
FTR:
...
$ runtest --version
Expect version is       5.44.1.11
Tcl version is          8.5
Framework version is    1.4.4
...

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug testsuite/30528] [gdb/testsuite] FAIL: gdb.dwarf2/per-bfd-sharing.exp: couldn't remove files in temporary cache dir

[cvs-commit at gcc dot gnu.org]

https://sourceware.org/bugzilla/show_bug.cgi?id=30528

--- Comment #3 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Tom de Vries <vries@sourceware.org>:

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=743d3f0945c625ce5647130b506eeb6940dfc12e

commit 743d3f0945c625ce5647130b506eeb6940dfc12e
Author: Tom de Vries <tdevries@suse.de>
Date:   Thu Oct 26 10:34:24 2023 +0200

    [gdb/testsuite] Add wait-for-index-cache in gdb.dwarf2/per-bfd-sharing.exp

    If we make writing an index-cache entry very slow by doing this in
    index_cache::store:
    ...
       try
         {
    +      sleep (15);
           index_cache_debug ("writing index cache for objfile %s",
                             bfd_get_filename (per_bfd->obfd));
    ...
    we run into:
    ...
    FAIL: gdb.dwarf2/per-bfd-sharing.exp: \
      couldn't remove files in temporary cache dir
    ...

    The FAIL happens because there is no index-cache entry in the cache dir.

    The problem is that gdb is killed (by gdb_exit) before the index-cache
entry
    is written.

    Fix this by using "maint wait-for-index-cache".

    Tested on x86_64-linux.

    PR testsuite/30528
    Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=30528

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug testsuite/30528] [gdb/testsuite] FAIL: gdb.dwarf2/per-bfd-sharing.exp: couldn't remove files in temporary cache dir

[vries at gcc dot gnu.org]

https://sourceware.org/bugzilla/show_bug.cgi?id=30528

Tom de Vries <vries at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
   Target Milestone|---                         |15.1
             Status|NEW                         |RESOLVED

--- Comment #4 from Tom de Vries <vries at gcc dot gnu.org> ---
Fixed.

-- 
You are receiving this mail because:
You are on the CC list for the bug.