public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
From: "Metzger, Markus T" <markus.t.metzger@intel.com>
To: Tom de Vries <tdevries@suse.de>
Cc: "gdb-patches@sourceware.org" <gdb-patches@sourceware.org>
Subject: RE: [PATCH] [gdb/testsuite] Add have_linux_btrace_bug
Date: Mon, 13 Feb 2023 19:17:33 +0000	[thread overview]
Message-ID: <DM8PR11MB5749303396B044638563E4C3DEDD9@DM8PR11MB5749.namprd11.prod.outlook.com> (raw)
In-Reply-To: <20230213181001.25142-1-tdevries@suse.de>

Hello Tom,

>The linux kernel commit 670638477aed ("perf/x86/intel/pt: Opportunistically
>use single range output mode"), added in version v5.5.0 had a bug that was
>fixed by commit ce0d998be927 ("perf/x86/intel/pt: Fix sampling using
>single range output") in version 6.1.0.
>
>The bug manifested for intel microarchitectures Rocket Lake, Raptor Lake and
>Alder Lake.

Actually, it's a h/w bug that got exposed by using single-range output.  It affects
Core processors starting from Ice Lake and it only affects Processor Trace.  Also,
it is only exposed by the py-record-btrace test, which does a lot of single-stepping.

It might be better to just add an XFAIL for that one test.  I'm not sure if maintaining
a processor list makes sense.  The kernel patch disables single-range for > 1 page
for all processors and does not try to maintain a list of affected processors.  We might
want to do the same in GDB and either disable that test for kernels between 5.5 and
6.1, or setup an XFAIL.

The Branch Trace Store issue you found seems to affect all btrace tests on ADL
E-cores.  This is a different issue.  I can reproduce it and I am currently debugging it.

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


  reply	other threads:[~2023-02-13 19:18 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-02-13 18:10 Tom de Vries
2023-02-13 19:17 ` Metzger, Markus T [this message]
2023-02-14 10:59   ` [PATCH] [gdb/testsuite] Add xfail in gdb.python/py-record-btrace.exp Tom de Vries
2023-02-14 11:34     ` Metzger, Markus T

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=DM8PR11MB5749303396B044638563E4C3DEDD9@DM8PR11MB5749.namprd11.prod.outlook.com \
    --to=markus.t.metzger@intel.com \
    --cc=gdb-patches@sourceware.org \
    --cc=tdevries@suse.de \
    /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).