[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH] ppc kvm: fix to compile
From: |
Alexander Graf |
Subject: |
Re: [Qemu-ppc] [PATCH] ppc kvm: fix to compile |
Date: |
Wed, 17 Jul 2013 01:12:10 +0200 |
On 16.07.2013, at 12:00, Alexey Kardashevskiy wrote:
> Signed-off-by: Alexey Kardashevskiy <address@hidden>
Why? sPAPRTCE is a typedef struct that's defined in include/hw/ppc/spapr.h
which is included from target-ppc/kvm.c. So it should work just fine.
Alex
> ---
> target-ppc/kvm.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/target-ppc/kvm.c b/target-ppc/kvm.c
> index 61f2737..606bdb9 100644
> --- a/target-ppc/kvm.c
> +++ b/target-ppc/kvm.c
> @@ -1628,7 +1628,7 @@ void *kvmppc_create_spapr_tce(uint32_t liobn, uint32_t
> window_size, int *pfd)
> return NULL;
> }
>
> - len = (window_size / SPAPR_TCE_PAGE_SIZE) * sizeof(sPAPRTCE);
> + len = (window_size / SPAPR_TCE_PAGE_SIZE) * sizeof(uint64_t);
> /* FIXME: round this up to page size */
>
> table = mmap(NULL, len, PROT_READ|PROT_WRITE, MAP_SHARED, fd, 0);
> @@ -1651,7 +1651,7 @@ int kvmppc_remove_spapr_tce(void *table, int fd,
> uint32_t window_size)
> return -1;
> }
>
> - len = (window_size / SPAPR_TCE_PAGE_SIZE)*sizeof(sPAPRTCE);
> + len = (window_size / SPAPR_TCE_PAGE_SIZE)*sizeof(uint64_t);
> if ((munmap(table, len) < 0) ||
> (close(fd) < 0)) {
> fprintf(stderr, "KVM: Unexpected error removing TCE table: %s",
> --
> 1.8.3.2
>
[Qemu-ppc] [PATCH 06/11] pseries: rework PAPR virtual SCSI, Anthony Liguori, 2013/07/15
[Qemu-ppc] [PATCH 01/11] target-ppc: Convert ppc cpu savevm to VMStateDescription, Anthony Liguori, 2013/07/15
[Qemu-ppc] [PATCH 07/11] pseries: savevm support for PAPR virtual SCSI, Anthony Liguori, 2013/07/15
[Qemu-ppc] [PATCH 02/11] pseries: savevm support for VIO devices, Anthony Liguori, 2013/07/15
[Qemu-ppc] [PATCH 03/11] pseries: savevm support for PAPR VIO logical lan, Anthony Liguori, 2013/07/15
[Qemu-ppc] [PATCH 11/11] xics: rename types to be sane and follow coding style, Anthony Liguori, 2013/07/15