[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC 01/29] migration: fix incorrect postcopy recved_bi
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [Qemu-devel] [RFC 01/29] migration: fix incorrect postcopy recved_bitmap |
Date: |
Mon, 31 Jul 2017 17:34:14 +0100 |
User-agent: |
Mutt/1.8.3 (2017-05-23) |
* Peter Xu (address@hidden) wrote:
> The bitmap setup during postcopy is incorrectly when the pgaes are huge
> pages. Fix it.
>
> Signed-off-by: Peter Xu <address@hidden>
> ---
> migration/postcopy-ram.c | 2 +-
> migration/ram.c | 8 ++++++++
> migration/ram.h | 2 ++
> 3 files changed, 11 insertions(+), 1 deletion(-)
>
> diff --git a/migration/postcopy-ram.c b/migration/postcopy-ram.c
> index 276ce12..952b73a 100644
> --- a/migration/postcopy-ram.c
> +++ b/migration/postcopy-ram.c
> @@ -578,7 +578,7 @@ static int qemu_ufd_copy_ioctl(int userfault_fd, void
> *host_addr,
> ret = ioctl(userfault_fd, UFFDIO_ZEROPAGE, &zero_struct);
> }
> if (!ret) {
> - ramblock_recv_bitmap_set(host_addr, rb);
> + ramblock_recv_bitmap_set_range(rb, host_addr, pagesize /
> getpagesize());
isn't that pagesize / qemu_target_page_size() ?
Other than that it looks OK.
> }
> return ret;
> }
> diff --git a/migration/ram.c b/migration/ram.c
> index 107ee9d..c93973c 100644
> --- a/migration/ram.c
> +++ b/migration/ram.c
> @@ -168,6 +168,14 @@ void ramblock_recv_bitmap_set(void *host_addr, RAMBlock
> *rb)
> set_bit_atomic(ramblock_recv_bitmap_offset(host_addr, rb),
> rb->receivedmap);
> }
>
> +void ramblock_recv_bitmap_set_range(RAMBlock *rb, void *host_addr,
> + size_t len)
> +{
> + bitmap_set(rb->receivedmap,
> + ramblock_recv_bitmap_offset(host_addr, rb),
> + len);
> +}
> +
> void ramblock_recv_bitmap_clear(void *host_addr, RAMBlock *rb)
> {
> clear_bit(ramblock_recv_bitmap_offset(host_addr, rb), rb->receivedmap);
> diff --git a/migration/ram.h b/migration/ram.h
> index b711552..84e8623 100644
> --- a/migration/ram.h
> +++ b/migration/ram.h
> @@ -55,6 +55,8 @@ void ram_handle_compressed(void *host, uint8_t ch, uint64_t
> size);
>
> int ramblock_recv_bitmap_test(void *host_addr, RAMBlock *rb);
> void ramblock_recv_bitmap_set(void *host_addr, RAMBlock *rb);
> +void ramblock_recv_bitmap_set_range(RAMBlock *rb, void *host_addr,
> + size_t len);
> void ramblock_recv_bitmap_clear(void *host_addr, RAMBlock *rb);
>
> #endif
> --
> 2.7.4
>
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK
- [Qemu-devel] [RFC 00/29] Migration: postcopy failure recovery, Peter Xu, 2017/07/28
- [Qemu-devel] [RFC 01/29] migration: fix incorrect postcopy recved_bitmap, Peter Xu, 2017/07/28
- Re: [Qemu-devel] [RFC 01/29] migration: fix incorrect postcopy recved_bitmap,
Dr. David Alan Gilbert <=
- [Qemu-devel] [RFC 02/29] migration: fix comment disorder in RAMState, Peter Xu, 2017/07/28
- [Qemu-devel] [RFC 03/29] io: fix qio_channel_socket_accept err handling, Peter Xu, 2017/07/28
- [Qemu-devel] [RFC 04/29] bitmap: introduce bitmap_invert(), Peter Xu, 2017/07/28
- [Qemu-devel] [RFC 05/29] bitmap: introduce bitmap_count_one(), Peter Xu, 2017/07/28