iscsi-test: return -2 when skipping tests

This lets a test driver distinguish failure from skipping unsupported
tests.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>

test-tool/0103_read10_rdprotect.c

@@ -72,13 +72,13 @@ int T0103_read10_rdprotect(const char *initiator, const char *url, int data_loss
 	if (inq->periperal_device_type != SCSI_INQUIRY_PERIPHERAL_DEVICE_TYPE_DIRECT_ACCESS) {
 		printf("LUN is not SBC device. Skipping test\n");
 		scsi_free_scsi_task(task);
-		return -1;
+		return -2;
 	}
 
 	if (inq->protect) {
 		printf("LUN is formatted with protection information. Skipping test\n");
 		scsi_free_scsi_task(task);
-		return -1;
+		return -2;
 	}
 
 	scsi_free_scsi_task(task);

test-tool/0104_read10_flags.c

@@ -66,7 +66,7 @@ int T0104_read10_flags(const char *initiator, const char *url, int data_loss _U_
 	if (inq->periperal_device_type != SCSI_INQUIRY_PERIPHERAL_DEVICE_TYPE_DIRECT_ACCESS) {
 		printf("LUN is not SBC device. Skipping test\n");
 		scsi_free_scsi_task(task);
-		return -1;
+		return -2;
 	}
 
 	/* Try out DPO : 1 */

test-tool/0170_unmap_simple.c

@@ -67,7 +67,7 @@ int T0170_unmap_simple(const char *initiator, const char *url, int data_loss, in
 
 	if (rc16->lbpme == 0){
 		printf("Logical unit is fully provisioned. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		scsi_free_scsi_task(task);
 		goto finished;
 	}
@@ -79,7 +79,7 @@ int T0170_unmap_simple(const char *initiator, const char *url, int data_loss, in
 
 	if (!data_loss) {
 		printf("data_loss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 	

test-tool/0171_unmap_zero.c

@@ -69,7 +69,7 @@ int T0171_unmap_zero(const char *initiator, const char *url, int data_loss, int
 
 	if (rc16->lbpme == 0){
 		printf("Logical unit is fully provisioned. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		scsi_free_scsi_task(task);
 		goto finished;
 	}
@@ -81,7 +81,7 @@ int T0171_unmap_zero(const char *initiator, const char *url, int data_loss, int
 
 	if (!data_loss) {
 		printf("data_loss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 	

test-tool/0180_writesame10_unmap.c

@@ -140,7 +140,7 @@ test2:
 	
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0181_writesame10_unmap_unaligned.c

@@ -67,7 +67,7 @@ int T0181_writesame10_unmap_unaligned(const char *initiator, const char *url, in
 
 	if (rc16->lbpme == 0){
 		printf("Logical unit is fully provisioned. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		scsi_free_scsi_task(task);
 		goto finished;
 	}
@@ -86,7 +86,7 @@ int T0181_writesame10_unmap_unaligned(const char *initiator, const char *url, in
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 	

test-tool/0182_writesame10_beyondeol.c

@@ -73,7 +73,7 @@ int T0182_writesame10_beyondeol(const char *initiator, const char *url, int data
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0183_writesame10_wrprotect.c

@@ -71,7 +71,7 @@ int T0183_writesame10_wrprotect(const char *initiator, const char *url, int data
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0184_writesame10_0blocks.c

@@ -75,7 +75,7 @@ int T0184_writesame10_0blocks(const char *initiator, const char *url, int data_l
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0190_writesame16_unmap.c

@@ -140,7 +140,7 @@ test2:
 	
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0191_writesame16_unmap_unaligned.c

@@ -67,7 +67,7 @@ int T0191_writesame16_unmap_unaligned(const char *initiator, const char *url, in
 
 	if (rc16->lbpme == 0){
 		printf("Logical unit is fully provisioned. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		scsi_free_scsi_task(task);
 		goto finished;
 	}
@@ -86,7 +86,7 @@ int T0191_writesame16_unmap_unaligned(const char *initiator, const char *url, in
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 	

test-tool/0192_writesame16_beyondeol.c

@@ -73,7 +73,7 @@ int T0192_writesame16_beyondeol(const char *initiator, const char *url, int data
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0193_writesame16_wrprotect.c

@@ -71,7 +71,7 @@ int T0193_writesame16_wrprotect(const char *initiator, const char *url, int data
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0194_writesame16_0blocks.c

@@ -75,7 +75,7 @@ int T0194_writesame16_0blocks(const char *initiator, const char *url, int data_l
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0201_read16_rdprotect.c

@@ -69,6 +69,7 @@ int T0201_read16_rdprotect(const char *initiator, const char *url, int data_loss
 	if(rc16->prot_en != 0) {
 		printf("device is formatted with protection information, skipping test\n");
 		scsi_free_scsi_task(task);
+		ret = -2;
 		goto finished;
 	}
 	scsi_free_scsi_task(task);

test-tool/0202_read16_flags.c

@@ -62,7 +62,7 @@ int T0202_read16_flags(const char *initiator, const char *url, int data_loss _U_
 	if (inq->periperal_device_type != SCSI_INQUIRY_PERIPHERAL_DEVICE_TYPE_DIRECT_ACCESS) {
 		printf("LUN is not SBC device. Skipping test\n");
 		scsi_free_scsi_task(task);
-		return -1;
+		return -2;
 	}
 
 	/* find the size of the LUN */

test-tool/0211_read12_rdprotect.c

@@ -69,6 +69,7 @@ int T0211_read12_rdprotect(const char *initiator, const char *url, int data_loss
 	if(rc16->prot_en != 0) {
 		printf("device is formatted with protection information, skipping test\n");
 		scsi_free_scsi_task(task);
+		ret = -2;
 		goto finished;
 	}
 	scsi_free_scsi_task(task);

test-tool/0212_read12_flags.c

@@ -62,7 +62,7 @@ int T0212_read12_flags(const char *initiator, const char *url, int data_loss _U_
 	if (inq->periperal_device_type != SCSI_INQUIRY_PERIPHERAL_DEVICE_TYPE_DIRECT_ACCESS) {
 		printf("LUN is not SBC device. Skipping test\n");
 		scsi_free_scsi_task(task);
-		return -1;
+		return -2;
 	}
 
 	/* find the size of the LUN */

test-tool/0220_write16_simple.c

@@ -73,7 +73,7 @@ int T0220_write16_simple(const char *initiator, const char *url, int data_loss,
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 	

test-tool/0221_write16_wrprotect.c

@@ -70,13 +70,14 @@ int T0221_write16_wrprotect(const char *initiator, const char *url, int data_los
 	if(rc16->prot_en != 0) {
 		printf("device is formatted with protection information, skipping test\n");
 		scsi_free_scsi_task(task);
+		ret = -2;
 		goto finished;
 	}
 	scsi_free_scsi_task(task);
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0222_write16_flags.c

@@ -63,7 +63,7 @@ int T0222_write16_flags(const char *initiator, const char *url, int data_loss, i
 	if (inq->periperal_device_type != SCSI_INQUIRY_PERIPHERAL_DEVICE_TYPE_DIRECT_ACCESS) {
 		printf("LUN is not SBC device. Skipping test\n");
 		scsi_free_scsi_task(task);
-		return -1;
+		return -2;
 	}
 
 	/* find the size of the LUN */
@@ -110,7 +110,7 @@ int T0222_write16_flags(const char *initiator, const char *url, int data_loss, i
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0223_write16_0blocks.c

@@ -75,7 +75,7 @@ int T0223_write16_0blocks(const char *initiator, const char *url, int data_loss,
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0224_write16_beyondeol.c

@@ -71,7 +71,7 @@ int T0224_write16_beyondeol(const char *initiator, const char *url, int data_los
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0230_write12_simple.c

@@ -73,7 +73,7 @@ int T0230_write12_simple(const char *initiator, const char *url, int data_loss,
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 	

test-tool/0231_write12_wrprotect.c

@@ -70,13 +70,14 @@ int T0231_write12_wrprotect(const char *initiator, const char *url, int data_los
 	if(rc16->prot_en != 0) {
 		printf("device is formatted with protection information, skipping test\n");
 		scsi_free_scsi_task(task);
+		ret = -2;
 		goto finished;
 	}
 	scsi_free_scsi_task(task);
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0232_write12_flags.c

@@ -63,7 +63,7 @@ int T0232_write12_flags(const char *initiator, const char *url, int data_loss, i
 	if (inq->periperal_device_type != SCSI_INQUIRY_PERIPHERAL_DEVICE_TYPE_DIRECT_ACCESS) {
 		printf("LUN is not SBC device. Skipping test\n");
 		scsi_free_scsi_task(task);
-		return -1;
+		return -2;
 	}
 
 	/* find the size of the LUN */
@@ -110,7 +110,7 @@ int T0232_write12_flags(const char *initiator, const char *url, int data_loss, i
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0233_write12_0blocks.c

@@ -75,7 +75,7 @@ int T0233_write12_0blocks(const char *initiator, const char *url, int data_loss,
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0234_write12_beyondeol.c

@@ -71,7 +71,7 @@ int T0234_write12_beyondeol(const char *initiator, const char *url, int data_los
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0260_get_lba_status_simple.c

@@ -68,7 +68,7 @@ int T0260_get_lba_status_simple(const char *initiator, const char *url, int data
 
 	if (rc16->lbpme == 0){
 		printf("Logical unit is fully provisioned. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		scsi_free_scsi_task(task);
 		goto finished;
 	}

test-tool/0264_get_lba_status_beyondeol.c

@@ -67,7 +67,7 @@ int T0264_get_lba_status_beyondeol(const char *initiator, const char *url, int d
 
 	if (rc16->lbpme == 0){
 		printf("Logical unit is fully provisioned. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		scsi_free_scsi_task(task);
 		goto finished;
 	}

test-tool/0290_write10_simple.c

@@ -73,7 +73,7 @@ int T0290_write10_simple(const char *initiator, const char *url, int data_loss,
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 	

test-tool/0291_write10_wrprotect.c

@@ -70,13 +70,14 @@ int T0291_write10_wrprotect(const char *initiator, const char *url, int data_los
 	if(rc16->prot_en != 0) {
 		printf("device is formatted with protection information, skipping test\n");
 		scsi_free_scsi_task(task);
+		ret = -2;
 		goto finished;
 	}
 	scsi_free_scsi_task(task);
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0292_write10_flags.c

@@ -63,7 +63,7 @@ int T0292_write10_flags(const char *initiator, const char *url, int data_loss, i
 	if (inq->periperal_device_type != SCSI_INQUIRY_PERIPHERAL_DEVICE_TYPE_DIRECT_ACCESS) {
 		printf("LUN is not SBC device. Skipping test\n");
 		scsi_free_scsi_task(task);
-		return -1;
+		return -2;
 	}
 
 	/* find the size of the LUN */
@@ -110,7 +110,7 @@ int T0292_write10_flags(const char *initiator, const char *url, int data_loss, i
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0293_write10_0blocks.c

@@ -75,7 +75,7 @@ int T0293_write10_0blocks(const char *initiator, const char *url, int data_loss,
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0294_write10_beyondeol.c

@@ -71,7 +71,7 @@ int T0294_write10_beyondeol(const char *initiator, const char *url, int data_los
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0300_readonly.c

@@ -89,6 +89,7 @@ int T0300_readonly(const char *initiator, const char *url, int data_loss, int sh
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
+		ret = -2;
 		goto finished;
 	}
 
@@ -137,6 +138,7 @@ int T0300_readonly(const char *initiator, const char *url, int data_loss, int sh
 	}
 	if (!(ms->device_specific_parameter & 0x80)) {
 		printf("Device is not read-only. Skipping test\n");
+		ret = -2;
 		goto finished;
 	}
 	scsi_free_scsi_task(task);

test-tool/0310_writeverify10_simple.c

@@ -73,7 +73,7 @@ int T0310_writeverify10_simple(const char *initiator, const char *url, int data_
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 	

test-tool/0311_writeverify10_wrprotect.c

@@ -69,6 +69,7 @@ int T0311_writeverify10_wrprotect(const char *initiator, const char *url, int da
 
 	if(rc16->prot_en != 0) {
 		printf("device is formatted with protection information, skipping test\n");
+		ret = -2;
 		scsi_free_scsi_task(task);
 		goto finished;
 	}
@@ -76,7 +77,7 @@ int T0311_writeverify10_wrprotect(const char *initiator, const char *url, int da
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0314_writeverify10_beyondeol.c

@@ -74,7 +74,7 @@ int T0314_writeverify10_beyondeol(const char *initiator, const char *url, int da
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0320_writeverify12_simple.c

@@ -73,7 +73,7 @@ int T0320_writeverify12_simple(const char *initiator, const char *url, int data_
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 	

test-tool/0321_writeverify12_wrprotect.c

@@ -70,13 +70,14 @@ int T0321_writeverify12_wrprotect(const char *initiator, const char *url, int da
 	if(rc16->prot_en != 0) {
 		printf("device is formatted with protection information, skipping test\n");
 		scsi_free_scsi_task(task);
+		ret = -2;
 		goto finished;
 	}
 	scsi_free_scsi_task(task);
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0324_writeverify12_beyondeol.c

@@ -74,7 +74,7 @@ int T0324_writeverify12_beyondeol(const char *initiator, const char *url, int da
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0330_writeverify16_simple.c

@@ -73,7 +73,7 @@ int T0330_writeverify16_simple(const char *initiator, const char *url, int data_
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 	

test-tool/0331_writeverify16_wrprotect.c

@@ -70,13 +70,14 @@ int T0331_writeverify16_wrprotect(const char *initiator, const char *url, int da
 	if(rc16->prot_en != 0) {
 		printf("device is formatted with protection information, skipping test\n");
 		scsi_free_scsi_task(task);
+		ret = -2;
 		goto finished;
 	}
 	scsi_free_scsi_task(task);
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0334_writeverify16_beyondeol.c

@@ -74,7 +74,7 @@ int T0334_writeverify16_beyondeol(const char *initiator, const char *url, int da
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0340_compareandwrite_simple.c

@@ -74,7 +74,7 @@ int T0340_compareandwrite_simple(const char *initiator, const char *url, int dat
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 	

test-tool/0341_compareandwrite_mismatch.c

@@ -74,7 +74,7 @@ int T0341_compareandwrite_mismatch(const char *initiator, const char *url, int d
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 	

test-tool/0343_compareandwrite_beyondeol.c

@@ -74,7 +74,7 @@ int T0343_compareandwrite_beyondeol(const char *initiator, const char *url, int
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0350_orwrite_simple.c

@@ -76,7 +76,7 @@ int T0350_orwrite_simple(const char *initiator, const char *url, int data_loss,
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 	

test-tool/0351_orwrite_wrprotect.c

@@ -70,13 +70,14 @@ int T0351_orwrite_wrprotect(const char *initiator, const char *url, int data_los
 	if(rc16->prot_en != 0) {
 		printf("device is formatted with protection information, skipping test\n");
 		scsi_free_scsi_task(task);
+		ret = -2;
 		goto finished;
 	}
 	scsi_free_scsi_task(task);
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0354_orwrite_beyondeol.c

@@ -74,7 +74,7 @@ int T0354_orwrite_beyondeol(const char *initiator, const char *url, int data_los
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0360_startstopunit_simple.c

@@ -100,7 +100,7 @@ int T0360_startstopunit_simple(const char *initiator, const char *url, int data_
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 	

test-tool/0361_startstopunit_pwrcnd.c

@@ -100,7 +100,7 @@ int T0361_startstopunit_pwrcnd(const char *initiator, const char *url, int data_
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 	

test-tool/0370_nomedia.c

@@ -131,6 +131,7 @@ int T0370_nomedia(const char *initiator, const char *url, int data_loss, int sho
 
 	if (!removable) {
 		printf("Media is not removable. Skipping test.\n");
+		ret = -2;
 		goto finished;
 	}
 
@@ -538,6 +539,7 @@ test16:
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping all WRITE tests\n");
+		ret = -2;
 		goto cleanup;
 	}
 

test-tool/0380_preventallow_simple.c

@@ -100,7 +100,7 @@ int T0380_preventallow_simple(const char *initiator, const char *url, int data_l
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 	

test-tool/0381_preventallow_eject.c

@@ -102,7 +102,7 @@ int T0381_preventallow_eject(const char *initiator, const char *url, int data_lo
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 	
@@ -112,6 +112,7 @@ int T0381_preventallow_eject(const char *initiator, const char *url, int data_lo
 
 	if (!removable) {
 		printf("Media is not removable. Skipping tests\n");
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0382_preventallow_itnexus_loss.c

@@ -104,7 +104,7 @@ int T0382_preventallow_itnexus_loss(const char *initiator, const char *url, int
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 	
@@ -114,6 +114,7 @@ int T0382_preventallow_itnexus_loss(const char *initiator, const char *url, int
 
 	if (!removable) {
 		printf("Media is not removable. Skipping tests\n");
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0383_preventallow_target_warm_reset.c

@@ -122,7 +122,7 @@ int T0383_preventallow_target_warm_reset(const char *initiator, const char *url,
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 	
@@ -132,6 +132,7 @@ int T0383_preventallow_target_warm_reset(const char *initiator, const char *url,
 
 	if (!removable) {
 		printf("Media is not removable. Skipping tests\n");
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0384_preventallow_target_cold_reset.c

@@ -122,7 +122,7 @@ int T0384_preventallow_target_cold_reset(const char *initiator, const char *url,
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 	
@@ -132,6 +132,7 @@ int T0384_preventallow_target_cold_reset(const char *initiator, const char *url,
 
 	if (!removable) {
 		printf("Media is not removable. Skipping tests\n");
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0385_preventallow_lun_reset.c

@@ -122,7 +122,7 @@ int T0385_preventallow_lun_reset(const char *initiator, const char *url, int dat
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 	
@@ -132,6 +132,7 @@ int T0385_preventallow_lun_reset(const char *initiator, const char *url, int dat
 
 	if (!removable) {
 		printf("Media is not removable. Skipping tests\n");
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0386_preventallow_2_it_nexuses.c

@@ -105,7 +105,7 @@ int T0386_preventallow_2_itl_nexuses(const char *initiator, const char *url, int
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 	
@@ -115,6 +115,7 @@ int T0386_preventallow_2_itl_nexuses(const char *initiator, const char *url, int
 
 	if (!removable) {
 		printf("Media is not removable. Skipping tests\n");
+		ret = -2;
 		goto finished;
 	}
 

test-tool/0390_mandatory_opcodes_sbc.c

@@ -119,6 +119,7 @@ int T0390_mandatory_opcodes_sbc(const char *initiator, const char *url, int data
 	if (inq->periperal_device_type != SCSI_INQUIRY_PERIPHERAL_DEVICE_TYPE_DIRECT_ACCESS) {
 		printf("Not a SBC device. Skipping test\n");
 		scsi_free_scsi_task(task);
+		ret = -2;
 		goto finished;
 	}
 
@@ -126,7 +127,7 @@ int T0390_mandatory_opcodes_sbc(const char *initiator, const char *url, int data
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 	

test-tool/1000_cmdsn_invalid.c

@@ -110,7 +110,7 @@ int T1000_cmdsn_invalid(const char *initiator, const char *url, int data_loss, i
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/1010_datasn_invalid.c

@@ -120,7 +120,7 @@ int T1010_datasn_invalid(const char *initiator, const char *url, int data_loss,
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}
 

test-tool/1020_bufferoffset_invalid.c

@@ -111,7 +111,7 @@ int T1020_bufferoffset_invalid(const char *initiator, const char *url, int data_
 
 	if (!data_loss) {
 		printf("--dataloss flag is not set. Skipping test\n");
-		ret = -1;
+		ret = -2;
 		goto finished;
 	}