[Scst-svn] SF.net SVN: scst:[572] trunk/scst/src/scst_targ.c

[<vl...@us...>]

Revision: 572
          http://scst.svn.sourceforge.net/scst/?rev=572&view=rev
Author:   vlnb
Date:     2008-11-18 18:42:09 +0000 (Tue, 18 Nov 2008)

Log Message:
-----------
Log level for "Warning: expected transfer length X for opcode Y" changed to be disabled by default for release build

Modified Paths:
--------------
    trunk/scst/src/scst_targ.c

Modified: trunk/scst/src/scst_targ.c
===================================================================
--- trunk/scst/src/scst_targ.c	2008-11-18 18:23:08 UTC (rev 571)
+++ trunk/scst/src/scst_targ.c	2008-11-18 18:42:09 UTC (rev 572)
@@ -594,25 +594,17 @@
 			}
 		}
 		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
 	}


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.