Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Daniel Jacobowitz <drow@false.org>
To: Jan Kratochvil <jan.kratochvil@redhat.com>, gdb-patches@sourceware.org
Subject: Recent separate debug file warning caused Debian regressions
Date: Mon, 02 Nov 2009 21:53:00 -0000	[thread overview]
Message-ID: <20091102215255.GA19425@caradoc.them.org> (raw)

One of your patches in the last few days caused at least these
regressions for me:

-PASS: gdb.base/annota1.exp: run until main breakpoint
+FAIL: gdb.base/annota1.exp: run until main breakpoint

 Running /space/fsf/commit/src/gdb/testsuite/gdb.mi/mi-async.exp ...
+FAIL: gdb.mi/mi-async.exp: start: send (failed to resume)
 PASS: gdb.mi/mi-async.exp: start: stop
 PASS: gdb.mi/mi-async.exp: CLI next: stop
+FAIL: gdb.mi/mi-async.exp: restart: send (failed to resume)
 PASS: gdb.mi/mi-async.exp: restart: stop

-PASS: gdb.mi/mi-nonstop-exit.exp: mi runto main
-PASS: gdb.mi/mi-nonstop-exit.exp: finished exec continue
-PASS: gdb.mi/mi-nonstop-exit.exp: breakpoint at main
-PASS: gdb.mi/mi-nonstop-exit.exp: mi runto main
-PASS: gdb.mi/mi-nonstop-exit.exp: finished exec continue (2)
+ERROR: Unable to start target
+ERROR: mi-nonstop-exit.exp tests suppressed

It's definitely one of the last four patches you committed, and I
suspect it's the warning patch.  I believe the problem is this
message:

warning: the debug information found in "/lib/libm-2.9.so" does not
match "/lib/libm.so.6" (CRC mismatch).

It's upsetting the testsuite.  I don't know why we're calculating a
CRC for /lib/libm-2.9.so, which is of course the same library - not
a separate debug file - so no wonder the CRC doesn't match, it's the
CRC of the debug info.

1421      /* First try in the same directory as the original file.  */
1422      strcpy (debugfile, dir);
1423      strcat (debugfile, basename);

It comes from this check.  We don't even compare the name (which is
likely to be the same, but in the case of GLIBC's symlinks, isn't).
Should we use stat to reject CRC'ing the file itself?  I know there's
a catch with inode number checks on some filesystems (Windows), but ld
has some code for this.

-- 
Daniel Jacobowitz
CodeSourcery


             reply	other threads:[~2009-11-02 21:53 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-11-02 21:53 Daniel Jacobowitz [this message]
2009-11-03 10:44 ` [patch] Fix false separate debuginfo warning for no .debug suffix [Re: Recent separate debug file warning caused Debian regressions] Jan Kratochvil
2009-11-03 17:00   ` Daniel Jacobowitz
2009-11-04 23:26     ` Jan Kratochvil
2009-11-05 20:04       ` Daniel Jacobowitz
2009-11-11  5:05         ` Jan Kratochvil

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20091102215255.GA19425@caradoc.them.org \
    --to=drow@false.org \
    --cc=gdb-patches@sourceware.org \
    --cc=jan.kratochvil@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox