[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <137eabdcb1494debd9f321ad6dc39443c084ad34.camel@gmail.com>
Date: Wed, 19 Feb 2025 17:21:31 -0800
From: Eduard Zingerman <eddyz87@...il.com>
To: Peilin Ye <yepeilin@...gle.com>
Cc: bpf@...r.kernel.org, linux-arm-kernel@...ts.infradead.org, bpf@...f.org,
Xu Kuohai <xukuohai@...weicloud.com>, David Vernet <void@...ifault.com>,
Alexei Starovoitov <ast@...nel.org>, Daniel Borkmann
<daniel@...earbox.net>, Andrii Nakryiko <andrii@...nel.org>, Martin KaFai
Lau <martin.lau@...ux.dev>, Song Liu <song@...nel.org>, Yonghong Song
<yonghong.song@...ux.dev>, John Fastabend <john.fastabend@...il.com>, KP
Singh <kpsingh@...nel.org>, Stanislav Fomichev <sdf@...ichev.me>, Hao Luo
<haoluo@...gle.com>, Jiri Olsa <jolsa@...nel.org>, Jonathan Corbet
<corbet@....net>, "Paul E. McKenney" <paulmck@...nel.org>, Puranjay Mohan
<puranjay@...nel.org>, Ilya Leoshkevich <iii@...ux.ibm.com>, Heiko Carstens
<hca@...ux.ibm.com>, Vasily Gorbik <gor@...ux.ibm.com>, Catalin Marinas
<catalin.marinas@....com>, Will Deacon <will@...nel.org>, Quentin Monnet
<qmo@...nel.org>, Mykola Lysenko <mykolal@...com>, Shuah Khan
<shuah@...nel.org>, Ihor Solodrai <ihor.solodrai@...ux.dev>, Yingchi Long
<longyingchi24s@....ac.cn>, Josh Don <joshdon@...gle.com>, Barret Rhoden
<brho@...gle.com>, Neel Natu <neelnatu@...gle.com>, Benjamin Segall
<bsegall@...gle.com>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH bpf-next v2 8/9] selftests/bpf: Add selftests for
load-acquire and store-release instructions
On Thu, 2025-02-20 at 01:08 +0000, Peilin Ye wrote:
[...]
> Then realized that I can't check __clang_major__ in .../prog_tests/*
> files (e.g. I was building prog_tests/verifier.c using GCC). I think
> ideally we should do something similar to prog{,_test}s/arena_atomics.c,
> i.e. use a global bool in BPF source to indicate if we should skip this
> test, but that seems to require non-trivial changes to
> prog_tests/verifier.c?
>
> For the purpose of this patchset, let me keep dummy_test(), like what we
> have now in verifier_ldsx.c.
Well, that's unfortunate.
Thank you for trying.
Powered by blists - more mailing lists