diff mbox

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

Message ID 5a5d97ef8180b2c1f2be80d50ed49c9934de39a2.1340371865.git.quintela@redhat.com
State New
Headers show

Commit Message

Juan Quintela June 22, 2012, 1:46 p.m. UTC 
If buffers are full, don't iterate, just exit.

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

Comments

Orit Wasserman June 25, 2012, 6:25 a.m. UTC | #1 
On 06/22/2012 04:46 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 file changed, 3 insertions(+)
> 
> diff --git a/savevm.c b/savevm.c
> index 40320be..9101bfb 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 40320be..9101bfb 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);