On 2/13/23 14:27, Warner Losh wrote:
> + case HW_NCPU:
> + if (oldlen) {
> + (*(int32_t *)holdp) = tswap32(bsd_get_ncpu());
> + }
> + holdlen = sizeof(int32_t);
> + ret = 0;
> + goto out;
Anything using SYSCTL_INT should use abi_int.
> +#if defined(TARGET_ARM)
> + case HW_FLOATINGPT:
> + if (oldlen) {
> + ARMCPU *cpu = env_archcpu(env);
> + *(abi_int *)holdp = cpu_isar_feature(aa32_vfp, cpu);
> + }
> + holdlen = sizeof(int32_t);
abi_int for consistency.
Otherwise,
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Makes sense.. Thanks!
Warner