From: Zied Guermazi <zied.guermazi@trande.de>
To: "Metzger, Markus T" <markus.t.metzger@intel.com>
Cc: Simon Marchi <simon.marchi@polymtl.ca>,
"gdb-patches@sourceware.org" <gdb-patches@sourceware.org>
Subject: Re: [PATCH 3/8] start/stop btrace with coresight etm and parse etm buffer. nat independant
Date: Tue, 30 Mar 2021 17:11:02 +0200 [thread overview]
Message-ID: <3daf0546-63dc-6504-c786-d509647a0043@trande.de> (raw)
In-Reply-To: <DM5PR11MB1690162B324352DE8C705458DE7E9@DM5PR11MB1690.namprd11.prod.outlook.com>
Hello Markus
the code was amended according to the outcome of the discussion. it
works fine. There is no blocking points anymore.
I am running the testsuite on i686, armv7 and armv8, to check any
possible regression. it takes long time on ARMv7. I guess it will finish
by the evening. I will send the patch suite after checking the result
(if everything goes fine)
Kind Regards
Zied Guermazi
On 29.03.21 16:01, Metzger, Markus T wrote:
>
> Hello Zied,
>
> >> For the point concerning the use of a vector of registers in btrace_insn:
>
> >> a vector of registers is added to the btrace_insn structure. This adds
> an overhead of 3 pointers when the vector is empty. It will be the
> overhead when not used (intel pt). I think this should be acceptable.
>
> >>
>
> >> That's doubling the current size; tripling it if we declared ICLASS
> 8b (as it
>
> >> should be). I don't think that's acceptable.
>
> >
>
> > on ARMv7 this is need for all instructions, to allow step and next to
> set the breakpoints properly in the code.
>
> >
>
> >> Is this to distinguish ARM and Thumb modes? How would the information
>
> >> be used?
>
> > [Zied] let me give you a better insight on this topic to find a better solution:
>
> > this information is needed for two purposes:
>
> I believe this was discussed in
> https://sourceware.org/pipermail/gdb/2021-March/049293.html.
>
> Is there anything left to discuss?
>
> Regards,
>
> Markus.
>
> Intel Deutschland GmbH
> Registered Address: Am Campeon 10, 85579 Neubiberg, Germany
> Tel: +49 89 99 8853-0, www.intel.de <http://www.intel.de>
> Managing Directors: Christin Eisenschmid, Sharon Heck, Tiffany Doon Silva
> Chairperson of the Supervisory Board: Nicole Lau
> Registered Office: Munich
> Commercial Register: Amtsgericht Muenchen HRB 186928
>
--
*Zied Guermazi*
founder
Trande UG
Leuschnerstraße 2
69469 Weinheim/Germany
Mobile: +491722645127
mailto:zied.guermazi@trande.de
*Trande UG*
Leuschnerstraße 2, D-69469 Weinheim; Telefon: +491722645127
Sitz der Gesellschaft: Weinheim- Registergericht: AG Mannheim HRB 736209
- Geschäftsführung: Zied Guermazi
*Confidentiality Note*
This message is intended only for the use of the named recipient(s) and
may contain confidential and/or privileged information. If you are not
the intended recipient, please contact the sender and delete the
message. Any unauthorized use of the information contained in this
message is prohibited.
next prev parent reply other threads:[~2021-03-30 15:11 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-02-26 2:45 [PATCH 0/8] extend branch tracing to use ARM CoreSight traces Zied Guermazi
2021-02-26 2:45 ` [PATCH 1/8] configure gdb build system for supporting btrace on arm processors Zied Guermazi
2021-03-04 16:35 ` Metzger, Markus T
2021-03-10 22:09 ` Zied Guermazi
2021-02-26 2:45 ` [PATCH 2/8] add btrace coresight related commands Zied Guermazi
2021-03-04 16:35 ` Metzger, Markus T
[not found] ` <8a71a9a3-ad14-b716-6d86-ece063061b02@trande.de>
2021-03-10 22:37 ` Zied Guermazi
2021-03-16 10:16 ` Metzger, Markus T
2021-03-19 15:21 ` Zied Guermazi
2021-02-26 2:45 ` [PATCH 3/8] start/stop btrace with coresight etm and parse etm buffer. nat independant Zied Guermazi
2021-03-04 16:35 ` Metzger, Markus T
2021-03-11 0:12 ` Zied Guermazi
2021-03-16 10:16 ` Metzger, Markus T
2021-03-19 15:29 ` Zied Guermazi
2021-03-29 14:01 ` Metzger, Markus T
2021-03-30 15:11 ` Zied Guermazi [this message]
2021-02-26 2:45 ` [PATCH 4/8] start/stop btrace with coresight etm and collect etm buffer on linux os Zied Guermazi
2021-02-26 2:45 ` [PATCH 5/8] fix issue: gdb hangs in the command following a commad returning with TARGET_WAITKIND_NO_HISTORY Zied Guermazi
2021-02-26 7:12 ` Aktemur, Tankut Baris
2021-03-10 20:39 ` Zied Guermazi
2021-02-26 2:46 ` [PATCH 6/8] add support for coresight btrace via remote protocol Zied Guermazi
2021-02-26 2:46 ` [PATCH 7/8] adapt btrace testcases for arm target Zied Guermazi
2021-02-26 2:46 ` [PATCH 8/8] document btrace support for arm targets using coresight etm traces Zied Guermazi
2021-02-26 7:36 ` Eli Zaretskii
2021-03-10 20:36 ` Zied Guermazi
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=3daf0546-63dc-6504-c786-d509647a0043@trande.de \
--to=zied.guermazi@trande.de \
--cc=gdb-patches@sourceware.org \
--cc=markus.t.metzger@intel.com \
--cc=simon.marchi@polymtl.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).