qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: ronniesahlberg@gmail.com
To: qemu-devel@nongnu.org
Subject: [Qemu-devel] iSCSI block driver support
Date: Fri,  4 Feb 2011 15:37:55 +1100	[thread overview]
Message-ID: <1296794276-1683-1-git-send-email-ronniesahlberg@gmail.com> (raw)

The following patch adds a new block driver to QEMU/KVM for iSCSI.
This utilizes the userspace client library for iscsi at https://github.com/sahlberg/libiscsi and will link with this library if available.

This allows using iSCSI resources with QEMU/KVM without making them visible to the underlying host. Which is very useful when having very large number of iscsi devices, or when you for other reasons do not want to expose these devicdes to others.

  
I have used this patch and installed a RHEL6 system from an iscsi dvd onto a iscsi disk. I have done extensive testing.
Both normally and also running under valgrind (which is very slow)

./x86_64-softmmu/qemu-system-x86_64 -m 1024 --enable-kvm -cdrom iscsi://127.0.0.1:3262/iqn.ronnie.test/2 -drive if=virtio,file=iscsi://127.0.0.1:3262/iqn.ronnie.test/1 -boot c

Please review and/or apply.



This is version 3 of the patch :
Version 3 adds :
  - Fix a bug when writing freed data to the disk
  - Convert TAB to spaces
  - Add tracing functions (From Stefan Hajnoczi)
  - Add a io_flush() handler

-----

    Version 2:
    Changes from Blue Swirl's suggestions
    - Change naming of structures and typedefs to match coding style
    - Use CONFIG_ISCSI in the makefile to conditionally compile iscsi.c
    - Missing spaces added around operators
    - Use uint8_t instead of unsigned char for buffer pointer
    - Use a temporary variable for the result of long functions, to move them
      out from the if(...) expressions. Making the code easier to read.
    - Create a function sector_qemu2lun() and use it instead of performing
      the same conversion explicitely at several palces in the code.
    - Use the name opaque instead of private_data
    - Use memset() instead of bzero()
    - Use the common library function to parse the iSCSI URL
    - Add support for CHAP authentication

---------
 Makefile.objs |    1
 block/iscsi.c |  542 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 configure     |   31 +++
 trace-events  |    7
 4 files changed, 581 insertions(+)

             reply	other threads:[~2011-02-04  4:38 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-02-04  4:37 ronniesahlberg [this message]
2011-02-04  4:37 ` [Qemu-devel] [PATCH] iSCSI support ronniesahlberg
2011-02-19 15:34   ` Stefan Hajnoczi
2011-02-27  4:28     ` ronnie sahlberg
2011-02-19 16:37 ` [Qemu-devel] iSCSI block driver support Stefan Hajnoczi
  -- strict thread matches above, loose matches on Subject: below --
2011-02-27  5:04 Ronnie Sahlberg

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=1296794276-1683-1-git-send-email-ronniesahlberg@gmail.com \
    --to=ronniesahlberg@gmail.com \
    --cc=qemu-devel@nongnu.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).