On Fri, Aug 16, 2024 at 05:59:08PM +0100, Andre Przywara wrote:
Mark Brown broonie@kernel.org 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.