From: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
To: qemu-devel@nongnu.org, qemu-block@nongnu.org
Cc: armbru@redhat.com, dgilbert@redhat.com, mreitz@redhat.com,
kwolf@redhat.com, pbonzini@redhat.com, eblake@redhat.com,
vsementsov@virtuozzo.com, den@openvz.org
Subject: [Qemu-devel] [PATCH v3 0/5] nbd export qmp interface
Date: Fri, 19 Jan 2018 16:57:14 +0300 [thread overview]
Message-ID: <20180119135719.24745-1-vsementsov@virtuozzo.com> (raw)
v3:
hmp patch deleted
02: tweak commit message and comments
tweak error message
05: use unix socket instead of tcp (more common practice in iotests,
and we do not need port)
rebase on new error message in 02
v2:
01: tweak comment
add Eric's r-b
02: new patch
03: rewritten, to move form 'bool force' flag to 'enum mode' parameter
04: add Eric's r-b
05: improve commit message
tweak comment
06: rebase on 03 changes
make PEP8 happy
some other tweaks
I've left nbd_port variable hard-set to 10900. I think all such things
should be fixed together, and it is simple to change in future
nbd_port = '10900'
to
nbd_port = iotests.get_free_port()
if needed.
[Unfortunately, qmp query-nbd-server is not finished yet, coming soon,
but may be after my vocation on the next week]
Vladimir Sementsov-Ogievskiy (5):
qapi: add name parameter to nbd-server-add
qapi: add nbd-server-remove
iotest 147: add cases to test new @name parameter of nbd-server-add
iotests: implement QemuIoInteractive class
iotest 201: new test for qmp nbd-server-remove
qapi/block.json | 50 +++++++++++++-
include/block/nbd.h | 1 +
blockdev-nbd.c | 38 ++++++++--
hmp.c | 5 +-
nbd/server.c | 20 ++++++
tests/qemu-iotests/147 | 68 ++++++++++++++----
tests/qemu-iotests/147.out | 4 +-
tests/qemu-iotests/201 | 156 ++++++++++++++++++++++++++++++++++++++++++
tests/qemu-iotests/201.out | 5 ++
tests/qemu-iotests/group | 1 +
tests/qemu-iotests/iotests.py | 38 ++++++++++
11 files changed, 362 insertions(+), 24 deletions(-)
create mode 100644 tests/qemu-iotests/201
create mode 100644 tests/qemu-iotests/201.out
--
2.11.1
next reply other threads:[~2018-01-19 13:57 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-01-19 13:57 Vladimir Sementsov-Ogievskiy [this message]
2018-01-19 13:57 ` [Qemu-devel] [PATCH v3 1/5] qapi: add name parameter to nbd-server-add Vladimir Sementsov-Ogievskiy
2018-01-19 13:57 ` [Qemu-devel] [PATCH v3 2/5] qapi: add nbd-server-remove Vladimir Sementsov-Ogievskiy
2018-01-22 22:33 ` Eric Blake
2018-01-19 13:57 ` [Qemu-devel] [PATCH v3 3/5] iotest 147: add cases to test new @name parameter of nbd-server-add Vladimir Sementsov-Ogievskiy
2018-01-19 13:57 ` [Qemu-devel] [PATCH v3 4/5] iotests: implement QemuIoInteractive class Vladimir Sementsov-Ogievskiy
2018-01-19 13:57 ` [Qemu-devel] [PATCH v3 5/5] iotest 201: new test for qmp nbd-server-remove Vladimir Sementsov-Ogievskiy
2018-01-23 15:12 ` Eric Blake
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=20180119135719.24745-1-vsementsov@virtuozzo.com \
--to=vsementsov@virtuozzo.com \
--cc=armbru@redhat.com \
--cc=den@openvz.org \
--cc=dgilbert@redhat.com \
--cc=eblake@redhat.com \
--cc=kwolf@redhat.com \
--cc=mreitz@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-block@nongnu.org \
--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 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.