Skip to content

Remove need for -iscsi argument

Daniel P. Berrange qemu-devel 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest2 patches 2 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/2iscsi: reduce code duplication parsing -iscsi optsblock/iscsi.c+48 -640 replies
2/2iscsi: support most -iscsi opts against block dev optsblock/iscsi.c block/iscsi.h vl.c+136 -540 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [Qemu-devel] [PATCH 0/2] Remove need for -iscsi argument

This small series removes the need for using the -iscsi argument by directly supporting all the -iscsi options against the block device config eg instead of

$QEMU \
-object secret,id=sec0,data=123456 \
-drive driver=iscsi,file=iscsi://catbus/iqn.fedora:kvm/2 \
-iscsi initiator-name=fish,user=test,password-secret=sec0,timeout=234

you can now do it all against -drive

$QEMU \
-object secret,id=sec0,data=123456 \
-drive driver=iscsi,file=iscsi://catbus/iqn.fedora:kvm/2,\
initiator-name=fish,user=test,password-secret=sec0,timeout=234

This fixes the problem that you cannot associate -drive instances with -iscsi args reliably due to IQNs containing chracters that QEMU forbids in QemuOpts ID strings.

Pino had previously sent a patch to do this for initiator-name back in the 2.6 dev cycle

https://lists.nongnu.org/archive/html/qemu-block/2016-04/msg00246.html

This series supports all the -iscsi option names, except for 'password' which is intentionally left out as it is insecure and replaced by 'password-secret'.

Daniel P. Berrange (2):
iscsi: reduce code duplication parsing -iscsi opts
iscsi: support most -iscsi opts against block dev opts

 block/iscsi.c | 145 ++++++++++++++++++++++++++++++----------------------------
 block/iscsi.h |  79 ++++++++++++++++++++++++++++++++
 vl.c          |  36 ++++-----------
 3 files changed, 163 insertions(+), 97 deletions(-)
 create mode 100644 block/iscsi.h
Signature · 1 line
2.9.3