Re: [PATCH] KVM: arm64: selftests: Check ID regs are immutable after a failed run

--/eGGtsQu6VHOQjKh
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

On Tue, Aug 04, 2026 at 07:30:44PM +0100, Fuad Tabba wrote:
> On Tue, 4 Aug 2026 at 19:14, Mark Brown <[email protected]> wrote:

> > > +     TEST_ASSERT(r < 0 && errno == EBUSY,
> > > +                 "ID reg write after failed first run: r=%d errno=%d",
> > > +                 r, errno);
> > > +     TEST_ASSERT_EQ(vcpu_get_reg(vcpu, reg), val);

> > > +     kvm_vm_free(vm);
> > > +     ksft_test_result_pass("%s\n", __func__);

> > Mixing TEST_ASSERT() and ksft_test_result_ in the same test program also
> > interacts really poorly with automation, TEST_ASSERT() just kills the

> Agreed, but it looks file-wide rather than specific to this test, so I
> would rather leave it to your series than convert one function. Happy
> to follow once that lands.

Unfortunately at the minute it's not even consistent within the program -
the MPAM tests in there do use ksft_ based reporting, while they do so
with inconsistent strings you at least get all the results out if it's
just them that blow up.

--/eGGtsQu6VHOQjKh
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmpyMScACgkQJNaLcl1U
h9AAbgf/W+3LTMl/N5/eGuB3SteA/AMMoqc9R14TDJVBLdsyQ54rto/8rO1S6R2K
6z5XXdzQEyAUun4VTtWRdVNC4EBGCgFmx6iL0NIA7Mq/Hgg9Y5Kg4O9J+AZhUa7s
QkyroJpscInKTT2Z6AwulpLV1DGOuoaNsWykIUzLjTT3Fy1rzBh9gGSc4SkGOXjL
iZndvJCwijyJr0Co+C/6kXSffdmtoLXShJ0GQJTjgjDzlNo/Ofp9sQyY55n3+fxW
18Onf1DAZXOpaa+XZ+SxVFPasuCDoutSoPJgVPygau5lbWJkU2bLzX24FT5QHKK0
tkolz+CDfa4jgTcf0U5aMUw5ihJTfg==
=oRTP
-----END PGP SIGNATURE-----

--/eGGtsQu6VHOQjKh--