Changeset 3310 in vbox for trunk/src/VBox/Devices

Timestamp:

Jun 27, 2007 10:11:43 AM (17 years ago)

Author:

vboxsync

Message:

don't flood the release log with messages about failed passthrough ATA commands if the device does not exist

File:

• trunk/src/VBox/Devices/Storage/DevATA.cpp (modified) (1 diff)

trunk/src/VBox/Devices/Storage/DevATA.cpp

@@ -1856 +1856 @@
     {
         if (s->cErrors++ < MAX_LOG_REL_ERRORS)
-            LogRel(("PIIX3 ATA: LUN#%d: CD-ROM passthrough command (%#04x) error %d %Vrc\n", s->iLUN, s->aATAPICmd[0], uATAPISenseKey, rc));
+        {
+            uint8_t u8Cmd = s->aATAPICmd[0];
+            do
+            {
+                /* don't log superflous errors */
+                if (    rc == VERR_DEV_IO_ERROR
+                    && (   u8Cmd == SCSI_TEST_UNIT_READY
+                        || u8Cmd == SCSI_READ_CAPACITY
+                        || u8Cmd == SCSI_READ_TOC_PMA_ATIP))
+                    break;
+                LogRel(("PIIX3 ATA: LUN#%d: CD-ROM passthrough command (%#04x) error %d %Vrc\n", s->iLUN, u8Cmd, uATAPISenseKey, rc));
+            } while (0);
+        }
         atapiCmdError(s, uATAPISenseKey, 0);
         /* This is a drive-reported error. atapiCmdError() sets both the error