From: Peter Lieven <pl@dlhnet.de>
To: "qemu-devel@nongnu.org" <qemu-devel@nongnu.org>
Cc: Kevin Wolf <kwolf@redhat.com>,
Paolo Bonzini <pbonzini@redhat.com>,
ronnie sahlberg <ronniesahlberg@gmail.com>
Subject: Re: [Qemu-devel] iscsi / block segfault regression
Date: Thu, 21 Mar 2013 12:03:20 +0100 [thread overview]
Message-ID: <514AE8F8.6010301@dlhnet.de> (raw)
In-Reply-To: <514AE4CF.70009@dlhnet.de>
git bisect results:
lieven@lieven-pc:~/git/qemu$ git bisect bad
f500a6d3c2b9ef0bb06d0080d91d8ed3c1d68f58 is the first bad commit
commit f500a6d3c2b9ef0bb06d0080d91d8ed3c1d68f58
Author: Kevin Wolf <kwolf@redhat.com>
Date: Mon Nov 12 17:35:27 2012 +0100
block: Avoid second open for format probing
This fixes problems that are caused by the additional open/close cycle
of the existing format probing, for example related to qemu-nbd without
-t option or file descriptor passing.
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
:100644 100644 c4f5566b5de6cfa371d7b60dd8b0e1a2da7fd55f 2ec3afebfe6a6c12b573cba9ad36de800623fe6a M block.c
On 21.03.2013 11:45, Peter Lieven wrote:
> Hi,
>
> I currently try to debug a regression in git master with iscsi that happens with some iscsi volumes.
> I added debugging output to all aio calls into block/iscsi.c and it seems that immediately after
> brdv_open something is overwriting bs->opaque with a new pointer.
>
> ~/git/qemu$ x86_64-softmmu/qemu-system-x86_64 -k de -enable-kvm -m 1024 -drive format=iscsi,file=iscsi://172.21.200.31/iqn.2001-05.com.equallogic:0-8a0906-3c0f4e007-dce91829d0450add-lieven-storage-migration-test-hd0/0,if=virtio,cache=writeback,aio=native
> -vnc :1 -boot dc -monitor stdio
> iscsi open start
> iscsi_full_connect_sync: iscsilun = 140063347224208, iscsilun->iscsi = 140063347225536, bs = 140063347221696, filename=iscsi://172.21.200.31/iqn.2001-05.com.equallogic:0-8a0906-3c0f4e007-dce91829d0450add-lieven-storage-migration-test-hd0/0
> iscsi open finish ret=0
> iscsi_aio_flush: iscsilun = 140063347230736, iscsilun->iscsi = 0, bs = 140063347221696, filename=iscsi://172.21.200.31/iqn.2001-05.com.equallogic:0-8a0906-3c0f4e007-dce91829d0450add-lieven-storage-migration-test-hd0/0
> iscsi_aio_flush_acb: iscsilun = 140063347230736, iscsilun->iscsi = 0, bs = 140063347221696, filename=iscsi://172.21.200.31/iqn.2001-05.com.equallogic:0-8a0906-3c0f4e007-dce91829d0450add-lieven-storage-migration-test-hd0/0
> Speicherzugriffsfehler (Speicherabzug geschrieben)
>
> bs address is unchanged, but iscsilun address is changed after connection it seems to an area with zeroed memory. the pointer to the iscsi context
> is thus NULL.
>
> iscsilun, iscsilun->iscsi and bs are addresses.
>
> Peter
>
next prev parent reply other threads:[~2013-03-21 11:03 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-03-21 10:45 [Qemu-devel] iscsi / block segfault regression Peter Lieven
2013-03-21 11:03 ` Peter Lieven [this message]
2013-03-21 11:10 ` Peter Lieven
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=514AE8F8.6010301@dlhnet.de \
--to=pl@dlhnet.de \
--cc=kwolf@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=ronniesahlberg@gmail.com \
/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.