diff mbox

[3/7] No need to iterate if we already are over the limit

Message ID 187b850a0cec3aacfa75663a16575684f3170ef6.1337710679.git.quintela@redhat.com
State New
Headers show

Commit Message

Juan Quintela May 22, 2012, 6:32 p.m. UTC 
If buffers are full, don't iterate, just exit.

Signed-off-by: Juan Quintela <quintela@redhat.com>
---
 savevm.c |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

Comments

Orit Wasserman June 14, 2012, 11:03 a.m. UTC | #1 
On 05/22/2012 09:32 PM, Juan Quintela wrote:
> If buffers are full, don't iterate, just exit.
> 
> Signed-off-by: Juan Quintela <quintela@redhat.com>
> ---
>  savevm.c |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)
> 
> diff --git a/savevm.c b/savevm.c
> index 779bd22..6bc71b1 100644
> --- a/savevm.c
> +++ b/savevm.c
> @@ -1625,6 +1625,9 @@ int qemu_savevm_state_iterate(QEMUFile *f)
>          if (se->save_live_state == NULL)
>              continue;
> 
> +        if (qemu_file_rate_limit(f)) {
> +            return 0;
> +        }
>          trace_savevm_section_start();
>          /* Section type */
>          qemu_put_byte(f, QEMU_VM_SECTION_PART);

Reviewed-by: Orit Wasserman <owasserm@redhat.com>
diff mbox

Patch

diff --git a/savevm.c b/savevm.c
index 779bd22..6bc71b1 100644
--- a/savevm.c
+++ b/savevm.c
@@ -1625,6 +1625,9 @@  int qemu_savevm_state_iterate(QEMUFile *f)
         if (se->save_live_state == NULL)
             continue;

+        if (qemu_file_rate_limit(f)) {
+            return 0;
+        }
         trace_savevm_section_start();
         /* Section type */
         qemu_put_byte(f, QEMU_VM_SECTION_PART);