All of lore.kernel.org
 help / color / mirror / Atom feed
From: Namhyung Kim <namhyung@kernel.org>
To: Ian Rogers <irogers@google.com>
Cc: Arnaldo Carvalho de Melo <acme@kernel.org>,
	Dmitry Dolgov <9erthalion6@gmail.com>,
	linux-perf-users@vger.kernel.org,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [RFC PATCH v1] perf tests: Test annotate with data type profiling and rust
Date: Wed, 4 Feb 2026 13:52:19 -0800	[thread overview]
Message-ID: <aYO_k4G-iF3CUY02@google.com> (raw)
In-Reply-To: <CAP-5=fU-aJYKAcbFeFYcgMfpvt8oY4BTxEgczP_rvUAiJt+U-Q@mail.gmail.com>

On Tue, Feb 03, 2026 at 10:15:30AM -0800, Ian Rogers wrote:
> On Tue, Feb 3, 2026 at 6:45 AM Arnaldo Carvalho de Melo <acme@kernel.org> wrote:
> >
> > On Sun, Feb 01, 2026 at 11:33:54AM +0100, Dmitry Dolgov wrote:
> > > > On Fri, Jan 30, 2026 at 09:44:47AM +0100, Dmitry Dolgov wrote:
> > > > > I'm not managing to reproduce your results, can you please elaborate
> > > > > some more?
> > > >
> > > > That's most unexpected for me, the test output I see looks like this:
> > > >
> > > >          : 54     for _ in 1..count {
> > > >     0.00 :   5423b8: mov    %ecx,0x7c(%rsp)
> > > >     0.00 :   5423bc: mov    %eax,0x80(%rsp)
> > > >     0.00 :   5423c3: lea    0x7c(%rsp),%rdi
> > > >    12.28 :   5423c8: call   541f00 <core::iter::range::<impl core::iter::traits::iterator::Iterator for core::ops::range::Range<A>>::next>
> > > >     0.00 :   5423cd: mov    %eax,0x20(%rsp)
> > > >    10.21 :   5423d1: jmp    5423d3 <test_rs+0xb3>
> > > >     0.00 :   5423d3: mov    0x20(%rsp),%eax
> > > >     0.00 :   5423d7: mov    %eax,%eax
> > > >     0.00 :   5423d9: test   $0x1,%rax
> > > >     0.29 :   5423df: je     5423f6 <test_rs+0xd6>
> > > >          : 65     b.data1 += 1;
> > > >    13.69 :   5423e1: mov    0x48(%rsp),%rax     # data-type: struct Buf +0x18 (data1)
> > > >     0.00 :   5423e6: add    $0x1,%rax
> > > >     0.00 :   5423ea: mov    %rax,0x18(%rsp)     # data-type: u64 +0
> > > >     0.00 :   5423ef: setb   %al
> > > >    11.17 :   5423f2: jb     54243a <test_rs+0x11a>
> > > >     0.00 :   5423f4: jmp    542426 <test_rs+0x106>
> > > >     0.00 :   5423f6: lea    0x30(%rsp),%rdi
> > > >          : 73     }
> > > >
> > > > Where '# data-type: struct Buf' is as far as I understand a manifestation of
> > > > data type profiling succeeding. But my environment has slightly different
> > > > version of rust -- I need to figure out what's going on here, thanks.
> > >
> > > I've being testing this in the same way, but without any rust code --
> > > just on a simple datasym workload. Surprisingly, it seems that
> > > 'data-type: buf' marks are disappearing even in this case starting from:
> > >
> > > c31040085914f1188720073baa43d1483693c0a3 (perf dwarf-regs: Clean up x86
> > > dwarf_regnum code)
> >
> > > I'm not sure yet why this is happening, but don't see anything in the
> > > commit mentioning that -- since there are no code-with-type tests yet,
> > > maybe an unexpected side effect?
> >
> > Ian, can you please take a look at this?
> 
> That particular commit is largely adding an i386 to dwarf register
> number mapping table but it also expands the x86-64 mapping table:
> https://web.git.kernel.org/pub/scm/linux/kernel/git/perf/perf-tools-next.git/commit/?h=perf-tools-next&id=c31040085914f1188720073baa43d1483693c0a3
> The code to search the table is largely unchanged so my guess would be
> that the new x86-64 entries are causing a dwarf register to be
> returned in cases where previously this was -ENOENT. This may have
> broken the type profiling, but Namhyung would be the expert on that.

Any chance it can get an invalid e_machine?  I'm not sure if it handles
EM_HOST properly.

Thanks,
Namhyung


  reply	other threads:[~2026-02-04 21:52 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-01-27  8:30 [RFC PATCH v1] perf tests: Test annotate with data type profiling and rust Dmitrii Dolgov
2026-01-27 17:00 ` Arnaldo Carvalho de Melo
2026-01-30  8:44   ` Dmitry Dolgov
2026-02-01 10:33     ` Dmitry Dolgov
2026-02-03 14:45       ` Arnaldo Carvalho de Melo
2026-02-03 18:15         ` Ian Rogers
2026-02-04 21:52           ` Namhyung Kim [this message]
2026-02-04 22:18             ` Ian Rogers

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=aYO_k4G-iF3CUY02@google.com \
    --to=namhyung@kernel.org \
    --cc=9erthalion6@gmail.com \
    --cc=acme@kernel.org \
    --cc=irogers@google.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-perf-users@vger.kernel.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.