public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
From: fche@redhat.com (Frank Ch. Eigler)
To: Simon Marchi <simark@simark.ca>
Cc: "Alexandra Hájková" <ahajkova@redhat.com>, gdb-patches@sourceware.org
Subject: Re: [PATCH] gdb: defer warnings when loading separate debug files
Date: Mon, 19 Dec 2022 08:04:01 -0500	[thread overview]
Message-ID: <87cz8fr1cv.fsf@redhat.com> (raw)
In-Reply-To: <a581d641-8c49-3ce3-37e9-b55c6d03de04@simark.ca>

Simon Marchi via Gdb-patches <gdb-patches@sourceware.org> writes:

> [...]
> I don't really like this behavior change.  The situations you described:
>
>  - a separate debug file without any actual debug information
>  - CRC checksum that doesn't match
>
> ... are abnormal situations which still deserve being warned about, I
> think.  If the files are there, it's because they are meant to be
> used,

That's a bit of a leap.  They can happen totally accidentally, when a
sysadmin or cron job does an automated update of the system, and for
whatever reason, the debug packages got missed.  On some distros, this
can easily happen.  Then the remnants are not "meant to be used" at all,
they are just fossils.

> so if something prevents GDB from using them, I want to know.
> Silencing the warning just makes investigating "why doesn't GDB read
> my separate debug file" harder.

If GDB did find the correct debug file one way or the other, then why
would you care, never mind bother investigate?  It's a non-problem.


> I can understand why this can be a bit confusing to the user, but
> the warning is still factually correct.  For instance, the one you
> quoted:
>
>     warning: the debug information found in
>     "/usr/lib/debug//lib64/ld-2.32.so.debug" does not match
>     "/lib64/ld-linux-x86-64.so.2" (CRC mismatch).

Ideally, users should really only be warned about things that they can
do something about.  Being warned about the same thing that only a
sysadmin can possibly fix, and repeatedly, seems more like noise.

- FChE


  parent reply	other threads:[~2022-12-19 13:04 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-12-12 12:15 AlexandraH=?UTF-8?B?w6E=?=jkov=?UTF-8?B?w6E=?=ahajkova
2022-12-15 15:57 ` Bruno Larsen
2022-12-15 16:35 ` Simon Marchi
     [not found]   ` <CAJVr-EOShPVS_ZBvfHqKkbPK1O3d+92DDbpEoeBVr1_s7_XWYA@mail.gmail.com>
2022-12-19 12:28     ` Fwd: " Alexandra Petlanova Hajkova
2022-12-19 13:04   ` Frank Ch. Eigler [this message]
2022-12-19 14:43   ` Andrew Burgess
2022-12-20  2:29     ` Simon Marchi

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=87cz8fr1cv.fsf@redhat.com \
    --to=fche@redhat.com \
    --cc=ahajkova@redhat.com \
    --cc=gdb-patches@sourceware.org \
    --cc=simark@simark.ca \
    /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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).