[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 4/7] migration: Do chunk page in postcopy_each_ram_send_disca
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [PATCH 4/7] migration: Do chunk page in postcopy_each_ram_send_discard() |
Date: |
Wed, 8 Dec 2021 17:39:42 +0000 |
User-agent: |
Mutt/2.1.3 (2021-09-10) |
* Peter Xu (peterx@redhat.com) wrote:
> Right now we loop ramblocks for twice, the 1st time chunk the dirty bits with
> huge page information; the 2nd time we send the discard ranges. That's not
> necessary - we can do them in a single loop.
>
> Signed-off-by: Peter Xu <peterx@redhat.com>
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
> ---
> migration/ram.c | 20 ++++++++++----------
> 1 file changed, 10 insertions(+), 10 deletions(-)
>
> diff --git a/migration/ram.c b/migration/ram.c
> index e3876181ab..ecc744d54d 100644
> --- a/migration/ram.c
> +++ b/migration/ram.c
> @@ -2473,6 +2473,8 @@ static int postcopy_send_discard_bm_ram(MigrationState
> *ms, RAMBlock *block)
> return 0;
> }
>
> +static void postcopy_chunk_hostpages_pass(MigrationState *ms, RAMBlock
> *block);
> +
> /**
> * postcopy_each_ram_send_discard: discard all RAMBlocks
> *
> @@ -2494,6 +2496,14 @@ static int
> postcopy_each_ram_send_discard(MigrationState *ms)
> RAMBLOCK_FOREACH_NOT_IGNORED(block) {
> postcopy_discard_send_init(ms, block->idstr);
>
> + /*
> + * Deal with TPS != HPS and huge pages. It discard any partially
> sent
> + * host-page size chunks, mark any partially dirty host-page size
> + * chunks as all dirty. In this case the host-page is the host-page
> + * for the particular RAMBlock, i.e. it might be a huge page.
> + */
> + postcopy_chunk_hostpages_pass(ms, block);
> +
> /*
> * Postcopy sends chunks of bitmap over the wire, but it
> * just needs indexes at this point, avoids it having
> @@ -2594,7 +2604,6 @@ static void
> postcopy_chunk_hostpages_pass(MigrationState *ms, RAMBlock *block)
> int ram_postcopy_send_discard_bitmap(MigrationState *ms)
> {
> RAMState *rs = ram_state;
> - RAMBlock *block;
>
> RCU_READ_LOCK_GUARD();
>
> @@ -2606,15 +2615,6 @@ int ram_postcopy_send_discard_bitmap(MigrationState
> *ms)
> rs->last_sent_block = NULL;
> rs->last_page = 0;
>
> - RAMBLOCK_FOREACH_NOT_IGNORED(block) {
> - /*
> - * Deal with TPS != HPS and huge pages. It discard any partially
> sent
> - * host-page size chunks, mark any partially dirty host-page size
> - * chunks as all dirty. In this case the host-page is the host-page
> - * for the particular RAMBlock, i.e. it might be a huge page.
> - */
> - postcopy_chunk_hostpages_pass(ms, block);
> - }
> trace_ram_postcopy_send_discard_bitmap();
>
> return postcopy_each_ram_send_discard(ms);
> --
> 2.32.0
>
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
- [PATCH 0/7] migration: Postcopy cleanup on ram disgard, Peter Xu, 2021/12/07
- [PATCH 2/7] migration: Don't return for postcopy_chunk_hostpages(), Peter Xu, 2021/12/07
- [PATCH 1/7] migration: Drop dead code of ram_debug_dump_bitmap(), Peter Xu, 2021/12/07
- [PATCH 3/7] migration: Drop postcopy_chunk_hostpages(), Peter Xu, 2021/12/07
- [PATCH 4/7] migration: Do chunk page in postcopy_each_ram_send_discard(), Peter Xu, 2021/12/07
- Re: [PATCH 4/7] migration: Do chunk page in postcopy_each_ram_send_discard(),
Dr. David Alan Gilbert <=
- [PATCH 5/7] migration: Drop return code for disgard ram process, Peter Xu, 2021/12/07
- [PATCH 6/7] migration: Dump sub-cmd name in loadvm_process_command tp, Peter Xu, 2021/12/07
- [PATCH 7/7] migration: Finer grained tracepoints for POSTCOPY_LISTEN, Peter Xu, 2021/12/07