From: Lei Li <lilei@linux.vnet.ibm.com>
To: Anthony Liguori <aliguori@us.ibm.com>
Cc: lagarcia@br.ibm.com, qemu-devel@nongnu.org, quintela@redhat.com
Subject: Re: [Qemu-devel] [PATCH 2/7] savevm: export qemu_save_device_state()
Date: Tue, 18 Jun 2013 14:05:46 +0800 [thread overview]
Message-ID: <51BFF8BA.90001@linux.vnet.ibm.com> (raw)
In-Reply-To: <87bo76hwly.fsf@codemonkey.ws>
On 06/17/2013 12:02 AM, Anthony Liguori wrote:
> Lei Li <lilei@linux.vnet.ibm.com> writes:
>
>> Signed-off-by: Lei Li <lilei@linux.vnet.ibm.com>
>> ---
>> include/sysemu/sysemu.h | 1 +
>> savevm.c | 2 +-
>> 2 files changed, 2 insertions(+), 1 deletions(-)
>>
>> diff --git a/include/sysemu/sysemu.h b/include/sysemu/sysemu.h
>> index 2fb71af..5b90027 100644
>> --- a/include/sysemu/sysemu.h
>> +++ b/include/sysemu/sysemu.h
>> @@ -80,6 +80,7 @@ int qemu_savevm_state_iterate(QEMUFile *f);
>> void qemu_savevm_state_complete(QEMUFile *f);
>> void qemu_savevm_state_cancel(void);
>> uint64_t qemu_savevm_state_pending(QEMUFile *f, uint64_t max_size);
>> +int qemu_save_device_state(QEMUFile *f);
>> int qemu_loadvm_state(QEMUFile *f);
> Please add comments describing what this function does. I know they're
> absent for the rest of the functions but we have to start somewhere.
OK, will do.
>
> Regards,
>
> Anthony Liguori
>
>>
>> /* SLIRP */
>> diff --git a/savevm.c b/savevm.c
>> index 2ce439f..e3ad5da 100644
>> --- a/savevm.c
>> +++ b/savevm.c
>> @@ -2006,7 +2006,7 @@ static int qemu_savevm_state(QEMUFile *f)
>> return ret;
>> }
>>
>> -static int qemu_save_device_state(QEMUFile *f)
>> +int qemu_save_device_state(QEMUFile *f)
>> {
>> SaveStateEntry *se;
>>
>> --
>> 1.7.7.6
--
Lei
next prev parent reply other threads:[~2013-06-18 6:06 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-06-16 15:37 [Qemu-devel] [PATCH 0/7 RFC] Localhost live migration Lei Li
2013-06-16 15:37 ` [Qemu-devel] [PATCH 1/7] migration: export MIG_STATE_xxx flags Lei Li
2013-06-16 16:01 ` Anthony Liguori
2013-06-18 6:03 ` Lei Li
2013-06-16 15:37 ` [Qemu-devel] [PATCH 2/7] savevm: export qemu_save_device_state() Lei Li
2013-06-16 16:02 ` Anthony Liguori
2013-06-18 6:05 ` Lei Li [this message]
2013-06-16 15:37 ` [Qemu-devel] [PATCH 3/7] arch_init: add ram_madvise_free() Lei Li
2013-06-16 16:04 ` Anthony Liguori
2013-06-18 6:11 ` Lei Li
2013-08-02 19:34 ` Michael R. Hines
2013-06-16 15:37 ` [Qemu-devel] [PATCH 4/7] migration-local: implementation of outgoing part Lei Li
2013-06-16 16:07 ` Anthony Liguori
2013-06-18 6:25 ` Lei Li
2013-06-16 15:37 ` [Qemu-devel] [PATCH 5/7] migration-local: implementation of incoming part Lei Li
2013-06-16 15:37 ` [Qemu-devel] [PATCH 6/7] migration-local: add option to command line for -incoming-local Lei Li
2013-06-16 15:37 ` [Qemu-devel] [PATCH 7/7] hmp: add hmp_localhost_migration interface Lei Li
2013-06-16 16:01 ` [Qemu-devel] [PATCH 0/7 RFC] Localhost live migration Anthony Liguori
2013-06-18 6:02 ` Lei Li
2013-06-17 12:57 ` Stefan Hajnoczi
2013-06-18 6:01 ` Lei Li
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=51BFF8BA.90001@linux.vnet.ibm.com \
--to=lilei@linux.vnet.ibm.com \
--cc=aliguori@us.ibm.com \
--cc=lagarcia@br.ibm.com \
--cc=qemu-devel@nongnu.org \
--cc=quintela@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.