diff options
| author | Avishay Traeger <AVISHAY@il.ibm.com> | 2011-04-06 10:45:36 +0300 |
|---|---|---|
| committer | Kevin Wolf <kwolf@redhat.com> | 2011-04-07 13:51:48 +0200 |
| commit | 155eb9aa09249874b4ff49e94c58595ad82d3abb (patch) | |
| tree | d7a2a659329775ecdae2f4d3f4116d5e174a3c7e /block-migration.c | |
| parent | 7d905f716bea633f2836e1d661387983aacdc6d6 (diff) | |
| download | qemu-155eb9aa09249874b4ff49e94c58595ad82d3abb.zip qemu-155eb9aa09249874b4ff49e94c58595ad82d3abb.tar.gz qemu-155eb9aa09249874b4ff49e94c58595ad82d3abb.tar.bz2 | |
Fix integer overflow in block migration bandwidth calculation
block_mig_state.reads is an int, and multiplying by BLOCK_SIZE yielded a
negative number, resulting in a negative bandwidth (running on a 32-bit
machine). Change order to avoid.
Signed-off-by: Avishay Traeger <avishay@il.ibm.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'block-migration.c')
| -rw-r--r-- | block-migration.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/block-migration.c b/block-migration.c index 8218bac..576e55a 100644 --- a/block-migration.c +++ b/block-migration.c @@ -140,7 +140,7 @@ static inline void add_avg_read_time(int64_t time) static inline long double compute_read_bwidth(void) { assert(block_mig_state.total_time != 0); - return (block_mig_state.reads * BLOCK_SIZE)/ block_mig_state.total_time; + return (block_mig_state.reads / block_mig_state.total_time) * BLOCK_SIZE; } static int bmds_aio_inflight(BlkMigDevState *bmds, int64_t sector) |