TESTS: improve verify tests to detect/abort on opcode not supported furthter
This commit is contained in:
@@ -28,7 +28,7 @@ int T0131_verify10_mismatch(const char *initiator, const char *url, int data_los
|
||||
struct scsi_task *vtask;
|
||||
struct scsi_readcapacity10 *rc10;
|
||||
int ret, i, lun;
|
||||
uint32_t block_size, num_blocks;
|
||||
uint32_t block_size;
|
||||
|
||||
printf("0131_verify10_mismatch:\n");
|
||||
printf("=======================\n");
|
||||
@@ -66,7 +66,6 @@ int T0131_verify10_mismatch(const char *initiator, const char *url, int data_los
|
||||
goto finished;
|
||||
}
|
||||
block_size = rc10->block_size;
|
||||
num_blocks = rc10->lba;
|
||||
scsi_free_scsi_task(task);
|
||||
|
||||
|
||||
@@ -83,14 +82,14 @@ int T0131_verify10_mismatch(const char *initiator, const char *url, int data_los
|
||||
printf("[FAILED]\n");
|
||||
printf("Failed to send read10 command: %s\n", iscsi_get_error(iscsi));
|
||||
ret = -1;
|
||||
goto finished;
|
||||
goto test2;
|
||||
}
|
||||
if (task->status != SCSI_STATUS_GOOD) {
|
||||
printf("[FAILED]\n");
|
||||
printf("Read10 command: failed with sense. %s\n", iscsi_get_error(iscsi));
|
||||
ret = -1;
|
||||
scsi_free_scsi_task(task);
|
||||
goto finished;
|
||||
goto test2;
|
||||
}
|
||||
|
||||
buf = task->datain.data;
|
||||
@@ -99,7 +98,7 @@ int T0131_verify10_mismatch(const char *initiator, const char *url, int data_los
|
||||
printf("Failed to access DATA-IN buffer %s\n", iscsi_get_error(iscsi));
|
||||
ret = -1;
|
||||
scsi_free_scsi_task(task);
|
||||
goto finished;
|
||||
goto test2;
|
||||
}
|
||||
/* flip a random byte in the data */
|
||||
buf[random() % task->datain.size] ^= 'X';
|
||||
@@ -110,6 +109,15 @@ int T0131_verify10_mismatch(const char *initiator, const char *url, int data_los
|
||||
printf("Failed to send verify10 command: %s\n", iscsi_get_error(iscsi));
|
||||
ret = -1;
|
||||
scsi_free_scsi_task(task);
|
||||
goto test2;
|
||||
}
|
||||
if (vtask->status == SCSI_STATUS_CHECK_CONDITION
|
||||
&& vtask->sense.key == SCSI_SENSE_ILLEGAL_REQUEST
|
||||
&& vtask->sense.ascq == SCSI_SENSE_ASCQ_INVALID_OPERATION_CODE) {
|
||||
printf("[SKIPPED]\n");
|
||||
printf("Opcode is not implemented on target\n");
|
||||
scsi_free_scsi_task(task);
|
||||
scsi_free_scsi_task(vtask);
|
||||
goto finished;
|
||||
}
|
||||
if (vtask->status == SCSI_STATUS_GOOD) {
|
||||
@@ -118,7 +126,7 @@ int T0131_verify10_mismatch(const char *initiator, const char *url, int data_los
|
||||
ret = -1;
|
||||
scsi_free_scsi_task(task);
|
||||
scsi_free_scsi_task(vtask);
|
||||
goto finished;
|
||||
goto test2;
|
||||
}
|
||||
|
||||
if (vtask->sense.key != SCSI_SENSE_MISCOMPARE) {
|
||||
@@ -127,7 +135,7 @@ int T0131_verify10_mismatch(const char *initiator, const char *url, int data_los
|
||||
ret = -1;
|
||||
scsi_free_scsi_task(task);
|
||||
scsi_free_scsi_task(vtask);
|
||||
goto finished;
|
||||
goto test2;
|
||||
}
|
||||
|
||||
scsi_free_scsi_task(task);
|
||||
@@ -135,6 +143,8 @@ int T0131_verify10_mismatch(const char *initiator, const char *url, int data_los
|
||||
}
|
||||
printf("[OK]\n");
|
||||
|
||||
test2:
|
||||
|
||||
finished:
|
||||
iscsi_logout_sync(iscsi);
|
||||
iscsi_destroy_context(iscsi);
|
||||
|
||||
Reference in New Issue
Block a user