All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Eugene Mavick" <m@mavick.dev>
To: "Peter Zijlstra" <peterz@infradead.org>, "Eugene Mavick" <m@mavick.dev>
Cc: "Will Deacon" <will@kernel.org>, "Boqun Feng" <boqun@kernel.org>,
	"Mark Rutland" <mark.rutland@arm.com>,
	"Gary Guo" <gary@garyguo.net>,
	"Steven Rostedt" <rostedt@goodmis.org>,
	"Masami Hiramatsu" <mhiramat@kernel.org>,
	"Mathieu Desnoyers" <mathieu.desnoyers@efficios.com>,
	"Andrew Morton" <akpm@linux-foundation.org>,
	"Dennis Zhou" <dennis@kernel.org>, "Tejun Heo" <tj@kernel.org>,
	"Christoph Lameter" <cl@gentwo.org>,
	<linux-kernel@vger.kernel.org>,
	<linux-trace-kernel@vger.kernel.org>, <linux-mm@kvack.org>
Subject: Re: [PATCH v4 0/5] tracing: add ref_trace_final_put tracing
Date: Tue, 04 Aug 2026 00:00:02 +0800	[thread overview]
Message-ID: <DKFFMJZBB5MU.3G8Y73IDRTTC3@mavick.dev> (raw)
In-Reply-To: <20260803083750.GA49951@noisy.programming.kicks-ass.net>

On Mon Aug 3, 2026 at 4:37 PM +08, Peter Zijlstra wrote:
> I'm not sure this Changelog is convincing me. To me this just looks like
> more weight / bloat without benefit.
The overhead is negligible; like other TRACE_EVENTS this is an if
statement when disabled.

Knowledge of when an object is considered
end-of-usable-life by the refcount implementation is one of the
first steps in UAF debugging. Alternatives to obtain this
information (GDB, watchpoints, manual instrumentation)
require live analysis.

When debugging from static information of remote origin,
like syzbot, this may not be practical. Live analysis requires bug
reproduction and manual effort.

Bug reproduction, especially in rare timing-sensitive UAFs that
thus require many iterations, may require significant
time/hardware resources.

This tracepoint eliminates significant effort/time cost, and
removes a hardware barrier for a common debugging scenario at a
negligible performance cost.

  reply	other threads:[~2026-08-03 16:00 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-08-01 10:59 [PATCH v4 0/5] tracing: add ref_trace_final_put tracing Eugene Mavick
2026-08-01 10:59 ` [PATCH v4 1/5] tracing: add ref_trace_final_put tracepoint Eugene Mavick
2026-08-01 10:59 ` [PATCH v4 2/5] refcount: " Eugene Mavick
2026-08-01 10:59 ` [PATCH v4 3/5] percpu-refcount: add ref_trace_final_put trace Eugene Mavick
2026-08-01 10:59 ` [PATCH v4 4/5] kunit: add test for ref_trace_final_put Eugene Mavick
2026-08-01 10:59 ` [PATCH v4 5/5] MAINTAINERS: add entries " Eugene Mavick
2026-08-03  8:37 ` [PATCH v4 0/5] tracing: add ref_trace_final_put tracing Peter Zijlstra
2026-08-03 16:00   ` Eugene Mavick [this message]
2026-08-03 12:40 ` Masami Hiramatsu

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=DKFFMJZBB5MU.3G8Y73IDRTTC3@mavick.dev \
    --to=m@mavick.dev \
    --cc=akpm@linux-foundation.org \
    --cc=boqun@kernel.org \
    --cc=cl@gentwo.org \
    --cc=dennis@kernel.org \
    --cc=gary@garyguo.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=linux-trace-kernel@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=mathieu.desnoyers@efficios.com \
    --cc=mhiramat@kernel.org \
    --cc=peterz@infradead.org \
    --cc=rostedt@goodmis.org \
    --cc=tj@kernel.org \
    --cc=will@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.