[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Qemu-trivial] [PATCH] help: add id suboption to -iscsi
From: |
Michael Tokarev |
Subject: |
Re: [Qemu-devel] [Qemu-trivial] [PATCH] help: add id suboption to -iscsi |
Date: |
Fri, 06 Dec 2013 23:04:53 +0400 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20131103 Icedove/17.0.10 |
06.12.2013 19:08, Paolo Bonzini wrote:
> --- a/qemu-options.hx
> +++ b/qemu-options.hx
> @@ -2089,7 +2089,7 @@ ETEXI
> DEF("iscsi", HAS_ARG, QEMU_OPTION_iscsi,
> "-iscsi [user=user][,password=password]\n"
> " [,header-digest=CRC32C|CR32C-NONE|NONE-CRC32C|NONE\n"
> - " [,initiator-name=iqn]\n"
> + " [,initiator-name=initiator-iqn][,id=target-iqn]\n"
> " iSCSI session parameters\n", QEMU_ARCH_ALL)
> STEXI
I'm not sure this is clearer, to me it looks more confusing
than before.
The thing is that "id" suboption is automatic and is handled
in the option parsing code explicitly.
As far as I can see, "id" suboption is actually never used by
iscsi code, only initiator-name is.
What this change is for?
Thanks,
/mjt