This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [patch v6 00/12] branch tracing support for Atom
- From: Jan Kratochvil <jan dot kratochvil at redhat dot com>
- To: "Metzger, Markus T" <markus dot t dot metzger at intel dot com>
- Cc: "palves at redhat dot com" <palves at redhat dot com>, "tromey at redhat dot com" <tromey at redhat dot com>, "kettenis at gnu dot org" <kettenis at gnu dot org>, "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>, "markus dot t dot metzger at gmail dot com" <markus dot t dot metzger at gmail dot com>
- Date: Tue, 1 Jan 2013 17:34:41 +0100
- Subject: Re: [patch v6 00/12] branch tracing support for Atom
- References: <1355760101-26237-1-git-send-email-markus.t.metzger@intel.com> <20121218091953.GF8054@host2.jankratochvil.net> <A78C989F6D9628469189715575E55B2307B421D6@IRSMSX102.ger.corp.intel.com> <20121218135437.GA16636@host2.jankratochvil.net> <A78C989F6D9628469189715575E55B2307B42B88@IRSMSX102.ger.corp.intel.com> <20121222130812.GA6195@host2.jankratochvil.net>
On Sat, 22 Dec 2012 14:08:12 +0100, Jan Kratochvil wrote:
> On Wed, 19 Dec 2012 10:59:37 +0100, Metzger, Markus T wrote:
> > If I just followed the "target record" command, I would lose the selective
> > enabling as well as the automatic enabling
>
> Pushing of the new target could be hooked to to_post_startup_inferior and
> to_post_attach, I hope that's enough.
When thinking about it the btrace target should stay pushed
after "record btrace auto". And contrary to the current record.c practice it
would no longer unpush itself during to_mourn_inferior, to_kill etc.
(The same wouhld apply to a new feature "record auto" as it is the same issue.
Just I do not want to get you into a too much work of improving record.c,
I can take some parts of record.c if you ask for.)
Also the keeping up to date of observer_attach_new_thread and
observer_detach_new_thread could be simplified by creating a new to_target
method would would be called together with observer_notify_new_thread.
Then one can forget about to_post_startup_inferior/to_post_attach, that was
a wrong advice.
Thanks,
Jan