Skip to content

Commit 7e442be

Browse files
Likhitha Korrapatinamhyung
authored andcommitted
perf tools: Fix is_compat_mode build break in ppc64
Commit 54f9aa1 ("tools/perf/powerpc/util: Add support to handle compatible mode PVR for perf json events") introduced to select proper JSON events in case of compat mode using auxiliary vector. But this caused a compilation error in ppc64 Big Endian. arch/powerpc/util/header.c: In function 'is_compat_mode': arch/powerpc/util/header.c:20:21: error: cast to pointer from integer of different size [-Werror=int-to-pointer-cast] 20 | if (!strcmp((char *)platform, (char *)base_platform)) | ^ arch/powerpc/util/header.c:20:39: error: cast to pointer from integer of different size [-Werror=int-to-pointer-cast] 20 | if (!strcmp((char *)platform, (char *)base_platform)) | Commit saved the getauxval(AT_BASE_PLATFORM) and getauxval(AT_PLATFORM) return values in u64 which causes the compilation error. Patch fixes this issue by changing u64 to "unsigned long". Fixes: 54f9aa1 ("tools/perf/powerpc/util: Add support to handle compatible mode PVR for perf json events") Signed-off-by: Likhitha Korrapati <likhitha@linux.ibm.com> Reviewed-by: Athira Rajeev <atrajeev@linux.ibm.com> Link: https://lore.kernel.org/r/20250321100726.699956-1-likhitha@linux.ibm.com Signed-off-by: Namhyung Kim <namhyung@kernel.org>
1 parent 9480cc1 commit 7e442be

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

tools/perf/arch/powerpc/util/header.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,8 @@
1414

1515
static bool is_compat_mode(void)
1616
{
17-
u64 base_platform = getauxval(AT_BASE_PLATFORM);
18-
u64 platform = getauxval(AT_PLATFORM);
17+
unsigned long base_platform = getauxval(AT_BASE_PLATFORM);
18+
unsigned long platform = getauxval(AT_PLATFORM);
1919

2020
if (!strcmp((char *)platform, (char *)base_platform))
2121
return false;

0 commit comments

Comments
 (0)