From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 7AC88C43458 for ; Mon, 6 Jul 2026 06:47:40 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 42F856B00F2; Mon, 6 Jul 2026 02:47:31 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 257936B00F7; Mon, 6 Jul 2026 02:47:31 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id EB3226B00F4; Mon, 6 Jul 2026 02:47:30 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0010.hostedemail.com [216.40.44.10]) by kanga.kvack.org (Postfix) with ESMTP id BD9FB6B00F1 for ; Mon, 6 Jul 2026 02:47:30 -0400 (EDT) Received: from smtpin18.hostedemail.com (lb01a-stub [10.200.18.249]) by unirelay10.hostedemail.com (Postfix) with ESMTP id 4E687C17F6 for ; Mon, 6 Jul 2026 06:47:30 +0000 (UTC) X-FDA: 84957420660.18.43F4130 Received: from mout-y-111.mailbox.org (mout-y-111.mailbox.org [91.198.250.236]) by imf27.hostedemail.com (Postfix) with ESMTP id 7683940006 for ; Mon, 6 Jul 2026 06:47:28 +0000 (UTC) Authentication-Results: imf27.hostedemail.com; dkim=none; spf=pass (imf27.hostedemail.com: domain of m@mavick.dev designates 91.198.250.236 as permitted sender) smtp.mailfrom=m@mavick.dev; dmarc=pass (policy=quarantine) header.from=mavick.dev ARC-Seal: i=1; a=rsa-sha256; d=hostedemail.com; s=arc-20220608; cv=none; t=1783320448; b=m2z8LJhoYzAcDKVA+bwGxBrCMsvgbOFRCjNthVAW0VXP0aIHzriGojAHslLz4sMSZGnMR7 Pbv4BPxs2T9jNqUOlwnb6/l9E2Vp9g8nNnfc2fA6w4hsB4g/kYLWbsvJpnrBG0t6TDOzLQ Mw6oInp5Ruij/j5dNBtfWePKoQJo3DA= ARC-Authentication-Results: i=1; imf27.hostedemail.com; dkim=none; spf=pass (imf27.hostedemail.com: domain of m@mavick.dev designates 91.198.250.236 as permitted sender) smtp.mailfrom=m@mavick.dev; dmarc=pass (policy=quarantine) header.from=mavick.dev ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1783320448; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=eRtKchl4rToxBOwHE1iEk42Um7GXykTshGoJLgl25Qg=; b=tCjlbaOl+TcmogBivZBhK5SVcH6XWk2m0unYenqLbG+LAgS1N+HkKfnINWpLpi4by149bx sAXkEt/XY1RPxi/pJOVy1MmmkfrDyj4FDRmFHOwkUyL3igMi7s9BX56qVxs7JzCvnp9+c5 lSQtgv7cPct7zwK55XkwMpmyNmzdU+w= Received: from smtp1.mailbox.org (smtp1.mailbox.org [IPv6:2001:67c:2050:b231:465::1]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange x25519 server-signature RSA-PSS (4096 bits) server-digest SHA512) (No client certificate requested) by mout-y-111.mailbox.org (Postfix) with ESMTPS id 4gt7rP64vtzMldG; Sun, 05 Jul 2026 02:38:13 +0200 (CEST) From: Eugene Mavick Date: Sun, 05 Jul 2026 08:36:59 +0800 Subject: [PATCH 4/4] kunit: add test for ref_trace_final_put MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Message-Id: <20260705-refcount-final-put-trace-v1-4-a41d3fd0e869@mavick.dev> References: <20260705-refcount-final-put-trace-v1-0-a41d3fd0e869@mavick.dev> In-Reply-To: <20260705-refcount-final-put-trace-v1-0-a41d3fd0e869@mavick.dev> To: Will Deacon , Peter Zijlstra , Boqun Feng , Mark Rutland , Gary Guo , Steven Rostedt , Masami Hiramatsu , Mathieu Desnoyers , Andrew Morton , Dennis Zhou , Tejun Heo , Christoph Lameter Cc: linux-kernel@vger.kernel.org, linux-trace-kernel@vger.kernel.org, linux-mm@kvack.org, Eugene Mavick X-Developer-Signature: v=1; a=openpgp-sha256; l=5737; i=m@mavick.dev; h=from:subject:message-id; bh=uO0yuH0cgFlwuh+60ouE/XunglpmQTZB7vngb53hITQ=; b=owGbwMvMwCU2V/5U3e1aE3/G02pJDFmey20VX05ecCd9rePKpKMlC8ovJHy8HfDHcdOvjy/yW 9mfS07t7ihlYRDjYpAVU2SpyUpwz1g5K9H46KQemDmsTCBDGLg4BWAiRzUYGSb2GsrfPvFUpdcn fe+6vx/NGYpPMf2OZv1y+ihbpIFEjzMjw4vN+y6UyN6cdY7J6d9Kr1P/ZhfP6sj+eyHIp12PK4x HhxUA X-Developer-Key: i=m@mavick.dev; a=openpgp; fpr=7C6A604768A99A6133C5928C9D1FCA7EDB7D344F X-Rspam-User: X-Rspamd-Server: rspam11 X-Rspamd-Queue-Id: 7683940006 X-Stat-Signature: jimdrfdpjgqh5qhj6ria87fcijg5mgs1 X-HE-Tag: 1783320448-441431 X-HE-Meta: U2FsdGVkX19OtXQHoe3M8Kd39ic4jiv/FKiolW5rGFAtXiYDNZjmgdokDfrreSvZWYhBdrIgruFIuykVbsd8EdNEIrQAwvXoNtV1NpTG83ptooqg+30QjHvqybN+v9jkNL5NJdeSPNQQegYQ0vxX4E7+hgm+W2C6miXQKmXW0R1oAepSqsw9wazDu68jTF6Jb4jqPqIN5Q6v5p6EplXza/JJwLX0WKhQxIaT5+MVAhNZLS1qH+b/UALp0xqaZvZym2nog8zBAWjo/HImNJ0zI9iBtLQ6lS7uS/VNu3iIMGTE9U42kez3mNO7dw7CmjUM4PK//qx+5wsm5qdYDA++9wmiH4PMWajkmyZGV+efViBh5hzGIfBip/6kP0W9fwYFHaLXzf2woH8m96NpZsEnxRTTg3/Iw+E3+PB6yboEaOZp4vSs4FMLzliHjC4P1a808nh+ys/XHlS4cigzBxe2glhjH7048QTLljYDAgQ1YIC0fBw1oQSmsFtX95r0YFdN9FSf2Clt/66s71PtKNALVOzgqYcVtAfIjOeeXw9py250OJEIoR0odypPazv0PB9CbQhZV6PTZ0oPfd3rn8cln0G0d27pgEVSFwgvT6W8Cq0be5DC0sezC43MvzdMXRJvdYwSVL8uIFqHWXkxDdR/mr75+nR5Ux+Hhr9tGo/L9Wd0tOVxdw6llj69Ok+rqwadXHZI8sYoSEwhvN7HTzMZT/RN62uv+o7o7uwzHLP4GnFF8dZF8M6FW6QFOX5op7Yy8HHVpP4BaGJcgK2ymPKAxtiZ9yvPoHnRjeUnodrwxlcumuILxDeTNk9cdDJKuk2lGxHj0JepyO+C9zcQ/xhdGNf3aHhjNxbWCvXvfZvxG5ORyFeukL4JilOjN0GKDaWWXJMevppHfLZlpJIhKGB3zraYAtbEEnK0XetJeRiMAQRn3Cdh9VVlMzj3vxjo9q63YAkXymm+KLO3lr4pbaz Vd4Xdm4M ZN4LIr6cQUJNozqUtuvqyRZomidVh+91ZdFX65VrqEPkLW9bF9hawcn/Bhg3YSSXfivICXjYZVMdBPah6n5GkIxv8D+OQBhypyvwqCJMQzZq6XHXNZChwkZFo2UC6NYkz5o2NlI8pnVKUU/cK04NLnmMuNMYrBYZz7LYevSN6qm1HpwgBEIJzthticnXvmjfsAjE0K/+dVFkLhrrmfN79UoNPXo2FRlFH0JZtoTtHtzzD0tNJBbiv6NXiP70BN3Ldk4+Y Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: Add a KUnit test suite for the ref_trace_final_put tracepoint. The test registers a probe function and triggers both refcount_t and percpu_ref final put paths, verifying that the tracepoint fires correctly and that the recorded fields match expected values. Signed-off-by: Eugene Mavick --- MAINTAINERS | 1 + lib/Kconfig | 10 ++++ lib/tests/Makefile | 1 + lib/tests/ref_trace_kunit.c | 138 ++++++++++++++++++++++++++++++++++++++++++++ 4 files changed, 150 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 8dab37726b9d..7c387ed746ee 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -4213,6 +4213,7 @@ F: include/*/atomic*.h F: include/linux/ref_trace.h F: include/linux/refcount.h F: lib/ref_trace.h +F: lib/tests/ref_trace_kunit.c F: scripts/atomic/ F: rust/kernel/sync/atomic.rs F: rust/kernel/sync/atomic/ diff --git a/lib/Kconfig b/lib/Kconfig index 00a9509636c1..c29b2ccb3c31 100644 --- a/lib/Kconfig +++ b/lib/Kconfig @@ -52,6 +52,16 @@ config PACKING_KUNIT_TEST When in doubt, say N. +config REF_TRACE_KUNIT_TEST + bool "ref_trace kunit test" if !KUNIT_ALL_TESTS + depends on KUNIT && FTRACE + default KUNIT_ALL_TESTS + help + This option enables the KUnit test suite for the ref_trace_final_put + tracepoint. + + If unsure, say N + config BITREVERSE tristate diff --git a/lib/tests/Makefile b/lib/tests/Makefile index 7e9c2fa52e35..828a030ad8c7 100644 --- a/lib/tests/Makefile +++ b/lib/tests/Makefile @@ -57,5 +57,6 @@ obj-$(CONFIG_USERCOPY_KUNIT_TEST) += usercopy_kunit.o obj-$(CONFIG_UTIL_MACROS_KUNIT) += util_macros_kunit.o obj-$(CONFIG_RATELIMIT_KUNIT_TEST) += test_ratelimit.o obj-$(CONFIG_UUID_KUNIT_TEST) += uuid_kunit.o +obj-$(CONFIG_REF_TRACE_KUNIT_TEST) += ref_trace_kunit.o obj-$(CONFIG_TEST_RUNTIME_MODULE) += module/ diff --git a/lib/tests/ref_trace_kunit.c b/lib/tests/ref_trace_kunit.c new file mode 100644 index 000000000000..2ece6c840fe0 --- /dev/null +++ b/lib/tests/ref_trace_kunit.c @@ -0,0 +1,138 @@ +// SPDX-License-Identifier: GPL-2.0 +#include +#include +#include +#include +#include +#include +#include +#include + +struct data { + unsigned long caller; + const char *fn; + const void *obj; + int *flag; + struct kunit *test; +}; + +struct data refc_chk; +struct data pcpu_chk; + +//called when tracepoint fires +static void probe( + void *ignore, + unsigned long caller, + const char *fn, + const void *obj) +{ + struct data *chk_val; + + char func_name[KSYM_SYMBOL_LEN]; + + sprint_symbol_no_offset(func_name, caller); + + if (!strcmp(func_name, "test_refcount")) { + chk_val = &refc_chk; + KUNIT_EXPECT_FALSE(chk_val->test, memcmp(obj, chk_val->obj, sizeof(refcount_t))); + } else if (!strcmp(func_name, "test_percpu")) { + chk_val = &pcpu_chk; + KUNIT_EXPECT_FALSE(chk_val->test, memcmp( + obj, chk_val->obj, sizeof(struct percpu_ref))); + } else { + //non test function origin trace events + return; + } + + struct kunit *test = chk_val->test; + int *flag = chk_val->flag; + + //ensure past flag writes are done before reading + KUNIT_EXPECT_EQ(test, 1, smp_load_acquire(flag)); + KUNIT_EXPECT_EQ(test, caller, chk_val->caller); + + smp_store_release(flag, 0); //signal probe completion +} + + + +static void test_refcount(struct kunit *test) +{ + refcount_t refc; + int flag_addr = 0; + + refc_chk.caller = (unsigned long)&test_refcount; + refc_chk.fn = "__refcount_sub_and_test"; + refc_chk.obj = &refc; + refc_chk.flag = &flag_addr; + refc_chk.test = test; + + int *flag = refc_chk.flag; + + KUNIT_EXPECT_FALSE(test, register_trace_ref_trace_final_put(probe, NULL)); + + refcount_set(&refc, 2); + + KUNIT_EXPECT_FALSE(test, refcount_dec_and_test(&refc)); + + smp_store_release(flag, 1); //signal final put can happen + + KUNIT_EXPECT_TRUE(test, refcount_dec_and_test(&refc)); + + wait_var_event(flag, smp_load_acquire(flag)); //wait for probe completion + unregister_trace_ref_trace_final_put(probe, NULL); +} + +static void dummy_release(struct percpu_ref *ref) {} + +static void test_percpu(struct kunit *test) +{ + struct percpu_ref pcpu_ref; + + int flag_addr = 1; + + pcpu_chk.caller = (unsigned long)&test_percpu; + pcpu_chk.fn = "percpu_ref_put_many"; + pcpu_chk.obj = &pcpu_ref; + pcpu_chk.flag = &flag_addr; + pcpu_chk.test = test; + + int *flag = pcpu_chk.flag; + + KUNIT_EXPECT_FALSE(test, register_trace_ref_trace_final_put(probe, NULL)); + + KUNIT_EXPECT_FALSE(test, percpu_ref_init(&pcpu_ref, dummy_release, 0, GFP_KERNEL)); + + percpu_ref_get(&pcpu_ref); + percpu_ref_get(&pcpu_ref); + + percpu_ref_put(&pcpu_ref); + percpu_ref_put(&pcpu_ref); + + percpu_ref_switch_to_atomic_sync(&pcpu_ref); + + smp_store_release(flag, 1); //signal final put can happen + + percpu_ref_put(&pcpu_ref); + + wait_var_event(flag, smp_load_acquire(flag)); //wait for probe completion + unregister_trace_ref_trace_final_put(probe, NULL); + percpu_ref_exit(&pcpu_ref); +} + +static struct kunit_case __refdata ref_trace_test_cases[] = { + KUNIT_CASE(test_refcount), + KUNIT_CASE(test_percpu), + {} +}; + +static struct kunit_suite ref_trace_test_suite = { + .name = "ref-trace", + .test_cases = ref_trace_test_cases, +}; + +kunit_test_suites(&ref_trace_test_suite); + +MODULE_AUTHOR("Eugene Mavick "); +MODULE_DESCRIPTION("KUnit test for ref_trace"); +MODULE_LICENSE("GPL"); -- 2.51.2