[PATCH 1/3] migration: Make threshold_size an uint64_t

peterx@redhat.com posted 3 patches 10 months, 2 weeks ago
Maintainers: Peter Xu <peterx@redhat.com>, Fabiano Rosas <farosas@suse.de>, John Snow <jsnow@redhat.com>, Cleber Rosa <crosa@redhat.com>
[PATCH 1/3] migration: Make threshold_size an uint64_t
Posted by peterx@redhat.com 10 months, 2 weeks ago
From: Peter Xu <peterx@redhat.com>

It's always used to compare against another uint64_t.  Make it always clear
that it's never a negative.

Signed-off-by: Peter Xu <peterx@redhat.com>
---
 migration/migration.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/migration/migration.h b/migration/migration.h
index 17972dac34..a589ae8650 100644
--- a/migration/migration.h
+++ b/migration/migration.h
@@ -296,7 +296,7 @@ struct MigrationState {
      * this threshold; it's calculated from the requested downtime and
      * measured bandwidth, or avail-switchover-bandwidth if specified.
      */
-    int64_t threshold_size;
+    uint64_t threshold_size;
 
     /* params from 'migrate-set-parameters' */
     MigrationParameters parameters;
-- 
2.43.0
Re: [PATCH 1/3] migration: Make threshold_size an uint64_t
Posted by Fabiano Rosas 10 months, 1 week ago
peterx@redhat.com writes:

> From: Peter Xu <peterx@redhat.com>
>
> It's always used to compare against another uint64_t.  Make it always clear
> that it's never a negative.
>
> Signed-off-by: Peter Xu <peterx@redhat.com>

Reviewed-by: Fabiano Rosas <farosas@suse.de>
Re: [PATCH 1/3] migration: Make threshold_size an uint64_t
Posted by Philippe Mathieu-Daudé 10 months, 2 weeks ago
On 17/1/24 08:58, peterx@redhat.com wrote:
> From: Peter Xu <peterx@redhat.com>
> 
> It's always used to compare against another uint64_t.  Make it always clear
> that it's never a negative.
> 
> Signed-off-by: Peter Xu <peterx@redhat.com>
> ---
>   migration/migration.h | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>