On Tue, Apr 03, 2018 at 04:13:36PM +0200, Kevin Wolf wrote: > Am 03.04.2018 um 16:03 hat Stefan Hajnoczi geschrieben: > > Commit 4486e89c219c0d1b9bd8dfa0b1dd5b0d51ff2268 ("vl: introduce > > vm_shutdown()") added a bdrv_drain_all() call. As a side-effect of the > > drain operation the block job iterates one more time than before. The > > 185 output no longer matches and the test is failing now. > > > > It may be possible to avoid the superfluous block job iteration, but > > that type of patch is not suitable late in the QEMU 2.12 release cycle. > > > > This patch simply updates the 185 output file. The new behavior is > > correct, just not optimal, so make the test pass again. > > > > Fixes: 4486e89c219c0d1b9bd8dfa0b1dd5b0d51ff2268 ("vl: introduce > > vm_shutdown()") > > Cc: Kevin Wolf <kw...@redhat.com> > > Cc: QingFeng Hao <ha...@linux.vnet.ibm.com> > > Signed-off-by: Stefan Hajnoczi <stefa...@redhat.com> > > Do you want to update the comment in the 185 test case code that > explains which values we expect?
Thanks, I have sent v2. Stefan
Description: PGP signature