[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v10 4/7] migration: API to clear bits of guest f
From: |
Peter Xu |
Subject: |
Re: [Qemu-devel] [PATCH v10 4/7] migration: API to clear bits of guest free pages from the dirty bitmap |
Date: |
Mon, 3 Dec 2018 13:10:25 +0800 |
User-agent: |
Mutt/1.10.1 (2018-07-13) |
On Mon, Dec 03, 2018 at 10:18:28AM +0800, Wei Wang wrote:
> This patch adds an API to clear bits corresponding to guest free pages
> from the dirty bitmap. Spilt the free page block if it crosses the QEMU
> RAMBlock boundary.
>
> Signed-off-by: Wei Wang <address@hidden>
> CC: Dr. David Alan Gilbert <address@hidden>
> CC: Juan Quintela <address@hidden>
> CC: Michael S. Tsirkin <address@hidden>
> CC: Peter Xu <address@hidden>
> ---
[...]
> +void qemu_guest_free_page_hint(void *addr, size_t len)
> +{
> + RAMBlock *block;
> + ram_addr_t offset;
> + size_t used_len, start, npages;
> + MigrationState *s = migrate_get_current();
> +
> + /* This function is currently expected to be used during live migration
> */
> + if (!migration_is_setup_or_active(s->state)) {
> + return;
> + }
> +
> + for (; len > 0; len -= used_len, addr += used_len) {
> + block = qemu_ram_block_from_host(addr, false, &offset);
> + if (unlikely(!block || offset > block->used_length)) {
Maybe >=? My fault if it is...
Reviewed-by: Peter Xu <address@hidden>
Thanks,
--
Peter Xu
- [Qemu-devel] [PATCH v10 0/7] virtio-balloon: free page hint support, Wei Wang, 2018/12/02
- [Qemu-devel] [PATCH v10 2/7] bitmap: bitmap_count_one_with_offset, Wei Wang, 2018/12/02
- [Qemu-devel] [PATCH v10 1/7] bitmap: fix bitmap_count_one, Wei Wang, 2018/12/02
- [Qemu-devel] [PATCH v10 3/7] migration: use bitmap_mutex in migration_bitmap_clear_dirty, Wei Wang, 2018/12/02
- [Qemu-devel] [PATCH v10 4/7] migration: API to clear bits of guest free pages from the dirty bitmap, Wei Wang, 2018/12/02
- Re: [Qemu-devel] [PATCH v10 4/7] migration: API to clear bits of guest free pages from the dirty bitmap,
Peter Xu <=
- [Qemu-devel] [PATCH v10 6/7] migration/ram.c: add a function to disable the bulk stage, Wei Wang, 2018/12/02
- [Qemu-devel] [PATCH v10 5/7] migration/ram.c: add a notifier chain for precopy, Wei Wang, 2018/12/02
- [Qemu-devel] [PATCH v10 7/7] virtio-balloon: VIRTIO_BALLOON_F_FREE_PAGE_HINT, Wei Wang, 2018/12/02