From: Boaz Harrosh <bharrosh@panasas.com>
To: James Bottomley <James.Bottomley@HansenPartnership.com>,
Andrew Morton <akpm@linux-foundation.org>,
Mike Christie <michaelc@cs.wisc.edu>,
FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>,
Jeff Garzik <jeff@garzik.org>, open-osd ml <osd-dev@open-osd.org>,
linux-scsi <linux-scsi@vger.kernel.org>
Cc: linux-kernel <linux-kernel@vger.kernel.org>,
Sami.Iren@seagate.com, Pete Wyckoff <pw@padd.com>
Subject: [PATCH 08/18] osd_ktests: Add basic OSD tests
Date: Tue, 4 Nov 2008 18:44:44 +0200 [thread overview]
Message-ID: <1225817084-6063-1-git-send-email-bharrosh@panasas.com> (raw)
In-Reply-To: <491073BB.4000900@panasas.com>
Currently testing what is implemented in the osd_initiator library.
That is - format, create/remove partition, create/remove object,
read and write to objects.
This test passes against the IBM-OSD-SIM OSDv1 target.
Signed-off-by: Boaz Harrosh <bharrosh@panasas.com>
Reviewed-by: Benny Halevy <bhalevy@panasas.com>
---
drivers/scsi/osd/Kbuild | 2 +-
drivers/scsi/osd/osd_ktests.c | 331 +++++++++++++++++++++++++++++++++++++++++
drivers/scsi/osd/osd_ktests.h | 4 +
drivers/scsi/osd/osd_uld.c | 2 +-
4 files changed, 337 insertions(+), 2 deletions(-)
create mode 100644 drivers/scsi/osd/osd_ktests.c
diff --git a/drivers/scsi/osd/Kbuild b/drivers/scsi/osd/Kbuild
index f39f82f..da00ddb 100644
--- a/drivers/scsi/osd/Kbuild
+++ b/drivers/scsi/osd/Kbuild
@@ -29,5 +29,5 @@ libosd-objs := osd_initiator.o
obj-$(CONFIG_SCSI_OSD_INITIATOR) += libosd.o
# osd.ko - SCSI ULD and char-device
-osd-objs := osd_uld.o
+osd-objs := osd_uld.o osd_ktests.o
obj-$(CONFIG_SCSI_OSD_ULD) += osd.o
diff --git a/drivers/scsi/osd/osd_ktests.c b/drivers/scsi/osd/osd_ktests.c
new file mode 100644
index 0000000..f620915
--- /dev/null
+++ b/drivers/scsi/osd/osd_ktests.c
@@ -0,0 +1,331 @@
+/*
+ * osd_ktests.c - An osd_initiator library in-kernel test suite
+ * called by the osd_uld module
+ *
+ * Copyright (C) 2008 Panasas Inc. All rights reserved.
+ *
+ * Authors:
+ * Boaz Harrosh <bharrosh@panasas.com>
+ * Benny Halevy <bhalevy@panasas.com>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ *
+ * 1. Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution.
+ * 3. Neither the name of the Panasas company nor the names of its
+ * contributors may be used to endorse or promote products derived
+ * from this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED ``AS IS'' AND ANY EXPRESS OR IMPLIED
+ * WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF
+ * MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
+ * DISCLAIMED. IN NO EVENT SHALL THE REGENTS OR CONTRIBUTORS BE LIABLE
+ * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR
+ * BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF
+ * LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
+ * NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
+ * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ *
+ */
+#include <asm/unaligned.h>
+#include <linux/vmalloc.h>
+#include <scsi/scsi_device.h>
+
+#include <scsi/osd_initiator.h>
+#include <scsi/osd_sec.h>
+
+#include "osd_ktests.h"
+#include "osd_debug.h"
+
+enum {
+ K = 1024,
+ M = 1024 * K,
+ G = 1024 * M,
+};
+
+const u64 format_total_capacity = 128 * M;
+const osd_id first_par_id = 0x17171717L;
+const osd_id first_obj_id = 0x18181818L;
+const unsigned BUFF_SIZE = PAGE_SIZE;
+
+const int num_partitions = 1;
+const int num_objects = 2; /* per partition */
+
+int test_exec(struct osd_request *or, void *caps, const struct osd_obj_id *obj)
+{
+ int ret;
+
+ osd_sec_init_nosec_doall_caps(caps, obj, false, true);
+ ret = osd_finalize_request(or, 0, caps, NULL);
+ if (ret)
+ return ret;
+
+ ret = osd_execute_request(or);
+ /* osd_req_decode_sense(or, ret); */
+ return ret;
+}
+
+#define KTEST_START_REQ(osd_dev, or) do { \
+ or = osd_start_request(osd_dev, GFP_KERNEL); \
+ if (!or) { \
+ OSD_ERR("Error @%s:%d: osd_start_request", __func__,\
+ __LINE__); \
+ return -ENOMEM; \
+ } \
+} while (0)
+
+#define KTEST_EXEC_END(or, obj, g_caps, msg) do { \
+ ret = test_exec(or, g_caps, obj); \
+ osd_end_request(or); \
+ if (ret) { \
+ OSD_ERR("Error executing "msg" => %d\n", ret); \
+ return ret; \
+ } \
+ OSD_DEBUG(msg "\n"); \
+} while (0)
+
+int ktest_format(struct osd_dev *osd_dev)
+{
+ struct osd_request *or;
+ u8 g_caps[OSD_CAP_LEN];
+ int ret;
+
+ KTEST_START_REQ(osd_dev, or);
+ osd_req_format(or, format_total_capacity);
+ KTEST_EXEC_END(or, &osd_root_object, g_caps, "format");
+ return 0;
+}
+
+int ktest_creat_par(struct osd_dev *osd_dev)
+{
+ struct osd_request *or;
+ u8 g_caps[OSD_CAP_LEN];
+ int ret;
+ int p;
+
+ for (p = 0; p < num_partitions; p++) {
+ struct osd_obj_id par = {
+ .partition = first_par_id + p,
+ .id = 0
+ };
+
+ KTEST_START_REQ(osd_dev, or);
+ osd_req_create_partition(or, par.partition);
+ KTEST_EXEC_END(or, &par, g_caps, "create_partition");
+ }
+
+ return 0;
+}
+
+int ktest_creat_obj(struct osd_dev *osd_dev)
+{
+ struct osd_request *or;
+ u8 g_caps[OSD_CAP_LEN];
+ int ret;
+ int p, o;
+
+ for (p = 0; p < num_partitions; p++)
+ for (o = 0; o < num_objects; o++) {
+ struct osd_obj_id obj = {
+ .partition = first_par_id + p,
+ .id = first_obj_id + o
+ };
+
+ KTEST_START_REQ(osd_dev, or);
+ osd_req_create_object(or, &obj);
+ KTEST_EXEC_END(or, &obj, g_caps, "create_object");
+ }
+
+ return 0;
+}
+
+int ktest_write_obj(struct osd_dev *osd_dev, void *write_buff)
+{
+ struct request_queue *req_q = osd_dev->scsi_dev->request_queue;
+ struct osd_request *or;
+ u8 g_caps[OSD_CAP_LEN];
+ int ret;
+ int p, o; u64 offset = 0;
+ struct bio *write_bio;
+
+ for (p = 0; p < num_partitions; p++)
+ for (o = 0; o < num_objects; o++) {
+ struct osd_obj_id obj = {
+ .partition = first_par_id + p,
+ .id = first_obj_id + o
+ };
+
+ KTEST_START_REQ(osd_dev, or);
+ write_bio = bio_map_kern(req_q, write_buff,
+ BUFF_SIZE, GFP_KERNEL);
+ if (!write_bio) {
+ OSD_ERR("!!! Failed to allocate write BIO\n");
+ return -ENOMEM;
+ }
+
+ osd_req_write(or, &obj, write_bio, offset);
+ KTEST_EXEC_END(or, &obj, g_caps, "write");
+ write_bio = NULL; /* released by scsi_midlayer */
+ offset += BUFF_SIZE;
+ }
+
+ return 0;
+}
+
+int ktest_read_obj(struct osd_dev *osd_dev, void *write_buff, void *read_buff)
+{
+ struct request_queue *req_q = osd_dev->scsi_dev->request_queue;
+ struct osd_request *or;
+ u8 g_caps[OSD_CAP_LEN];
+ int ret;
+ int p, o; u64 offset = 0;
+ struct bio *read_bio;
+
+ for (p = 0; p < num_partitions; p++)
+ for (o = 0; o < num_objects; o++) {
+ struct osd_obj_id obj = {
+ .partition = first_par_id + p,
+ .id = first_obj_id + o
+ };
+
+ KTEST_START_REQ(osd_dev, or);
+ read_bio = bio_map_kern(req_q, read_buff,
+ BUFF_SIZE, GFP_KERNEL);
+ if (!read_bio) {
+ OSD_ERR("!!! Failed to allocate read BIO\n");
+ return -ENOMEM;
+ }
+
+ osd_req_read(or, &obj, read_bio, offset);
+ KTEST_EXEC_END(or, &obj, g_caps, "read");
+ read_bio = NULL;
+ if (memcmp(read_buff, write_buff, BUFF_SIZE))
+ OSD_ERR("!!! Read did not compare");
+ offset += BUFF_SIZE;
+ }
+
+ return 0;
+}
+
+int ktest_remove_obj(struct osd_dev *osd_dev)
+{
+ struct osd_request *or;
+ u8 g_caps[OSD_CAP_LEN];
+ int ret;
+ int p, o;
+
+ for (p = 0; p < num_partitions; p++)
+ for (o = 0; o < num_objects; o++) {
+ struct osd_obj_id obj = {
+ .partition = first_par_id + p,
+ .id = first_obj_id + o
+ };
+
+ KTEST_START_REQ(osd_dev, or);
+ osd_req_remove_object(or, &obj);
+ KTEST_EXEC_END(or, &obj, g_caps, "remove_object");
+ }
+
+ return 0;
+}
+
+int ktest_remove_par(struct osd_dev *osd_dev)
+{
+ struct osd_request *or;
+ u8 g_caps[OSD_CAP_LEN];
+ int ret;
+ int p;
+
+ for (p = 0; p < num_partitions; p++) {
+ struct osd_obj_id par = {
+ .partition = first_par_id + p,
+ .id = 0
+ };
+
+ KTEST_START_REQ(osd_dev, or);
+ osd_req_remove_partition(or, par.partition);
+ KTEST_EXEC_END(or, &par, g_caps, "remove_partition");
+ }
+
+ return 0;
+}
+
+int do_test_17(struct osd_dev *od)
+{
+ void *write_buff = NULL;
+ void *read_buff = NULL;
+ int ret = -ENOMEM, i;
+
+/* osd_format */
+ if (ktest_format(od))
+ goto dev_fini;
+
+/* create some partition */
+ if (ktest_creat_par(od))
+ goto dev_fini;
+/* list partition see if they're all there */
+/* create some objects on some partitions */
+ if (ktest_creat_obj(od))
+ goto dev_fini;
+
+/* Alloc some buffers and bios */
+/* write_buff = kmalloc(BUFF_SIZE, or->alloc_flags);*/
+/* read_buff = kmalloc(BUFF_SIZE, or->alloc_flags);*/
+ write_buff = (void *)__get_free_page(GFP_KERNEL);
+ read_buff = (void *)__get_free_page(GFP_KERNEL);
+ if (!write_buff || !read_buff) {
+ OSD_ERR("!!! Failed to allocate memory for test\n");
+ goto dev_fini;
+ }
+ for (i = 0; i < BUFF_SIZE / 4; i++)
+ ((int *)write_buff)[i] = i;
+ OSD_DEBUG("allocate buffers\n");
+
+/* write to objects */
+ ret = ktest_write_obj(od, write_buff);
+ if (ret)
+ goto dev_fini;
+
+/* read from objects and compare to write */
+ ret = ktest_read_obj(od, write_buff, read_buff);
+ if (ret)
+ goto dev_fini;
+
+/* List all objects */
+
+/* Write with get_attr */
+/* Write with set_attr */
+/* Write with set_attr + get_attr */
+/* Read with set_attr */
+/* Read with get_attr */
+/* Read with get_attr + set_attr */
+/* remove objects */
+ ret = ktest_remove_obj(od);
+ if (ret)
+ goto dev_fini;
+
+/* remove partitions */
+ ret = ktest_remove_par(od);
+ if (ret)
+ goto dev_fini;
+
+/* good and done */
+ OSD_INFO("test17: All good and done\n");
+dev_fini:
+ if (read_buff)
+ free_page((ulong)read_buff);
+ if (write_buff)
+ free_page((ulong)write_buff);
+
+ return ret;
+}
diff --git a/drivers/scsi/osd/osd_ktests.h b/drivers/scsi/osd/osd_ktests.h
index a9e5e00..b625ee5 100644
--- a/drivers/scsi/osd/osd_ktests.h
+++ b/drivers/scsi/osd/osd_ktests.h
@@ -24,4 +24,8 @@
enum { OSD_TEST_ALL = 17 };
+#ifdef __KERNEL__
+extern int do_test_17(struct osd_dev *od);
+#endif /* __KERNEL__ */
+
#endif /*ndef __OSD_KTESTS_H__*/
diff --git a/drivers/scsi/osd/osd_uld.c b/drivers/scsi/osd/osd_uld.c
index 0229301..1ce5eba 100644
--- a/drivers/scsi/osd/osd_uld.c
+++ b/drivers/scsi/osd/osd_uld.c
@@ -127,7 +127,7 @@ static long osd_uld_ioctl(struct file *file, unsigned int cmd,
switch (cmd) {
case OSD_TEST_ALL:
OSD_DEBUG("Kernel test %d: osd_uld_device=%p\n", cmd, oud);
- ret = 0;
+ ret = do_test_17(&oud->od);
break;
default:
OSD_ERR("Unknown osd_uld_ioctl %d\n", cmd);
--
1.6.0.1
next prev parent reply other threads:[~2008-11-04 16:50 UTC|newest]
Thread overview: 85+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <48876009.8010701@panasas.com>
2008-11-04 16:09 ` [PATCHSET 00/18] open-osd: OSD Initiator library for Linux Boaz Harrosh
2008-11-04 16:17 ` [PATCH 01/18] major.h: char-major number for OSD device driver Boaz Harrosh
2008-11-26 17:33 ` [osd-dev] " Boaz Harrosh
2008-11-26 18:07 ` Randy Dunlap
2008-11-04 16:42 ` [PATCH 02/18] scsi: OSD_TYPE Boaz Harrosh
2008-11-04 16:44 ` [PATCH 03/18] libosd: OSDv1 Headers Boaz Harrosh
2008-11-04 19:10 ` Andrew Morton
2008-11-04 19:42 ` Jörn Engel
2008-11-04 20:29 ` Jörn Engel
2008-11-05 13:00 ` Boaz Harrosh
2008-11-05 12:54 ` Boaz Harrosh
2008-11-05 13:09 ` James Bottomley
2008-11-05 13:29 ` Boaz Harrosh
2008-11-09 14:52 ` [PATCH 03/18 ver2] " Boaz Harrosh
2008-11-09 17:45 ` Jörn Engel
2008-11-10 8:47 ` Boaz Harrosh
2008-11-10 15:17 ` Jörn Engel
2008-11-10 17:29 ` Randy Dunlap
2008-11-12 13:10 ` Boaz Harrosh
2008-11-12 16:48 ` Randy Dunlap
2008-11-12 17:09 ` Boaz Harrosh
2008-11-12 17:15 ` Johannes Berg
2008-11-12 13:13 ` [PATCH 03/18 ver3] " Boaz Harrosh
2008-11-12 18:59 ` Randy Dunlap
2008-11-13 9:38 ` Boaz Harrosh
2008-11-13 12:25 ` [PATCH 03/18 ver4] " Boaz Harrosh
2008-11-13 18:16 ` Randy Dunlap
2008-11-13 15:41 ` [osd-dev] " Benny Halevy
2008-11-04 16:44 ` [PATCH 04/18] libosd: OSDv1 preliminary implementation Boaz Harrosh
2008-11-04 18:03 ` Sam Ravnborg
2008-11-05 13:12 ` Boaz Harrosh
2008-11-09 14:55 ` [osd-dev] " Boaz Harrosh
2008-11-10 5:37 ` Randy Dunlap
2008-11-10 9:00 ` Boaz Harrosh
2008-11-05 16:39 ` [Patch] Always include <linux/types.h> Jörn Engel
2008-11-05 17:23 ` Alexey Dobriyan
2008-11-05 19:16 ` Jörn Engel
2008-11-05 19:48 ` Andreas Schwab
2008-11-05 20:02 ` Jörn Engel
2008-11-05 20:32 ` Alexey Dobriyan
2008-11-07 8:02 ` Jörn Engel
2008-11-05 20:20 ` Alexey Dobriyan
2008-11-05 17:48 ` Boaz Harrosh
2008-11-04 19:16 ` [PATCH 04/18] libosd: OSDv1 preliminary implementation Andrew Morton
2008-11-05 13:44 ` Boaz Harrosh
2008-11-09 14:50 ` [PATCH 04/18 ver2] " Boaz Harrosh
2008-11-04 16:44 ` [PATCH 05/18] osd_uld: OSD scsi ULD Boaz Harrosh
2008-11-04 16:44 ` [PATCH 06/18] osd_uld: API for retrieving osd devices from Kernel Boaz Harrosh
2008-11-04 16:44 ` [PATCH 07/18] osd_test: User-mode application to run the OSD tests Boaz Harrosh
2008-11-04 16:44 ` Boaz Harrosh [this message]
2008-11-04 16:44 ` [PATCH 09/18] libosd: attributes Support Boaz Harrosh
2008-11-04 16:44 ` [PATCH 10/18] osd_ktests: Test Attribute lists Boaz Harrosh
2008-11-04 16:44 ` [PATCH 11/18] libosd: OSD Security processing stubs Boaz Harrosh
2008-11-04 16:44 ` [PATCH 12/18] libosd: Add Flush and List-objects support Boaz Harrosh
2008-11-04 16:44 ` [PATCH 13/18] libosd: Not implemented commands Boaz Harrosh
2008-11-04 16:44 ` [PATCH 14/18] libosd: OSD version 2 Support Boaz Harrosh
2008-11-04 16:44 ` [PATCH 15/18] libosd: OSDv2 auto detection Boaz Harrosh
2008-11-04 16:44 ` [PATCH 16/18] osd: Documentation for OSD library Boaz Harrosh
2008-11-04 16:44 ` [PATCH 17/18] osd: Kconfig file for in-tree builds Boaz Harrosh
2008-11-04 16:44 ` [PATCH 18/18] scsi: Add osd library to build system Boaz Harrosh
2008-11-04 19:19 ` [PATCHSET 00/18] open-osd: OSD Initiator library for Linux Andrew Morton
2008-11-05 13:56 ` Boaz Harrosh
2008-11-09 14:58 ` Boaz Harrosh
2008-11-09 23:26 ` Stephen Rothwell
2008-11-10 12:52 ` Boaz Harrosh
2008-11-10 13:04 ` Stephen Rothwell
2008-12-22 12:32 ` Boaz Harrosh
2008-12-22 12:37 ` [PATCH 01/18] major.h: char-major number for OSD device driver Boaz Harrosh
2008-12-22 12:39 ` [PATCH 02/18] scsi: OSD_TYPE Boaz Harrosh
2008-12-22 12:41 ` [PATCH 03/18] libosd: OSDv1 Headers Boaz Harrosh
2008-12-22 12:43 ` [PATCH 04/18] libosd: OSDv1 preliminary implementation Boaz Harrosh
2008-12-22 12:46 ` [PATCH 05/18] osd_uld: OSD scsi ULD Boaz Harrosh
2008-12-22 12:49 ` [PATCH 06/18] osd_uld: API for retrieving osd devices from Kernel Boaz Harrosh
2008-12-22 12:51 ` [PATCH 07/18] osd_ktests: Add basic OSD tests Boaz Harrosh
2008-12-22 12:55 ` [PATCH 08/18] libosd: attributes Support Boaz Harrosh
2008-12-22 12:57 ` [PATCH 09/18] osd_ktests: Test Attribute lists Boaz Harrosh
2008-12-22 13:00 ` [PATCH 10/18] libosd: OSD Security processing stubs Boaz Harrosh
2008-12-22 13:02 ` [PATCH 11/18] libosd: Add Flush and List-objects support Boaz Harrosh
2008-12-22 13:04 ` [PATCH 12/18] libosd: Not implemented commands Boaz Harrosh
2008-12-22 13:07 ` [PATCH 13/18] libosd: OSD version 2 Support Boaz Harrosh
2008-12-22 13:09 ` [PATCH 14/18] libosd: OSDv2 auto detection Boaz Harrosh
2008-12-22 13:13 ` [PATCH 15/18] libosd: SCSI/OSD Sense decoding support Boaz Harrosh
2008-12-22 13:16 ` [PATCH 16/18] osd: Documentation for OSD library Boaz Harrosh
2008-12-22 13:18 ` [PATCH 17/18] osd: Kconfig file for in-tree builds Boaz Harrosh
2008-12-22 13:20 ` [PATCH 18/18] scsi: Add osd library to build system Boaz Harrosh
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=1225817084-6063-1-git-send-email-bharrosh@panasas.com \
--to=bharrosh@panasas.com \
--cc=James.Bottomley@HansenPartnership.com \
--cc=Sami.Iren@seagate.com \
--cc=akpm@linux-foundation.org \
--cc=fujita.tomonori@lab.ntt.co.jp \
--cc=jeff@garzik.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=michaelc@cs.wisc.edu \
--cc=osd-dev@open-osd.org \
--cc=pw@padd.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
all inboxes | Powered by JetHome®