[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] migration/postcopy: fix document of postcopy_se
From: |
Wei Yang |
Subject: |
Re: [Qemu-devel] [PATCH] migration/postcopy: fix document of postcopy_send_discard_bm_ram() |
Date: |
Fri, 12 Jul 2019 10:29:03 +0800 |
User-agent: |
Mutt/1.9.4 (2018-02-28) |
On Thu, Jul 11, 2019 at 10:34:27AM +0100, Dr. David Alan Gilbert wrote:
>* Wei Yang (address@hidden) wrote:
>> Commit 6b6712efccd3 ('ram: Split dirty bitmap by RAMBlock') changes the
>> parameter of postcopy_send_discard_bm_ram(), while left the document
>> part untouched.
>>
>> This patch correct the document and fix one typo by hand.
>>
>> Signed-off-by: Wei Yang <address@hidden>
>> ---
>> migration/ram.c | 5 ++---
>> 1 file changed, 2 insertions(+), 3 deletions(-)
>>
>> diff --git a/migration/ram.c b/migration/ram.c
>> index 246efe6939..410e0f89fe 100644
>> --- a/migration/ram.c
>> +++ b/migration/ram.c
>> @@ -2763,8 +2763,7 @@ void
>> ram_postcopy_migrated_memory_release(MigrationState *ms)
>> *
>> * @ms: current migration state
>> * @pds: state for postcopy
>> - * @start: RAMBlock starting page
>> - * @length: RAMBlock size
>> + * @block: RAMBlock to discard
>> */
>> static int postcopy_send_discard_bm_ram(MigrationState *ms,
>> PostcopyDiscardState *pds,
>> @@ -2961,7 +2960,7 @@ static void
>> postcopy_chunk_hostpages_pass(MigrationState *ms, bool unsent_pass,
>> }
>>
>> /**
>> - * postcopy_chuck_hostpages: discrad any partially sent host page
>> + * postcopy_chuck_hostpages: discard any partially sent host page
>
>While we're here we should probably fix the name of the function as
>well! s/chuck/chunk/
>
Ah, didn't notice this :)
Do you like me to send v2 to fix this?
>Dave
>
>> *
>> * Utility for the outgoing postcopy code.
>> *
>> --
>> 2.19.1
>>
>--
>Dr. David Alan Gilbert / address@hidden / Manchester, UK
--
Wei Yang
Help you, Help me