FUJITA Tomonori wrote: > On Tue, 6 Jan 2009 16:17:16 +0200 > Boaz Harrosh <bharrosh@xxxxxxxxxxx> wrote: > >> 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@xxxxxxxxxxx> >> Reviewed-by: Benny Halevy <bhalevy@xxxxxxxxxxx> >> --- >> 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 4730cdc..6fb823f 100644 >> --- a/drivers/scsi/osd/Kbuild >> +++ b/drivers/scsi/osd/Kbuild >> @@ -35,5 +35,5 @@ libosd-y := osd_initiator.o >> obj-$(CONFIG_SCSI_OSD_INITIATOR) += libosd.o >> >> # osd.ko - SCSI ULD and char-device >> -osd-y := osd_uld.o >> +osd-y := 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..fba4934 >> --- /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 > > Why do we need in-kernel test suite? It must be in kernel, because it tests the in-kernel library. However it could be in it's own Module, which makes it more complicated, but doable. Please note that the submitted tests are the bare minimum and will be enhanced in the future. Perhaps later when the test grow I will put them in their own module. The test is most valuable when bringing up a new target/initiator to check for compatibility. I'll send a patch ASAP that adds an option to Kconfigure it out of compilation. Thanks Boaz -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
