diff mbox

[09/54] ram: Change byte_xfer_now type to uint64_t

Message ID 20170406130913.2232-10-quintela@redhat.com
State New
Headers show

Commit Message

Juan Quintela April 6, 2017, 1:08 p.m. UTC 
Signed-off-by: Juan Quintela <quintela@redhat.com>
---
 migration/ram.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Dr. David Alan Gilbert April 7, 2017, 9:39 a.m. UTC | #1 
* Juan Quintela (quintela@redhat.com) wrote:
> Signed-off-by: Juan Quintela <quintela@redhat.com>

OK, subject is a bit misleading since it also does _prev,
but yes:

Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>

> ---
>  migration/ram.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/migration/ram.c b/migration/ram.c
> index aeef563..d13674f 100644
> --- a/migration/ram.c
> +++ b/migration/ram.c
> @@ -158,7 +158,7 @@ struct RAMState {
>      /* last time we did a full bitmap_sync */
>      int64_t time_last_bitmap_sync;
>      /* bytes transferred at start_time */
> -    int64_t bytes_xfer_prev;
> +    uint64_t bytes_xfer_prev;
>  };
>  typedef struct RAMState RAMState;
>  
> @@ -660,7 +660,7 @@ static void migration_bitmap_sync(RAMState *rs)
>      RAMBlock *block;
>      MigrationState *s = migrate_get_current();
>      int64_t end_time;
> -    int64_t bytes_xfer_now;
> +    uint64_t bytes_xfer_now;
>  
>      rs->bitmap_sync_count++;
>  
> -- 
> 2.9.3
> 
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
Peter Xu April 11, 2017, 3:49 a.m. UTC | #2 
On Thu, Apr 06, 2017 at 03:08:28PM +0200, Juan Quintela wrote:
> Signed-off-by: Juan Quintela <quintela@redhat.com>

Reviewed-by: Peter Xu <peterx@redhat.com>

-- peterx
diff mbox

Patch

diff --git a/migration/ram.c b/migration/ram.c
index aeef563..d13674f 100644
--- a/migration/ram.c
+++ b/migration/ram.c
@@ -158,7 +158,7 @@  struct RAMState {
     /* last time we did a full bitmap_sync */
     int64_t time_last_bitmap_sync;
     /* bytes transferred at start_time */
-    int64_t bytes_xfer_prev;
+    uint64_t bytes_xfer_prev;
 };
 typedef struct RAMState RAMState;
 
@@ -660,7 +660,7 @@  static void migration_bitmap_sync(RAMState *rs)
     RAMBlock *block;
     MigrationState *s = migrate_get_current();
     int64_t end_time;
-    int64_t bytes_xfer_now;
+    uint64_t bytes_xfer_now;
 
     rs->bitmap_sync_count++;