Xen project Mailing List

[Xen-devel] [PATCH v7 0/5] libbxl: add support for pvscsi, iteration 7

From: Olaf Hering <olaf@xxxxxxxxx>

Date: Fri, 5 Feb 2016 10:12:50 +0000

Delivery-date: Fri, 05 Feb 2016 10:13:12 +0000

List-id: Xen developer discussion <xen-devel.lists.xen.org>

Port vscsi=[] and scsi-{attach,detach,list} commands from xend to libxl. libvirt uses its existing SCSI support: http://lists.xenproject.org/archives/html/xen-devel/2015-04/msg02963.html targetcli/rtslib has to be aware of xen-scsiback (upstream unresponsive): http://article.gmane.org/gmane.linux.scsi.target.devel/8146 TODO: - check if "detach <vdev>" can be done with single host/single device instead of the ->remove field - check if a transaction should be used in libxl__device_vscsi_add - maybe use events instead of polling for "state" changes in reconfigure (libxl__wait_for_backend vs. libxl__ev_devstate_wait) - comply with tools/libxl/CODING_STYLE everywhere - document pvscsi in xen wiki Changes between v6 and v7: - rebase to 'staging' (3b971de) - Introduce libxl_device_vscsictrl_append_vscsidev - Add libxl__vscsi_collect_ctrls and used it in libxl_device_vscsictrl_list - Convert type of lun from u32 to u64 per SCSI spec - Use vscsi_saved in libxl__device_vscsictrl_add - Assign unique vscsidev_id per vscsictrl - Rename vscsi_dev to vscsidev in function names - Rename variables in libxl_vscsiinfo - Rename locals to refer to ctrl/dev - Rename various strings from host to ctrl - Rename local variables vscsi_ctrl to vscsictrl - Rename libxl_device_vscsictrl->vscsi_devs to vscsidevs - Remove libxl_device_vscsidev->remove, rework detach - Rename libxl_device_vscsidev->vscsi_dev_id to vscsidev_id - Rename local variables vscsi_host to vscsi_ctrl - Rename local variables v_hst to v_ctrl - Remove libxl_device_vscsictrl->v_hst - Rename libxl_vscsi_dev to libxl_device_vscsidev - Rename libxl_device_vscsi to libxl_device_vscsictrl Changes between v5 and v6: - rebase to 'staging' (a7b39c8 and b7e7ad8) - Fix off-by-one in xlu__vscsi_compare_udev - xl.cfg: use options instead of option - xl.cfg: fix grammar for pdev/options - xl.cfg: fix Usually typo - Remove next_vscsi_dev_id from libxl_device_vscsi - Use XLU_WWN_LEN also in libxl_vscsi.c http://lists.xenproject.org/archives/html/xen-devel/2015-11/msg01446.html Changes between v4 and v5: - vscsiif.h: refer to backend_domid - Set update_json in libxl_device_vscsi_remove - Remove comment from libxl__device_vscsi_add - Remove debug LOG from libxl__device_vscsi_reconfigure - Move local nb variable in libxl__device_vscsi_reconfigure - Make be_path const in libxl__device_vscsi_reconfigure - Adjust libxl__device_vscsi_dev_backend_set to avoid long lines - Adjust libxl__device_vscsi_dev_backend_rm to avoid long lines - Use CTX in libxl__device_vscsi_dev_backend_rm - Make be_path const in libxl__device_vscsi_dev_backend_rm - xl.cfg: Its typo - xl.cfg: Use persistent instead of persistant - Rename feature_host to scsi_raw_cmds - target-create-xen-scsiback.sh: detect pvops and xenlinux - Wrap long lines in main_vscsilist - Call libxl_vscsiinfo_dispose unconditional - Let scsi-list print p-dev instead of p-devname - Handle broken vscsi device entry in xenstore - Split libxl__vscsi_fill_host from libxl_device_vscsi_list - Make xlu_vscsi_append_dev static - Remove reference to pvscsi.txt from xenstore-paths.markdown - xl.cfg: update Linux and xenlinux - xl.cfg: refer to backend domain instead of dom0 - xl.cfg: be more verbose what persistant format is - return if libxl__device_vscsi_dev_backend_set fails in libxl__device_vscsi_new_backend - target-create-xen-scsiback.sh: set also alias for libvirt http://lists.xenproject.org/archives/html/xen-devel/2015-05/msg00523.html Changes between v3 and v4: - Use libxl__device_nextid in libxl__device_vscsi_add - Remove check for duplicate pdev assignment from libxl_device_vscsi_get_host - Caller provides libxl_device_vscsi to libxl_device_vscsi_get_host - Define LIBXL_HAVE_VSCSI - Remove init_val from libxl_vscsi_pdev_type - Move some functions from libxl to libxlu - Introduce libxl_device_vscsi->next_vscsi_dev_id to handle holes - Use Struct in KeyedUnion for ocaml idl - docs: Mention pvscsi in xl and xl.cfg - Turn feature_host into a defbool and add checking - Support pvops and /dev/ nodes in config - Wrap entire libxlu_vscsi.c with ifdef linux - Set remove flag in libxl_device_vscsi_list - Fix vscsiif path in xenstore-paths.markdown - vscsiif.h: add some notes about xenstore layout - Add copyright to libxlu_vscsi.c and libxl_vscsi.c - Scripts to create and delete xen-scsiback nodes in Linux target framework - Remove pvscsi.txt http://lists.xenproject.org/archives/html/xen-devel/2015-04/msg01949.html Changes between v2 and v3: - Adjust change for vscsiif.h - Support "naa.wwn:lun" notation in pvops kernel - Add example for pvops kernel using targetcli patch required for python-rtslib: http://article.gmane.org/gmane.linux.scsi.target.devel/8146 - Use vdev variable in libxl_device_vscsi_parse http://lists.xenproject.org/archives/html/xen-devel/2015-03/msg00734.html Changes between v1 and v2: - ported to current staging http://lists.xenproject.org/archives/html/xen-devel/2015-03/msg00030.html Initial attempt was sent a year ago: http://lists.xenproject.org/archives/html/xen-devel/2014-04/msg03958.html Most comments are addressed. This version has been tested with SLES as backend and frontend. This version has been tested with pvops as backend and SLES as frontend. Olaf Hering (5): vscsiif.h: fix WWN notation for p-dev property docs: add vscsi to xenstore-paths.markdown libxl: add support for vscsi vscsiif.h: add some notes about xenstore layout Scripts to create and delete xen-scsiback nodes in Linux target framework docs/man/xl.cfg.pod.5 | 55 +++ docs/man/xl.pod.1 | 18 + docs/misc/xenstore-paths.markdown | 10 + tools/libxl/Makefile | 2 + tools/libxl/libxl.c | 578 +++++++++++++++++++++++++ tools/libxl/libxl.h | 35 ++ tools/libxl/libxl_create.c | 1 + tools/libxl/libxl_device.c | 2 + tools/libxl/libxl_internal.h | 14 + tools/libxl/libxl_types.idl | 53 +++ tools/libxl/libxl_types_internal.idl | 1 + tools/libxl/libxl_vscsi.c | 344 +++++++++++++++ tools/libxl/libxlu_vscsi.c | 718 +++++++++++++++++++++++++++++++ tools/libxl/libxlutil.h | 18 + tools/libxl/xl.h | 3 + tools/libxl/xl_cmdimpl.c | 208 ++++++++- tools/libxl/xl_cmdtable.c | 15 + tools/misc/Makefile | 4 + tools/misc/target-create-xen-scsiback.sh | 135 ++++++ tools/misc/target-delete-xen-scsiback.sh | 41 ++ xen/include/public/io/vscsiif.h | 70 ++- 21 files changed, 2323 insertions(+), 2 deletions(-) create mode 100644 tools/libxl/libxl_vscsi.c create mode 100644 tools/libxl/libxlu_vscsi.c create mode 100755 tools/misc/target-create-xen-scsiback.sh create mode 100755 tools/misc/target-delete-xen-scsiback.sh _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel

©2013 Xen Project, A Linux Foundation Collaborative Project. All Rights Reserved.
Linux Foundation is a registered trademark of The Linux Foundation.
Xen Project is a trademark of The Linux Foundation.