Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Thu, 27 Apr 2023 11:56:53 -0700
In-Reply-To: <CABRcYm+O-_GGhnAmJW6_=9vKeKSvzVLcxBRq3Pfjb3W0_HNjhw@mail.gmail.com>
Mime-Version: 1.0
References: <20230427143207.635263-1-revest@chromium.org> <CABRcYm+O-_GGhnAmJW6_=9vKeKSvzVLcxBRq3Pfjb3W0_HNjhw@mail.gmail.com>
Message-ID: <ZErFdVXHhEdJ/m3G@google.com>
Subject: Re: [PATCH bpf-next] selftests/bpf: Update the aarch64 tests deny list
From:   Stanislav Fomichev <sdf@google.com>
To:     Florent Revest <revest@chromium.org>
Cc:     bpf@vger.kernel.org, linux-kernel@vger.kernel.org, ast@kernel.org,
        daniel@iogearbox.net, andrii@kernel.org, kpsingh@kernel.org,
        mykolal@fb.com, martin.lau@linux.dev, song@kernel.org,
        xukuohai@huaweicloud.com, mark.rutland@arm.com
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

On 04/27, Florent Revest wrote:
> On Thu, Apr 27, 2023 at 4:32=E2=80=AFPM Florent Revest <revest@chromium.o=
rg> wrote:
> >
> > This patch updates the list of BPF selftests which are known to fail so
> > the BPF CI can validate the tests which pass now.
>=20
> Note: I tested this denylist a few months back by sending a manual PR
> to https://github.com/kernel-patches/bpf.
> At the time, it worked
> https://github.com/kernel-patches/bpf/actions/runs/4106542133/jobs/708551=
4761
> (even though there seemed to be a known flake in the gcc variant but
> unrelated to the new arch support)
>=20
> Every time I wanted to have the CI run on my PRs, I had to annoy Manu
> by email (I wouldn't have the rights to trigger the CI by myself
> otherwise). So I haven't tested this *actual* patch rebased on the
> current CI before sending it to the list. (e.g. the
> module_fentry_shadow test has been added since then and I just assumed
> it would pass in CI like the rest)
>=20
> My understanding is that this patch should soon be picked up by the
> testing bot and we can use that CI run to check that everything works
> as intended. Let's wait for a CI green light before merging this! :)
> If there are errors I'll send a v2

Looks green now:

Acked-by: Stanislav Fomichev <sdf@google.com>

https://github.com/kernel-patches/bpf/actions/runs/4822595792/jobs/85907322=
78