RE: [PATCH v4 10/10] btrace: Extend ptwrite event decoding.

["Willgerodt, Felix" <felix.willgerodt@intel.com>]

> -----Original Message-----
> From: Eli Zaretskii <eliz@gnu.org>
> Sent: Dienstag, 10. Mai 2022 16:12
> To: Willgerodt, Felix <felix.willgerodt@intel.com>
> Cc: Metzger, Markus T <markus.t.metzger@intel.com>; gdb-
> patches@sourceware.org
> Subject: Re: [PATCH v4 10/10] btrace: Extend ptwrite event decoding.
> 
> > From: "Willgerodt, Felix" <felix.willgerodt@intel.com>
> > CC: "Metzger, Markus T" <markus.t.metzger@intel.com>,
> > 	"gdb-patches@sourceware.org" <gdb-patches@sourceware.org>
> > Date: Tue, 10 May 2022 13:43:02 +0000
> >
> > > The last line is too long, and will cause overfull hbox errors when
> > > producing the printed manual.  Please make it shorter, or break into 2
> > > lines.
> > >
> >
> > I made it shorter. Is there any easy way to figure this out? I don't know
> what
> > Is short enough
> 
> I think the rule of thumb is that anything longer than 70 characters
> in @smallexample should be split.
>

Thanks that is good to know!

> > and I couldn't see any  "Overfull \hbox" message for
> > python.texi in gdb.log or in the output for "make pdf".
> 
> I believe that's because we have @finalout in gdb.texinfo.
>

For what it's worth: There are some overfull messages, even
for python.texi. Just not for my changes.

> > > You say "hardware and operating system support", but say nothing about
> > > the OSes on which this can be supported.
> > >
> >
> > Any OS can choose to support it at some point. Right now it is Linux only
> afaik.
> > But do we want to say that? It might be out-dated at some point.
> 
> I don't know.  How probable is it that this will be supported on other
> systems soon enough?

I don't know. Microsoft has added some IPT support for performance
monitoring recently:
https://docs.microsoft.com/en-us/windows-server/virtualization/hyper-v/manage/performance-monitoring-hardware
I am not aware what that means for debuggers. GDB currently doesn't
Support IPT on Windows afaik, but I have never tried it.
But it certainly doesn't seem completely out of the picture for the future.

Thanks,
Felix
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