Changeset 69 for trunk/src/os2ahci/os2ahci.c

Timestamp:

Jan 7, 2011, 4:54:46 PM (15 years ago)

Author:

markus

Message:

fixed ATAPI packet commands; added cache flush in shutdown handler; allow write cache for NCQ commands

File:

• trunk/src/os2ahci/os2ahci.c (modified) (3 diffs)

trunk/src/os2ahci/os2ahci.c

@@ -107 +107 @@
     break;
 
+  case CMDShutdown:
+    rc = exit_drv(((RPSAVERESTORE _far *) req)->FuncCode);
+    break;
+
   default:
     rc = STDON | STATUS_ERR_UNKCMD;
@@ -284 +288 @@
   cprintf(exit_msg);
   return(STDON | ERROR_I24_QUIET_INIT_FAIL);
+}
+
+/******************************************************************************
+ * Device driver exit handler. This handler is called when OS/2 shuts down and
+ * flushes the write caches of all attached devices.
+ *
+ * NOTE: Errors are ignored because there's no way we could stop the shutdown
+ *       or do something about the error, unless retrying endlessly is
+ *       considered an option.
+ */
+USHORT exit_drv(int func)
+{
+  int a;
+  int p;
+  int d;
+
+  dprintf("exit_drv(%d) called\n", func);
+
+  if (func == 0) {
+    /* we're only interested in the second phase of the shutdown */
+    return(STDON);
+  }
+
+  for (a = 0; a < ad_info_cnt; a++) {
+    AD_INFO *ai = ad_infos + a;
+
+    for (p = 0; p <= ai->port_max; p++) {
+      for (d = 0; d <= ai->ports[p].dev_max; d++) {
+        if (ai->ports[p].devs[d].present) {
+          ahci_flush_cache(ai, p, d);
+        }
+      }
+    }
+  }
+
+  return(STDON);
 }
 
@@ -1035 +1075 @@
 
 /******************************************************************************
+ * Timeout handler for I/O commands. Since timeout handling can involve
+ * lengthy operations like port resets, the main code is located in a
+ * separate function which is invoked via a context hook.
+ */
+void _cdecl _far timeout_callback(ULONG timer_handle, ULONG p1,
+                                  ULONG p2)
+{
+  IORBH _far *iorb = (IORBH _far *) p1;
+  int a = iorb_unit_adapter(iorb);
+  int p = iorb_unit_port(iorb);
+
+  ADD_CancelTimer(timer_handle);
+  dprintf("timeout for IORB %Fp\n", iorb);
+
+  /* Move the timed-out IORB to the abort queue. Since it's possible that the
+   * IORB has completed after the timeout has expired but before we got to
+   * this line of code, we'll check the return code of iorb_queue_del(): If it
+   * returns an error, the IORB must have completed a few microseconds ago and
+   * there is no timeout.
+   */
+  spin_lock(drv_lock);
+  if (iorb_queue_del(&ad_infos[a].ports[p].iorb_queue, iorb) == 0) {
+    iorb_queue_add(&abort_queue, iorb);
+    iorb->ErrorCode = IOERR_ADAPTER_TIMEOUT;
+  }
+  spin_unlock(drv_lock);
+
+  /* Trigger abort processing function. We don't really care whether this
+   * succeeds because the only reason why it would fail should be multiple
+   * calls to DevHelp_ArmCtxHook() before the context hook had a chance to
+   * start executing, which leaves two scenarios:
+   *
+   *  - We succeded in arming the context hook. Fine.
+   *
+   *  - We armed the context hook a second time before it had a chance to
+   *    start executing. In this case, the already scheduled context hook
+   *    will process our IORB as well.
+   */
+  DevHelp_ArmCtxHook(0, reset_ctxhook_h);
+}
+
+/******************************************************************************
  * small_code_ - this dummy func resolves the undefined reference linker
  * error that occurrs when linking WATCOM objects with DDK's link.exe