All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kevin Wolf <kwolf@redhat.com>
To: Michael Tokarev <mjt@tls.msk.ru>
Cc: pbonzini@redhat.com, stefanha@linux.vnet.ibm.com,
	xiawenc@linux.vnet.ibm.com, anthony@codemonkey.ws,
	qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH] add text about how to use qemu-nbd with qemu
Date: Wed, 04 Jul 2012 11:56:30 +0200	[thread overview]
Message-ID: <4FF4134E.4080905@redhat.com> (raw)
In-Reply-To: <4FF34BBD.2050101@msgid.tls.msk.ru>

Am 03.07.2012 21:45, schrieb Michael Tokarev:
> On 03.07.2012 15:05, xiawenc@linux.vnet.ibm.com wrote:
>> From: Wenchao Xia <xiawenc@linux.vnet.ibm.com>
>>
>>   Qemu system emulator reports only fails that make people confused
>> about why, when it is invoked with nbd block device tring to connect
>> qemu-nbd server. In fact qemu will try connect server for several
>> times but server only accept one connect by default.
>>   I paid some times to find out why it fails. This patch add special
>> text to explain how to use it in such case.
>>
>> Signed-off-by: Wenchao Xia <xiawenc@linux.vnet.ibm.com>
>> ---
>>  qemu-nbd.texi |    6 ++++++
>>  1 files changed, 6 insertions(+), 0 deletions(-)
>>
>> diff --git a/qemu-nbd.texi b/qemu-nbd.texi
>> index 6955d90..014040a 100644
>> --- a/qemu-nbd.texi
>> +++ b/qemu-nbd.texi
>> @@ -44,6 +44,12 @@ Export QEMU disk image using NBD protocol.
>>    display this help and exit
>>  @item -V, --version
>>    output version information and exit
>> +
>> +  Note: When qemu-nbd was used to export a disk that would be used by QEMU
>> +block device, -t or --persistent must be set for that QEMU nbd client would try
>> +connect more than one time. For eg:
>> +  qemu-nbd ./simple.img -p 10809 -t
>> +  qemu -hdb nbd:127.0.0.1:10809
> 
> It is not accurate.
> 
> qemu connect to nbd server not "several" but exactly two times by default.
> The first connection it does (it need not be a separate connection
> actually, and this, I'd say, a bug in qemu) in order to guess the format
> of the internal structure inside - be it raw, or qcow, or anything else.

I think I agree that we should get rid of this behaviour. It's going to
bite us with fd passing as well. It should be possible to open the
protocol first and the use that open BlockDriverState both for probing
and for the real open.

Kevin

      parent reply	other threads:[~2012-07-04  9:56 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-03 11:05 [Qemu-devel] [PATCH] add text about how to use qemu-nbd with qemu xiawenc
2012-07-03 11:55 ` Paolo Bonzini
2012-07-04  2:21   ` Wayne Xia
2012-07-05  3:29   ` Wayne Xia
2012-07-05  6:42     ` Paolo Bonzini
2012-07-05  8:13       ` Michael Tokarev
2012-07-05  8:34         ` Paolo Bonzini
2012-07-06  8:01           ` Wayne Xia
2012-07-09  7:11             ` Wayne Xia
2012-07-09  7:15               ` Paolo Bonzini
2012-07-03 16:18 ` 陳韋任 (Wei-Ren Chen)
2012-07-04  2:26   ` Wayne Xia
2012-07-03 19:45 ` Michael Tokarev
2012-07-04  2:14   ` Wayne Xia
2012-07-04  7:35     ` Paolo Bonzini
2012-07-04  9:56   ` Kevin Wolf [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=4FF4134E.4080905@redhat.com \
    --to=kwolf@redhat.com \
    --cc=anthony@codemonkey.ws \
    --cc=mjt@tls.msk.ru \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@linux.vnet.ibm.com \
    --cc=xiawenc@linux.vnet.ibm.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.