From: Pedro Alves <pedro@codesourcery.com>
To: gdb-patches@sourceware.org, danny.backx@scarlet.be
Subject: Re: Post mortem debugging for Windows CE
Date: Wed, 06 May 2009 20:47:00 -0000 [thread overview]
Message-ID: <200905062148.03196.pedro@codesourcery.com> (raw)
In-Reply-To: <1241640256.4083.253.camel@dannypc>
On Wednesday 06 May 2009 21:04:16, Danny Backx wrote:
> I've done a couple of tests. (See below, lengthy.) The bottom line :
> - I should have used solib-search-path the first time you told me, not
> after the second time :-)
> - without setting solib-search-path, the output for both live and post-
> mortem (minidump) sessions is a list of DLLs without address range.
> - with setting solib-search-path, the output of both live and postmortem
> sessions shows addresses for those DLLs found
Right, that is because GDB only shows that info if it finds
symbols for the shared library:
From To Syms Read Shared Object Library
No bad.dll
^^
vs:
From To Syms Read Shared Object Library
0x01451000 0x01456044 Yes /home/danny/tmp/arm/test/pedro/bad.dll
^^^
On Wednesday 06 May 2009 21:04:16, Danny Backx wrote:
> > BTW, does "info threads" show the correct list of threads?
>
> Not yet. The place where I found the register contents until now was not
> 'thread-safe' - there's only one set of registers there.
>
> The only additional info I found points to memory regions that the minidump
> doesn't contain a copy of.
>
> Further poking on MSDN reveals a table that explains the contents of "context
> dumps" (limited in size) vs. "system dumps" and "complete dumps".
> See http://msdn.microsoft.com/en-us/library/ms939598.aspx .
Ah, I see. If the data ain't there, there's nothing we can do
about it.
> Also http://msdn.microsoft.com/en-us/library/ms939607.aspx describes
> registry settings to influence dump handling. Can't find a way to change
> the type of dump though.
I think that if you clean this up and post it, it will be nice
addition already. You'll need the copyright assignment issues sorted
first before we can accept the code though, for binutils (bfd)
and gdb.
A few suggestions:
- Please forgive me if you know this already.
Your minidump bfd code should work on all hosts, 64-bit, 32-bit,
big endian or little endian. This means that code like:
typedef struct FooStructureMSDefinesToBePresentInTheFile
{
DWORD foo;
char b;
int c;
} FooStructureMSDefinesToBePresentInTheFile;
FooStructureMSDefinesToBePresentInTheFile bar;
read (fildes, &bar, sizeof (bar));
... is unacceptable. You need to use the bfd_get_32 and friends
to parse the data on the file.
- Be sure to follow the coding standards:
http://www.gnu.org/prep/standards/html_node/Formatting.html
Standard nit-picks:
space between function name and '('.
double space after periods in comments.
tabs are prefered to spaces.
- Don't forget a ChangeLog entry.
--
Pedro Alves
next prev parent reply other threads:[~2009-05-06 20:47 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-04-28 14:45 Danny Backx
2009-04-28 16:02 ` Pedro Alves
2009-04-28 17:48 ` Danny Backx
2009-04-28 19:18 ` Danny Backx
2009-04-29 7:10 ` Eli Zaretskii
2009-04-29 7:19 ` Danny Backx
2009-04-29 7:44 ` Eli Zaretskii
2009-04-30 7:25 ` Danny Backx
2009-04-30 9:58 ` Pedro Alves
2009-05-01 14:35 ` Danny Backx
2009-05-01 15:37 ` Pedro Alves
2009-05-05 13:31 ` Danny Backx
2009-05-05 16:53 ` Pedro Alves
2009-05-06 14:13 ` Danny Backx
2009-05-06 14:34 ` Pedro Alves
2009-05-06 20:05 ` Danny Backx
2009-05-06 20:47 ` Pedro Alves [this message]
2009-05-07 1:21 ` Danny Backx
2009-05-10 19:21 ` Pedro Alves
2009-05-11 14:03 ` Danny Backx
2009-05-11 15:00 ` Danny Backx
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=200905062148.03196.pedro@codesourcery.com \
--to=pedro@codesourcery.com \
--cc=danny.backx@scarlet.be \
--cc=gdb-patches@sourceware.org \
/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