[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 6/6] hw/arm/virt: Implement kvm-steal-time
From: |
Peter Maydell |
Subject: |
Re: [PATCH v4 6/6] hw/arm/virt: Implement kvm-steal-time |
Date: |
Thu, 8 Oct 2020 21:39:59 +0100 |
On Thu, 1 Oct 2020 at 07:17, Andrew Jones <drjones@redhat.com> wrote:
>
> We add the kvm-steal-time CPU property and implement it for machvirt.
> A tiny bit of refactoring was also done to allow pmu and pvtime to
> use the same vcpu device helper functions.
> + if (pvtime_size > pvtime_reg_size) {
> + error_report("pvtime requires a %ld byte memory region for "
> + "%d CPUs, but only %ld has been reserved",
> + pvtime_size, max_cpus, pvtime_reg_size);
> + exit(1);
> + }
This turns out not to compile on Windows:
../../hw/arm/virt.c:1693:30: error: format '%ld' expects argument of
type 'long int', but argument 2 has type 'hwaddr {aka long long
unsigned int}' [-Werror=format=]
error_report("pvtime requires a %ld byte memory region for "
^
I'm going to squash this fix into the pullreq:
--- a/hw/arm/virt.c
+++ b/hw/arm/virt.c
@@ -1690,8 +1690,9 @@ static void virt_cpu_post_init(VirtMachineState
*vms, int max_cpus,
pvtime_size = REAL_HOST_PAGE_ALIGN(pvtime_size);
if (pvtime_size > pvtime_reg_size) {
- error_report("pvtime requires a %ld byte memory region for "
- "%d CPUs, but only %ld has been reserved",
+ error_report("pvtime requires a %" HWADDR_PRId
+ " byte memory region for %d CPUs,"
+ " but only %" HWADDR_PRId " has been reserved",
pvtime_size, max_cpus, pvtime_reg_size);
exit(1);
}
thanks
-- PMM
- [PATCH v4 0/6] hw/arm/virt: Introduce kvm-steal-time, Andrew Jones, 2020/10/01
- [PATCH v4 1/6] linux headers: sync to 5.9-rc7, Andrew Jones, 2020/10/01
- [PATCH v4 2/6] target/arm/kvm: Make uncalled stubs explicitly unreachable, Andrew Jones, 2020/10/01
- [PATCH v4 3/6] hw/arm/virt: Move post cpu realize check into its own function, Andrew Jones, 2020/10/01
- [PATCH v4 4/6] hw/arm/virt: Move kvm pmu setup to virt_cpu_post_init, Andrew Jones, 2020/10/01
- [PATCH v4 5/6] tests/qtest: Restore aarch64 arm-cpu-features test, Andrew Jones, 2020/10/01
- [PATCH v4 6/6] hw/arm/virt: Implement kvm-steal-time, Andrew Jones, 2020/10/01
- Re: [PATCH v4 6/6] hw/arm/virt: Implement kvm-steal-time,
Peter Maydell <=
- Re: [PATCH v4 0/6] hw/arm/virt: Introduce kvm-steal-time, Peter Maydell, 2020/10/08