[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 04/18] savevm: set right return value for qemu_f
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 04/18] savevm: set right return value for qemu_file_rate_limit |
Date: |
Fri, 23 Aug 2013 11:14:26 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130805 Thunderbird/17.0.8 |
Il 23/08/2013 11:11, Lei Li ha scritto:
>
> In qemu_savevm_state_iterate(), if the return of ram_save_iterate < 0,
> it will set error in QEMUFile by qemu_file_set_error(f, ret), the error
> reported by qemu_file_get_error() in the QEMUFile is got from
> qemu_file_set_error() by reading the f->last_error, right?
>
> And now as qemu_file_rate_limit() never return negative value, what's the
> meaning for the check: if (qemu_file_rate_limit(f) < 0) in
> ram_save_iterate()?
I only see a "while ((ret = qemu_file_rate_limit(f)) == 0)", no
less-than-zero check.
Are we looking at the same code? :)
Paolo
- [Qemu-devel] [PATCH 0/18 RFC v3] Localhost migration, Lei Li, 2013/08/21
- [Qemu-devel] [PATCH 01/18] migration: export MIG_STATE_xxx flags, Lei Li, 2013/08/21
- [Qemu-devel] [PATCH 02/18] savevm: export qemu_save_device_state(), Lei Li, 2013/08/21
- [Qemu-devel] [PATCH 03/18] rename is_active to is_block_active, Lei Li, 2013/08/21
- [Qemu-devel] [PATCH 04/18] savevm: set right return value for qemu_file_rate_limit, Lei Li, 2013/08/21
- Re: [Qemu-devel] [PATCH 04/18] savevm: set right return value for qemu_file_rate_limit, Paolo Bonzini, 2013/08/21
- Re: [Qemu-devel] [PATCH 04/18] savevm: set right return value for qemu_file_rate_limit, Lei Li, 2013/08/22
- Re: [Qemu-devel] [PATCH 04/18] savevm: set right return value for qemu_file_rate_limit, Paolo Bonzini, 2013/08/23
- Re: [Qemu-devel] [PATCH 04/18] savevm: set right return value for qemu_file_rate_limit, Lei Li, 2013/08/23
- Re: [Qemu-devel] [PATCH 04/18] savevm: set right return value for qemu_file_rate_limit,
Paolo Bonzini <=
- Re: [Qemu-devel] [PATCH 04/18] savevm: set right return value for qemu_file_rate_limit, Lei Li, 2013/08/23
- Re: [Qemu-devel] [PATCH 04/18] savevm: set right return value for qemu_file_rate_limit, Paolo Bonzini, 2013/08/23
- Re: [Qemu-devel] [PATCH 04/18] savevm: set right return value for qemu_file_rate_limit, Lei Li, 2013/08/23
[Qemu-devel] [PATCH 05/18] savevm: add comments for qemu_file_get_error(), Lei Li, 2013/08/21
[Qemu-devel] [PATCH 06/18] bugfix: wrong error set by ram_control_load_hook(), Lei Li, 2013/08/21