Patchwork [03/14] use XFER_LIMIT_RATIO consistently

login
register
mail settings
Submitter Juan Quintela
Date Jan. 15, 2013, 11:18 a.m.
Message ID <1358248702-14278-4-git-send-email-quintela@redhat.com>
Download mbox | patch
Permalink /patch/212137/
State New
Headers show

Comments

Juan Quintela - Jan. 15, 2013, 11:18 a.m.
From: Paolo Bonzini <pbonzini@redhat.com>

Previous patch missed this case

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Juan Quintela <quintela@redhat.com>
---
 migration.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Eric Blake - Jan. 15, 2013, 7:45 p.m.
On 01/15/2013 04:18 AM, Juan Quintela wrote:
> From: Paolo Bonzini <pbonzini@redhat.com>
> 
> Previous patch missed this case

Might be worth mentioning _which_ previous patch, now that there is
quite a gap in the git history.

> 
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> Signed-off-by: Juan Quintela <quintela@redhat.com>
> ---
>  migration.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/migration.c b/migration.c
> index c69e864..d6ec3e8 100644
> --- a/migration.c
> +++ b/migration.c
> @@ -650,7 +650,7 @@ static int64_t buffered_set_rate_limit(void *opaque, int64_t new_rate)
>          new_rate = SIZE_MAX;
>      }
> 
> -    s->xfer_limit = new_rate / 10;
> +    s->xfer_limit = new_rate / XFER_LIMIT_RATIO;
> 
>  out:
>      return s->xfer_limit;
>

Patch

diff --git a/migration.c b/migration.c
index c69e864..d6ec3e8 100644
--- a/migration.c
+++ b/migration.c
@@ -650,7 +650,7 @@  static int64_t buffered_set_rate_limit(void *opaque, int64_t new_rate)
         new_rate = SIZE_MAX;
     }

-    s->xfer_limit = new_rate / 10;
+    s->xfer_limit = new_rate / XFER_LIMIT_RATIO;

 out:
     return s->xfer_limit;