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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 786BCC624D4 for ; Wed, 2 Sep 2026 06:15:20 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date:Subject:Cc :To:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=Ge4kNm/RpEzImRWm1Gepia3eh+dSgPmhU9fPFoloPEI=; b=wDMovdIgIUPd21 Wfa1SP1gM+GoKk8VBWQXkOLMCjBfnKFUabWY7iTad06nw0FLWniy6ul7HWwtg/bUFCOn7Qi7jPNyP Vv0hPaGw/iUnwKLRcdBcWIQacQwFilvKfbfd+BQ9q7FUN7AGLNl4mHZLEirXtFb64J3YeppRj+OuK 7RSrL5nG2AZTTi3K5bIIDeHMRHIObn2qjHmPA/HHIl+sLQQzEHeaLeXW9vI03YMdEt7+uVdqpNAVa UFsDzh4+OrFLUfIW8R7LEMcRL8vZdPZWIclR0b46pJfkNcoVWUxlV6OGkZimvevwG0gOGXJurNmz/ 8UFyRxywrOmaxocTWntg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1x1eFV-0000000DqLU-39DT; Wed, 02 Sep 2026 06:15:05 +0000 Received: from out30-99.freemail.mail.aliyun.com ([115.124.30.99]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1x1eFR-0000000DqKP-0kdh for linux-riscv@lists.infradead.org; Wed, 02 Sep 2026 06:15:03 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.alibaba.com; s=default; t=1788329697; h=From:To:Subject:Date:Message-ID:MIME-Version; bh=UL0E16epy44PNt8yQn/2+BDsOBNXFG2a9/inh0OfbBo=; b=NBamitoT3fEpHtGF+XbnQCFf3UlkdZ5VLSiIpct76hVGuwnBfip+dXwmEfkjTq1iWt2lyS+K73XbsXGdGWTaf9dM/bBoxDINqLiVQeCIf4CCb/75CCMlXORaWW4b9lkgLoqdqlN9hW1rvLcczpEQnTbyF6w60ukYcjs45tQQ1NU= X-Alimail-AntiSpam: AC=PASS;BC=-1|-1;BR=01201311R101e4;CH=green;DM=||false|;DS=||;FP=0|-1|-1|-1|0|-1|-1|-1;HT=maildocker-contentspam033045133197;MF=cp0613@linux.alibaba.com;NM=1;PH=DS;RN=21;SR=0;TI=SMTPD_---0XABSX4-_1788329692; Received: from DESKTOP-S9E58SO.localdomain(mailfrom:cp0613@linux.alibaba.com fp:SMTPD_---0XABSX4-_1788329692 cluster:ay36) by smtp.aliyun-inc.com; Wed, 02 Sep 2026 14:14:56 +0800 From: Chen Pei To: ast@kernel.org, daniel@iogearbox.net, andrii@kernel.org, memxor@gmail.com, bjorn@kernel.org, puranjay@kernel.org Cc: ihor.solodrai@linux.dev, eddyz87@gmail.com, martin.lau@linux.dev, song@kernel.org, yonghong.song@linux.dev, jolsa@kernel.org, emil@etsalapatis.com, pulehui@huawei.com, pjw@kernel.org, palmer@dabbelt.com, guoren@kernel.org, bpf@vger.kernel.org, linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org, stable@vger.kernel.org Subject: [PATCH bpf v2] bpf, riscv: Make arena support depend on ZACAS Date: Wed, 2 Sep 2026 14:14:51 +0800 Message-ID: <20260902061451.1416-1-cp0613@linux.alibaba.com> X-Mailer: git-send-email 2.43.0 MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260901_231501_789785_B8ECA4F8 X-CRM114-Status: GOOD ( 10.03 ) X-BeenThere: linux-riscv@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-riscv" Errors-To: linux-riscv-bounces+linux-riscv=archiver.kernel.org@lists.infradead.org The arena range tree allocates its nodes with kmalloc_nolock() since commit f8c67d8550ee ("bpf: Use kmalloc_nolock() in range tree"). kmalloc_nolock() requires slab caches with cmpxchg128 support (__CMPXCHG_DOUBLE); on riscv cmpxchg128 is provided by the ZACAS extension. On systems without ZACAS every arena map creation fails with a misleading -ENOMEM. Report the missing support instead: make bpf_jit_supports_arena() return system_has_cmpxchg128() where it is defined, so arena map creation fails with -EOPNOTSUPP on systems without ZACAS. The macro is only defined when both CONFIG_RISCV_ISA_ZACAS and CONFIG_TOOLCHAIN_HAS_ZACAS are enabled, so guard it with #ifdef the same way mm/slab.h consumes it, and reject arena otherwise. This matches how arena BPF_CMPXCHG instructions are already gated on ZACAS in bpf_jit_supports_insn(). Fixes: f8c67d8550ee ("bpf: Use kmalloc_nolock() in range tree") Cc: stable@vger.kernel.org Signed-off-by: Chen Pei --- Changes since v1: - Guard system_has_cmpxchg128() with #ifdef instead of calling it unconditionally: the macro is only defined when both CONFIG_RISCV_ISA_ZACAS and CONFIG_TOOLCHAIN_HAS_ZACAS are enabled (as reported by sashiko-bot), so v1 broke the build when either was disabled. This mirrors how mm/slab.h consumes the macro. Why #ifdef rather than rv_ext_enabled(ZACAS)? The predicates differ exactly in the configurations that matter: scenario (ISA_ZACAS/TOOLCHAIN/hw) v1 rv_ext_enabled #ifdef ISA=n or TOOLCHAIN=n build fails rejects rejects ISA=y TOOLCHAIN=n hw has ZACAS build fails accepts, then rejects -ENOMEM again ISA=y TOOLCHAIN=y hw has ZACAS exact exact exact rv_ext_enabled(ZACAS) does not check CONFIG_TOOLCHAIN_HAS_ZACAS, but slab's cmpxchg128 - and thus kmalloc_nolock() - does require it, so on an old toolchain with ZACAS hardware it would accept arena maps and bring back the very -ENOMEM failure this patch fixes. The #ifdef form builds in every configuration and matches exactly the kmalloc_nolock() availability gate in mm/slab.h. This issue was reported by sashiko-bot: https://sashiko.dev/#/patchset/20260901120013.16104-1-cp0613@linux.alibaba.com?part=1 Question for reviewers: should the arena selftests gate on ZACAS, e.g. probing it via riscv_hwprobe() (RISCV_ISA_EXT_ZACAS) and SKIPping cleanly on systems without the extension? arch/riscv/net/bpf_jit_comp64.c | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/arch/riscv/net/bpf_jit_comp64.c b/arch/riscv/net/bpf_jit_comp64.c index 74efe4b138d2..151031e97a24 100644 --- a/arch/riscv/net/bpf_jit_comp64.c +++ b/arch/riscv/net/bpf_jit_comp64.c @@ -2128,7 +2128,15 @@ bool bpf_jit_supports_ptr_xchg(void) bool bpf_jit_supports_arena(void) { - return true; + /* + * The arena range tree uses kmalloc_nolock(), which needs + * cmpxchg128, provided by ZACAS on riscv. + */ +#ifdef system_has_cmpxchg128 + return system_has_cmpxchg128(); +#else + return false; +#endif } bool bpf_jit_supports_insn(struct bpf_insn *insn, bool in_arena) -- 2.50.1 _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv