[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/2] Make a "gdb" dprintf that tells us load addresses.
From: |
Robbie Harwood |
Subject: |
Re: [PATCH 2/2] Make a "gdb" dprintf that tells us load addresses. |
Date: |
Fri, 15 Oct 2021 16:55:17 -0400 |
Glenn Washburn <development@efficientek.com> writes:
> Robbie Harwood <rharwood@redhat.com> wrote:
>> Glenn Washburn <development@efficientek.com> writes:
>>
>>> I don't agree with suppressing file:line for debug output. I have
>>> some changes in the pipeline which will allow selectively disabling
>>> conditions when also using "debug=all". Suppressing file:line makes
>>> it harder to figure out where the debug output is coming from to
>>> find the condition to disable it.
>>
>> Grep for add-symbol-file or /usr/lib/debug etc. will get there.
>
> I'm not following. Are you suggesting grepping for "add-symbol-file"
> in the source? If so, I don't find that an acceptable answer to the
> sysadmin trying to figure out hwo to disable that log message.
I don't understand your target audience - they have access to the
source, but can't search it? Do I have that right?
If so, it seems unlikely that we're going to agree.
Be well,
--Robbie
signature.asc
Description: PGP signature
- [PATCH 0/2] Print out load addresses for .text and .data, Robbie Harwood, 2021/10/11
- [PATCH 2/2] Make a "gdb" dprintf that tells us load addresses., Robbie Harwood, 2021/10/11
- Re: [PATCH 2/2] Make a "gdb" dprintf that tells us load addresses., Glenn Washburn, 2021/10/12
- Re: [PATCH 2/2] Make a "gdb" dprintf that tells us load addresses., Robbie Harwood, 2021/10/14
- Re: [PATCH 2/2] Make a "gdb" dprintf that tells us load addresses., Glenn Washburn, 2021/10/15
- Re: [PATCH 2/2] Make a "gdb" dprintf that tells us load addresses.,
Robbie Harwood <=
- Re: [PATCH 2/2] Make a "gdb" dprintf that tells us load addresses., Glenn Washburn, 2021/10/15
- Re: [PATCH 2/2] Make a "gdb" dprintf that tells us load addresses., Robbie Harwood, 2021/10/19
- Re: [PATCH 2/2] Make a "gdb" dprintf that tells us load addresses., Glenn Washburn, 2021/10/19
- Re: [PATCH 2/2] Make a "gdb" dprintf that tells us load addresses., Thomas Schmitt, 2021/10/20
[PATCH 1/2] Add grub_qdprintf() - grub_dprintf() without the file+line number., Robbie Harwood, 2021/10/11