[PATCH] fix zero builds for "unknown" architectures
Coleen Phillimore
coleen.phillimore at oracle.com
Wed Sep 9 16:56:29 UTC 2015
I think I like patch #1 better. I will sponsor it and make sure it
works on my machine. Thank you for fixing this on the other platforms.
Coleen
On 9/9/15 12:19 PM, Matthias Klose wrote:
> seen with jdk9 / tag jdk9-b80. zero builds which don't match one of the hotspot
> architectures fail to build in src/os/linux/vm/os_linux.cpp, because there is no
> default / or else clause:
>
> const char* search_string = IA32_ONLY("model name") AMD64_ONLY("model name")
> IA64_ONLY("") SPARC_ONLY("cpu")
> ARM32_ONLY("Processor") PPC_ONLY("Processor")
> AARCH64_ONLY("Processor");
>
> and:
>
> strncpy(cpuinfo, IA32_ONLY("x86_32") AMD64_ONLY("x86_32")
> IA64_ONLY("IA64") SPARC_ONLY("sparcv9")
> ARM32_ONLY("ARM") PPC_ONLY("PPC64") AARCH64_ONLY("AArch64"),
> length);
>
> attached are two alternate patches how to fix this, either by not using the
> *_ONLY macros, or by defining an UNKOWN_ARCH_ONLY macro.
>
> Two other issues:
> - The zero builds only define ARM, not ARM32, so the clause
> should be ARM_ONLY (or at least an ARM_ONLY added).
> - The cpuinfo string seems to be wrong for AMD64_ONLY.
>
> Verified that zero builds without errors with on of these patches.
>
> Attaching both patches.
>
> Matthias
>
More information about the hotspot-dev
mailing list