From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:49474) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fIFwT-0006gU-W4 for qemu-devel@nongnu.org; Mon, 14 May 2018 12:03:18 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1fIFwT-0001FR-1M for qemu-devel@nongnu.org; Mon, 14 May 2018 12:03:18 -0400 Received: from mail-oi0-x242.google.com ([2607:f8b0:4003:c06::242]:35109) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1fIFwS-0001FC-Sq for qemu-devel@nongnu.org; Mon, 14 May 2018 12:03:16 -0400 Received: by mail-oi0-x242.google.com with SMTP id a6-v6so11176192oia.2 for ; Mon, 14 May 2018 09:03:16 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: References: <20180512092824.13848-1-stefanha@redhat.com> From: Peter Maydell Date: Mon, 14 May 2018 17:02:55 +0100 Message-ID: Content-Type: text/plain; charset="UTF-8" Subject: Re: [Qemu-devel] [PULL 0/8] Block patches List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Hajnoczi Cc: QEMU Developers , Qemu-block On 14 May 2018 at 14:15, Peter Maydell wrote: > On 12 May 2018 at 10:28, Stefan Hajnoczi wrote: >> The following changes since commit e5cd695266c5709308aa95b1baae499e4b5d4544: >> >> Merge remote-tracking branch 'remotes/cody/tags/block-pull-request' into staging (2018-05-08 17:05:58 +0100) >> >> are available in the Git repository at: >> >> git://github.com/stefanha/qemu.git tags/block-pull-request >> >> for you to fetch changes up to 31be8a2a97ecba7d31a82932286489cac318e9e9: >> >> block/file-posix: add x-check-page-cache=on|off option (2018-05-11 16:43:05 +0100) >> >> ---------------------------------------------------------------- >> Block pull request >> >> * Support -drive cache.direct=off live migration for POSIX files >> >> ---------------------------------------------------------------- > > With this I get test failures for x86-64 Linux host (debug enabled): > > MALLOC_PERTURB_=${MALLOC_PERTURB_:-$(( ${RANDOM:-0} % 255 + 1))} > gtester -k --verbose -m=quick tests/test-replication > TEST: tests/test-replication... (pid=15016) > /replication/primary/read: OK > /replication/primary/write: OK > /replication/primary/start: OK > /replication/primary/stop: OK > /replication/primary/do_checkpoint: OK > /replication/primary/get_error_all: OK > /replication/secondary/read: ** > ERROR:/home/petmay01/linaro/qemu-for-merges/tests/test-replication.c:119:test_blk_write: > assertion failed: (async_ret == 0) > FAIL > GTester: last random seed: R02Sf99d81651223b2fcdd0d7d601245eba3 > (pid=15131) > /replication/secondary/write: OK > /replication/secondary/start: ** > ERROR:/home/petmay01/linaro/qemu-for-merges/tests/test-replication.c:119:test_blk_write: > assertion failed: (async_ret == 0) > FAIL Sorry, this is a false alarm -- this test fails in this obscure way if the system has run out of disk space and the write returns ENOSPC. Ideally we should fix the test to print the errno. In the meantime I've deleted some large log files and will retest the pull :-) thanks -- PMM