From 575ba2244c5db240bc29fa493404eee39f693a2d Mon Sep 17 00:00:00 2001 From: David Disseldorp Date: Mon, 16 Sep 2019 14:46:28 +0200 Subject: [PATCH 04/12] test-tool/iSCSICmdSnTooHigh: override driver queue_pdu() callback In preparation for removing the test-tool iscsi_queue_pdu() dlsym overload hack, convert this test to use the regular transport driver callback for this purpose. Signed-off-by: David Disseldorp --- test-tool/test_iscsi_cmdsn_toohigh.c | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/test-tool/test_iscsi_cmdsn_toohigh.c b/test-tool/test_iscsi_cmdsn_toohigh.c index 59ce0ab..19c8b91 100644 --- a/test-tool/test_iscsi_cmdsn_toohigh.c +++ b/test-tool/test_iscsi_cmdsn_toohigh.c @@ -25,6 +25,7 @@ #include "iscsi-test-cu.h" static int change_cmdsn; +static struct iscsi_transport iscsi_drv_orig; static int my_iscsi_queue_pdu(struct iscsi_context *iscsi, struct iscsi_pdu *pdu) { @@ -40,7 +41,7 @@ static int my_iscsi_queue_pdu(struct iscsi_context *iscsi, struct iscsi_pdu *pdu } change_cmdsn = 0; - return 0; + return iscsi_drv_orig.queue_pdu(iscsi, pdu); } void test_iscsi_cmdsn_toohigh(void) @@ -64,7 +65,9 @@ void test_iscsi_cmdsn_toohigh(void) sd->iscsi_ctx->use_immediate_data = ISCSI_IMMEDIATE_DATA_NO; sd->iscsi_ctx->target_max_recv_data_segment_length = block_size; - local_iscsi_queue_pdu = my_iscsi_queue_pdu; + /* override transport queue_pdu callback for PDU manipulation */ + iscsi_drv_orig = *sd->iscsi_ctx->drv; + sd->iscsi_ctx->drv->queue_pdu = my_iscsi_queue_pdu; change_cmdsn = 1; /* we don't want autoreconnect since some targets will incorrectly * drop the connection on this condition. @@ -81,9 +84,11 @@ void test_iscsi_cmdsn_toohigh(void) logging(LOG_VERBOSE, "[FAILURE] We got a response from the target but SMDSN was outside of the window."); } - iscsi_set_noautoreconnect(sd->iscsi_ctx, 0); logging(LOG_VERBOSE, "Send a TESTUNITREADY with CMDSN == EXPCMDSN. should work again"); TESTUNITREADY(sd, EXPECT_STATUS_GOOD); + + /* restore transport callbacks */ + *(sd->iscsi_ctx->drv) = iscsi_drv_orig; } -- 1.8.3.1