[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 15/19] savevm: introduce qemu_savevm_trans_{begi
From: |
Yoshiaki Tamura |
Subject: |
[Qemu-devel] Re: [PATCH 15/19] savevm: introduce qemu_savevm_trans_{begin, commit}. |
Date: |
Fri, 28 Jan 2011 23:41:38 +0900 |
2011/1/28 Paolo Bonzini <address@hidden>:
> On 01/28/2011 08:21 AM, Yoshiaki Tamura wrote:
>>
>> +int qemu_savevm_trans_begin(Monitor *mon, QEMUFile *f, int init)
>> +{
>> + SaveStateEntry *se;
>> + int skipped = 0;
>> +
>> + QTAILQ_FOREACH(se,&savevm_handlers, entry) {
>> + int len, stage, ret;
>> +
>> + if (se->save_live_state == NULL) {
>> + continue;
>> + }
>> +
>> + /* Section type */
>> + qemu_put_byte(f, QEMU_VM_SECTION_START);
>> + qemu_put_be32(f, se->section_id);
>> +
>> + /* ID string */
>> + len = strlen(se->idstr);
>> + qemu_put_byte(f, len);
>> + qemu_put_buffer(f, (uint8_t *)se->idstr, len);
>> +
>> + qemu_put_be32(f, se->instance_id);
>> + qemu_put_be32(f, se->version_id);
>> +
>> + stage = init ? QEMU_VM_SECTION_START : QEMU_VM_SECTION_PART;
>> + ret = se->save_live_state(mon, f, stage, se->opaque);
>> + if (!ret) {
>> + skipped++;
>> + }
>> + }
>> +
>> + if (qemu_file_has_error(f)) {
>> + return -EIO;
>> + }
>> +
>> + return skipped;
>> +}
>> +
>
> Right now, this is very similar to qemu_savevm_state_begin and _iterate, but
> not quite. Perhaps you could abstract it to a single function that could be
> used everywhere live handlers are used. For example,
>
> /* section says which header to write; incremental == true forces to
> pass SECTION_PART instead of SECTION_START. In code:
>
> if (section == QEMU_VM_SECTION_START) {
> stage = incremental ? QEMU_VM_SECTION_PART : QEMU_VM_SECTION_START;
> } else {
> assert(incremental);
> stage = section;
> }
>
> */
> int qemu_savevm_state_live(Monitor *mon, QEMUFile *f, int section,
> int incremental)
>
> Likewise,
>
>> + QTAILQ_FOREACH(se, &savevm_handlers, entry) {
>> + int len;
>> +
>> + if (se->save_state == NULL && se->vmsd == NULL) {
>> + continue;
>> + }
>> +
>> + /* Section type */
>> + qemu_put_byte(f, QEMU_VM_SECTION_FULL);
>> + qemu_put_be32(f, se->section_id);
>> +
>> + /* ID string */
>> + len = strlen(se->idstr);
>> + qemu_put_byte(f, len);
>> + qemu_put_buffer(f, (uint8_t *)se->idstr, len);
>> +
>> + qemu_put_be32(f, se->instance_id);
>> + qemu_put_be32(f, se->version_id);
>> +
>> + vmstate_save(f, se);
>> + }
>
> this code is straight from qemu_savevm_state_complete and should be moved
> into its own function.
Looks reasonable to avoid bit rotten. Let me see what I can do.
Thanks,
Yoshi
>
> Paolo
>
> --
> To unsubscribe from this list: send the line "unsubscribe kvm" in
> the body of a message to address@hidden
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
- [Qemu-devel] [PATCH 16/19] migration: introduce migrate_ft_trans_{put, get}_ready(), and modify migrate_fd_put_ready() when ft_mode is on., (continued)
- [Qemu-devel] [PATCH 16/19] migration: introduce migrate_ft_trans_{put, get}_ready(), and modify migrate_fd_put_ready() when ft_mode is on., Yoshiaki Tamura, 2011/01/28
- [Qemu-devel] [PATCH 18/19] Introduce -k option to enable FT migration mode (Kemari)., Yoshiaki Tamura, 2011/01/28
- [Qemu-devel] [PATCH 11/19] ioport: insert event_tap_ioport() to ioport_write()., Yoshiaki Tamura, 2011/01/28
- [Qemu-devel] [PATCH 06/19] virtio: decrement last_avail_idx with inuse before saving., Yoshiaki Tamura, 2011/01/28
- [Qemu-devel] [PATCH 09/19] Introduce event-tap., Yoshiaki Tamura, 2011/01/28
- [Qemu-devel] [PATCH 07/19] Introduce fault tolerant VM transaction QEMUFile and ft_mode., Yoshiaki Tamura, 2011/01/28
- [Qemu-devel] [PATCH 04/19] qemu-char: export socket_set_nodelay()., Yoshiaki Tamura, 2011/01/28
- [Qemu-devel] [PATCH 12/19] Insert event_tap_mmio() to cpu_physical_memory_rw() in exec.c., Yoshiaki Tamura, 2011/01/28
- [Qemu-devel] [PATCH 15/19] savevm: introduce qemu_savevm_trans_{begin, commit}., Yoshiaki Tamura, 2011/01/28
- [Qemu-devel] [PATCH 08/19] savevm: introduce util functions to control ft_trans_file from savevm layer., Yoshiaki Tamura, 2011/01/28