This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] Fetch all registers before writing the core register notes.
- From: John Baldwin <jhb at freebsd dot org>
- To: Pedro Alves <palves at redhat dot com>
- Cc: gdb-patches at sourceware dot org
- Date: Mon, 16 Mar 2015 10:55:53 -0400
- Subject: Re: [PATCH] Fetch all registers before writing the core register notes.
- Authentication-results: sourceware.org; auth=none
- References: <2653052 dot Cuv9hkMi4i at ralph dot baldwin dot cx> <5506C7F2 dot 5040605 at redhat dot com>
On Monday, March 16, 2015 12:09:22 PM Pedro Alves wrote:
> On 03/14/2015 10:02 PM, John Baldwin wrote:
> > Without this, not all registers were present in the core generated by gcore.
> > For example, running 'gcore' on a program without examining the vector
> > registers (SSE or AVX) would store all the vector registers as zeros because
> > they were not pulled into the regcache. Running 'info vector' before 'gcore'
> > would store the correct values in the core since it populated the regcache.
> > For Linux processes, a similar operation is achieved somewhat by having the
> > thread iterator callback invoke target_fetch_registers on each thread before
> > its corresponding register notes are dumped.
> >
> > (I don't plan on including that level of detail in the commit log, just as
> > a way to explain the bug this change fixes.)
>
> I think it's good to have the detail in the log. If you felt the need to
> explain it for review, then a future archaeologist looking at git log/blame
> would likely appreciate the same info.
Fair enough, I will include it.
> > gdb/ChangeLog:
> >
> > * fbsd-tdep.c (fbsd_make_corefile_notes): Fetch all target registers
> > before writing core register notes.
>
> Looks fine to me.
>
> Thanks,
> Pedro Alves
>
--
John Baldwin