From: Fam Zheng <famz@redhat.com>
To: qemu-devel@nongnu.org
Cc: Stefan Hajnoczi <stefanha@redhat.com>,
Kevin Wolf <kwolf@redhat.com>, Max Reitz <mreitz@redhat.com>,
qemu-block@nongnu.org
Subject: [Qemu-devel] [PATCH v2 16/16] tests: Add test case for BLK_PERM_AIO_CONTEXT_CHANGE
Date: Wed, 19 Apr 2017 17:43:56 +0800 [thread overview]
Message-ID: <20170419094356.19826-17-famz@redhat.com> (raw)
In-Reply-To: <20170419094356.19826-1-famz@redhat.com>
Signed-off-by: Fam Zheng <famz@redhat.com>
---
tests/Makefile.include | 2 ++
tests/test-blk-perm.c | 59 ++++++++++++++++++++++++++++++++++++++++++++++++++
2 files changed, 61 insertions(+)
create mode 100644 tests/test-blk-perm.c
diff --git a/tests/Makefile.include b/tests/Makefile.include
index f3de81f..b38e090 100644
--- a/tests/Makefile.include
+++ b/tests/Makefile.include
@@ -56,6 +56,7 @@ gcov-files-test-thread-pool-y = thread-pool.c
gcov-files-test-hbitmap-y = util/hbitmap.c
check-unit-y += tests/test-hbitmap$(EXESUF)
gcov-files-test-hbitmap-y = blockjob.c
+check-unit-y += tests/test-blk-perm$(EXESUF)
check-unit-y += tests/test-blockjob$(EXESUF)
check-unit-y += tests/test-blockjob-txn$(EXESUF)
check-unit-y += tests/test-x86-cpuid$(EXESUF)
@@ -543,6 +544,7 @@ tests/test-coroutine$(EXESUF): tests/test-coroutine.o $(test-block-obj-y)
tests/test-aio$(EXESUF): tests/test-aio.o $(test-block-obj-y)
tests/test-aio-multithread$(EXESUF): tests/test-aio-multithread.o $(test-block-obj-y)
tests/test-throttle$(EXESUF): tests/test-throttle.o $(test-block-obj-y)
+tests/test-blk-perm$(EXESUF): tests/test-blk-perm.o $(test-block-obj-y)
tests/test-blockjob$(EXESUF): tests/test-blockjob.o $(test-block-obj-y) $(test-util-obj-y)
tests/test-blockjob-txn$(EXESUF): tests/test-blockjob-txn.o $(test-block-obj-y) $(test-util-obj-y)
tests/test-thread-pool$(EXESUF): tests/test-thread-pool.o $(test-block-obj-y)
diff --git a/tests/test-blk-perm.c b/tests/test-blk-perm.c
new file mode 100644
index 0000000..d6129ab
--- /dev/null
+++ b/tests/test-blk-perm.c
@@ -0,0 +1,59 @@
+/*
+ * Block permission tests
+ *
+ * Copyright Red Hat, Inc. 2017
+ *
+ * Authors:
+ * Fam Zheng <famz@redhat.com>
+ *
+ * This work is licensed under the terms of the GNU LGPL, version 2 or later.
+ * See the COPYING.LIB file in the top-level directory.
+ */
+
+#include "qemu/osdep.h"
+#include "qapi/error.h"
+#include "sysemu/block-backend.h"
+
+static void test_aio_context_success(void)
+{
+ BlockBackend *blk1 = blk_new(BLK_PERM_AIO_CONTEXT_CHANGE, BLK_PERM_ALL);
+ BlockBackend *blk2 = blk_new(BLK_PERM_AIO_CONTEXT_CHANGE, BLK_PERM_ALL);
+ BlockDriverState *bs = bdrv_open("null-co://", NULL, NULL, 0, &error_abort);
+
+ blk_insert_bs(blk1, bs, &error_abort);
+ blk_insert_bs(blk2, bs, &error_abort);
+
+ blk_unref(blk1);
+ blk_unref(blk2);
+ bdrv_unref(bs);
+}
+
+static void test_aio_context_failure(void)
+{
+ Error *local_err = NULL;
+ BlockBackend *blk1 = blk_new(BLK_PERM_AIO_CONTEXT_CHANGE,
+ BLK_PERM_ALL & ~BLK_PERM_AIO_CONTEXT_CHANGE);
+ BlockBackend *blk2 = blk_new(BLK_PERM_AIO_CONTEXT_CHANGE, BLK_PERM_ALL);
+ BlockDriverState *bs = bdrv_open("null-co://", NULL, NULL, 0, &error_abort);
+
+ blk_insert_bs(blk1, bs, &error_abort);
+ blk_insert_bs(blk2, bs, &local_err);
+
+ g_assert_nonnull(local_err);
+
+ blk_unref(blk1);
+ blk_unref(blk2);
+ bdrv_unref(bs);
+}
+
+int main(int argc, char **argv)
+{
+ bdrv_init();
+ qemu_init_main_loop(&error_abort);
+ g_test_init(&argc, &argv, NULL);
+ g_test_add_func("/block/perm/aio-context/success",
+ test_aio_context_success);
+ g_test_add_func("/block/perm/aio-context/failure",
+ test_aio_context_failure);
+ return g_test_run();
+}
--
2.9.3
next prev parent reply other threads:[~2017-04-19 9:45 UTC|newest]
Thread overview: 41+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-04-19 9:43 [Qemu-devel] [PATCH v2 00/16] block: Protect AIO context change with perm API Fam Zheng
2017-04-19 9:43 ` [Qemu-devel] [PATCH v2 01/16] block: Define BLK_PERM_AIO_CONTEXT_CHANGE Fam Zheng
2017-05-11 19:26 ` [Qemu-devel] [Qemu-block] " Stefan Hajnoczi
2017-04-19 9:43 ` [Qemu-devel] [PATCH v2 02/16] block-backend: Add blk_request_perm Fam Zheng
2017-05-11 19:32 ` [Qemu-devel] [Qemu-block] " Stefan Hajnoczi
2017-05-24 2:16 ` Fam Zheng
2017-04-19 9:43 ` [Qemu-devel] [PATCH v2 03/16] blockjob: Add BLK_PERM_AIO_CONTEXT_CHANGE shared perm on bs Fam Zheng
2017-05-11 19:33 ` Stefan Hajnoczi
2017-04-19 9:43 ` [Qemu-devel] [PATCH v2 04/16] blockjob: Allow aio context change on intermediate nodes Fam Zheng
2017-05-11 19:34 ` Stefan Hajnoczi
2017-04-19 9:43 ` [Qemu-devel] [PATCH v2 05/16] block: Propagate BLK_PERM_AIO_CONTEXT_CHANGE down the graph Fam Zheng
2017-05-11 19:35 ` [Qemu-devel] [Qemu-block] " Stefan Hajnoczi
2017-04-19 9:43 ` [Qemu-devel] [PATCH v2 06/16] backup: Request BLK_PERM_AIO_CONTEXT_CHANGE on target Fam Zheng
2017-05-11 19:41 ` [Qemu-devel] [Qemu-block] " Stefan Hajnoczi
2017-05-24 2:18 ` Fam Zheng
2017-05-31 9:39 ` Stefan Hajnoczi
2017-05-31 9:57 ` Fam Zheng
2017-06-01 13:26 ` Stefan Hajnoczi
2017-06-05 8:34 ` Fam Zheng
2017-04-19 9:43 ` [Qemu-devel] [PATCH v2 07/16] backup: Do initial aio context move of target via BB interface Fam Zheng
2017-05-11 20:00 ` [Qemu-devel] [Qemu-block] " Stefan Hajnoczi
2017-04-19 9:43 ` [Qemu-devel] [PATCH v2 08/16] mirror: Request aio context change permission on target Fam Zheng
2017-05-11 20:01 ` [Qemu-devel] [Qemu-block] " Stefan Hajnoczi
2017-04-19 9:43 ` [Qemu-devel] [PATCH v2 09/16] commit: Allow aio context change on s->base Fam Zheng
2017-05-11 20:06 ` Stefan Hajnoczi
2017-04-19 9:43 ` [Qemu-devel] [PATCH v2 10/16] mirror: Do initial aio context move of target via BB interface Fam Zheng
2017-05-11 20:27 ` [Qemu-devel] [Qemu-block] " Stefan Hajnoczi
2017-05-24 2:23 ` Fam Zheng
2017-04-19 9:43 ` [Qemu-devel] [PATCH v2 11/16] virtio-scsi: Request BLK_PERM_AIO_CONTEXT_CHANGE for dataplane Fam Zheng
2017-05-11 20:28 ` [Qemu-devel] [Qemu-block] " Stefan Hajnoczi
2017-04-19 9:43 ` [Qemu-devel] [PATCH v2 12/16] virtio-blk: " Fam Zheng
2017-05-11 20:29 ` [Qemu-devel] [Qemu-block] " Stefan Hajnoczi
2017-04-19 9:43 ` [Qemu-devel] [PATCH v2 13/16] blk: fix aio context loss on media change Fam Zheng
2017-05-11 20:31 ` [Qemu-devel] [Qemu-block] " Stefan Hajnoczi
2017-04-19 9:43 ` [Qemu-devel] [PATCH v2 14/16] nbd: Allow BLK_PERM_AIO_CONTEXT_CHANGE on BB Fam Zheng
2017-05-11 20:32 ` [Qemu-devel] [Qemu-block] " Stefan Hajnoczi
2017-04-19 9:43 ` [Qemu-devel] [PATCH v2 15/16] block: Add perm assertion on blk_set_aio_context Fam Zheng
2017-05-11 20:33 ` [Qemu-devel] [Qemu-block] " Stefan Hajnoczi
2017-04-19 9:43 ` Fam Zheng [this message]
2017-05-11 20:36 ` [Qemu-devel] [PATCH v2 16/16] tests: Add test case for BLK_PERM_AIO_CONTEXT_CHANGE Stefan Hajnoczi
2017-05-11 20:43 ` [Qemu-devel] [Qemu-block] [PATCH v2 00/16] block: Protect AIO context change with perm API Stefan Hajnoczi
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=20170419094356.19826-17-famz@redhat.com \
--to=famz@redhat.com \
--cc=kwolf@redhat.com \
--cc=mreitz@redhat.com \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@redhat.com \
/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).