On Mon, 11 May 2026 at 19:24, Peter Xu <[email protected]> wrote: > > Commit dd4fe8844b changed the reporting of expected downtime behavior, so > that the value will be calculated on-demand. One side effect on the change > is QEMU will allow the calculation to happen anytime even if there's no > transfer happening for a short while. > > PeterM reported an ubsan report from clang when running migration-test with > aarch64 binary on x86_64 hosts. I can also reproduce if I run the test > concurrently so some of the src QEMU may not get chance to push any data, > causing mbps to be 0: > > ../migration/migration.c:1051:12: runtime error: -nan is outside the range of > representable values of type 'long' > > Fix it by properly handle both Inf and Nan to return INT64_MAX. > > Add a rich comment, per PeterM's suggestion. > > Link: > https://lore.kernel.org/r/CAFEAcA-MYH6C39xO0OLx4-M5pKurJpurwRsMqZe9q=w-nsh...@mail.gmail.com > Reported-by: Peter Maydell <[email protected]> > Fixes: dd4fe8844b ("migration: Calculate expected downtime on demand") > Signed-off-by: Peter Xu <[email protected]> > ---
Reviewed-by: Peter Maydell <[email protected]> thanks -- PMM
