All of lore.kernel.org
 help / color / mirror / Atom feed
From: Steven Rostedt <rostedt@goodmis.org>
To: Sudip Mukherjee <sudipm.mukherjee@gmail.com>
Cc: linux-trace-devel@vger.kernel.org, Emil Thorsoe <ethorsoe@tuxera.com>
Subject: Re: [PATCH] libtraceevent utest: exit test if mmap fails
Date: Sat, 7 Nov 2026 16:20:22 -0500	[thread overview]
Message-ID: <20261107162022.65256a9d@fedora> (raw)
In-Reply-To: <au9QwCTKOwBCZ7f-@debian>

On Sat, 7 Nov 2026 16:24:32 +0000
Sudip Mukherjee <sudipm.mukherjee@gmail.com> wrote:

> Hi Steve,
> 
> On Tue, Jun 16, 2026 at 04:48:12PM -0400, Steven Rostedt wrote:
> > On Mon, 15 Jun 2026 20:25:08 +0100
> > Sudip Mukherjee <sudipm.mukherjee@gmail.com> wrote:
> >   
> > > Hi Steve,
> > > 
> > > On Fri, 29 May 2026 at 19:41, Steven Rostedt <rostedt@goodmis.org> wrote:  
> > > >
> > > > On Tue, 26 May 2026 19:42:19 +0100
> > > > Sudip Mukherjee <sudipm.mukherjee@gmail.com> wrote:
> > > >    
> > > > > CU_TEST() continues execution even if the test fails. It just records
> > > > > that the test has failed and moves on.
> > > > > So, when libtraceevent is built in a chroot, it can open
> > > > > /sys/kernel/btf/vmlinux but fails in mmap stage and since CU_TEST()
> > > > > does not stop so we get a segfault when tep_load_btf() tries to use
> > > > > the mapped buffer.
> > > > >
> > > > > Signed-off-by: Sudip Mukherjee <sudipm.mukherjee@gmail.com>    
> > > >
> > > > Thanks, but the real fix is here and I'm applying that one:
> > > >
> > > >   https://lore.kernel.org/all/d6e07cd6-cb78-4119-a048-920ca58b4830@tuxera.com/    
> > > 
> > > Tried it today, and the test still fails for me.  
> > 
> > Thanks for letting me know. I currently just started a new job and I'm
> > a bit overwhelmed. It may be a couple more weeks before I get to this.  
> 
> Not sure if you had a chance to look at it. I was looking at it now while
> updating the Debian package. It seems like btf_init() is failing when its
> doing if (btf->hdr->hdr_len < sizeof(*btf->hdr)). Adding a debug there shows,
> btf->hdr->hdr_len is 24 whereas sizeof(*btf->hdr) is 32.
> 
> Tried on a chroot and also on a VM, both shows the same problem in Debian.
> 

Bah, because I thought the fix was elsewhere I marked it as
"superseded" and forgot I did that. When you said it was different, I
was traveling and didn't bother to check patchwork to reset it back to
New.

   https://patchwork.kernel.org/project/linux-trace-devel/patch/20260526184219.408223-1-sudipm.mukherjee@gmail.com/

If it's not visible in patchwork, it's not visible to me :-/

I'll go and set it now to New. Thanks for keeping up on it.

-- Steve



  reply	other threads:[~2026-11-07 21:20 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-05-26 18:42 [PATCH] libtraceevent utest: exit test if mmap fails Sudip Mukherjee
2026-05-29 18:41 ` Steven Rostedt
2026-06-15 19:25   ` Sudip Mukherjee
2026-06-16 20:48     ` Steven Rostedt
2026-11-07 16:24       ` Sudip Mukherjee
2026-11-07 21:20         ` Steven Rostedt [this message]
2026-11-08 11:02           ` Sudip Mukherjee

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=20261107162022.65256a9d@fedora \
    --to=rostedt@goodmis.org \
    --cc=ethorsoe@tuxera.com \
    --cc=linux-trace-devel@vger.kernel.org \
    --cc=sudipm.mukherjee@gmail.com \
    /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.