On Fri, Aug 16, 2024 at 05:59:08PM +0100, Andre Przywara wrote:
> Mark Brown <broonie@xxxxxxxxxx> wrote:
> > On Fri, Aug 16, 2024 at 04:32:50PM +0100, Andre Przywara wrote:
> > > Fixes: e9b60476bea0 ("kselftest/arm64: Add utilities and a test to validate mte memory")
> > This is another one where calling it a fix seems like it's pushing it,
> > it's a modernisation rather than a correctness thing.
> Well, I get compiler warnings, so I thought "fix" would be adequate. But
> in general this confusion between pointers and integers sounds not good,
> and not using %p looks like a genuine bug to me (though it's admittedly
> working fine (TM) for now).
Those compiler warnings are a relatively new thing - they won't have
been triggered when the code was written, and practically speaking on
arm64 it's more of a C language correctness thing than a meaningful
change.
Attachment:
signature.asc
Description: PGP signature
