All of lore.kernel.org
 help / color / mirror / Atom feed
From: Eric Blake <eblake@redhat.com>
To: Alberto Garcia <berto@igalia.com>, qemu-devel@nongnu.org
Cc: qemu-block@nongnu.org
Subject: Re: [Qemu-devel] iotest 223 is flaky
Date: Tue, 5 Mar 2019 12:24:02 -0600	[thread overview]
Message-ID: <f2fab5f3-9321-9dac-450c-429caf9e55fb@redhat.com> (raw)
In-Reply-To: <w51y35tb8ul.fsf@maestria.local.igalia.com>

On 3/5/19 10:10 AM, Alberto Garcia wrote:
> I can make it fail by running it a few times:
> 
> 223 1s ... - output mismatch (see 223.out.bad)
> --- tests/qemu-iotests/223.out       2019-02-08 17:25:15.584387100 +0200
> +++ tests/qemu-iotests/223.out.bad     2019-03-05 18:05:16.855223976 +0200
> @@ -92,9 +92,10 @@
>   
>   === Use qemu-nbd as server ===
>   
> -[{ "start": 0, "length": 65536, "depth": 0, "zero": false, "data": false},
> -{ "start": 65536, "length": 2031616, "depth": 0, "zero": false, "data": true},
> -{ "start": 2097152, "length": 2097152, "depth": 0, "zero": false, "data": false}]
> +qemu-nbd: Failed to blk_new_open 'tests/qemu-iotests/scratch/t.qcow2': Failed to get shared "write" lock
> +Is another process using the image [tests/qemu-iotests/scratch/t.qcow2]?

I haven't been able to reproduce it yet, but see what might be the problem:

_send_qemu_cmd $QEMU_HANDLE '{"execute":"quit"}' "return"

echo
echo "=== Use qemu-nbd as server ==="
echo

nbd_server_start_unix_socket -r -f $IMGFMT -B b "$TEST_IMG"

There was nothing in the qemu process to let go of t.qcow2 prior to the 
quit request. And although the quit command may have had time to reply 
over the socket that the command was accepted, there may still be a 
delay before the process actually dies, where under sufficient load, the 
qemu-nbd process can start up during that window.

Patch coming up; since I couldn't reproduce the failure, I'd appreciate 
if you could confirm if it seems to help in your setup.

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3226
Virtualization:  qemu.org | libvirt.org

      reply	other threads:[~2019-03-05 18:24 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-03-05 16:10 [Qemu-devel] iotest 223 is flaky Alberto Garcia
2019-03-05 18:24 ` Eric Blake [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=f2fab5f3-9321-9dac-450c-429caf9e55fb@redhat.com \
    --to=eblake@redhat.com \
    --cc=berto@igalia.com \
    --cc=qemu-block@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.