From ec7d33e537e50b5dd5c1cee8ad62ead48e0fd2bc Mon Sep 17 00:00:00 2001
From: Vladislav Bolkhovitin <vst@vlnb.net>
Date: Tue, 18 Nov 2008 18:42:09 +0000
Subject: [PATCH] Log level for "Warning: expected transfer length X for opcode
 Y" changed to be disabled by default for release build

git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@572 d57e44dd-8a1f-0410-8b47-8ef2f437770f
---
 scst/src/scst_targ.c | 30 +++++++++++-------------------
 1 file changed, 11 insertions(+), 19 deletions(-)

diff --git a/scst/src/scst_targ.c b/scst/src/scst_targ.c
index e35a4abfe..327dcbd3c 100644
--- a/scst/src/scst_targ.c
+++ b/scst/src/scst_targ.c
@@ -594,25 +594,17 @@ static int scst_parse_cmd(struct scst_cmd *cmd)
 			}
 		}
 		if (unlikely(cmd->bufflen != cmd->expected_transfer_len)) {
-			static int repd;
-			if (repd < 100) {
-				/*
-				 * Intentionally unlocked. Few messages more
-				 * or less don't matter.
-				 */
-				repd++;
-				TRACE(TRACE_MINOR, "Warning: expected transfer "
-					"length %d for opcode 0x%02x (handler "
-					"%s, target %s) doesn't match decoded "
-					"value %d. Faulty initiator (e.g. "
-					"VMware is known to be such) or "
-					"scst_scsi_op_table should be updated?",
-					cmd->expected_transfer_len, cmd->cdb[0],
-					dev->handler->name, cmd->tgtt->name,
-					cmd->bufflen);
-				PRINT_BUFF_FLAG(TRACE_MINOR, "Suspicious CDB",
-					cmd->cdb, cmd->cdb_len);
-			}
+			TRACE(TRACE_MGMT_MINOR, "Warning: expected "
+				"transfer length %d for opcode 0x%02x "
+				"(handler %s, target %s) doesn't match "
+				"decoded value %d. Faulty initiator "
+				"(e.g. VMware is known to be such) or "
+				"scst_scsi_op_table should be updated?",
+				cmd->expected_transfer_len, cmd->cdb[0],
+				dev->handler->name, cmd->tgtt->name,
+				cmd->bufflen);
+			PRINT_BUFF_FLAG(TRACE_MINOR, "Suspicious CDB",
+				cmd->cdb, cmd->cdb_len);
 		}
 #endif
 	}