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
next prev parent 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.