Sync mly(4) with FreeBSD.
authorPeter Avalos <pavalos@dragonflybsd.org>
Mon, 21 Jan 2008 02:27:37 +0000 (02:27 +0000)
committerPeter Avalos <pavalos@dragonflybsd.org>
Mon, 21 Jan 2008 02:27:37 +0000 (02:27 +0000)
Obtained-from: FreeBSD

sys/conf/files
sys/dev/raid/mly/Makefile
sys/dev/raid/mly/mly.c
sys/dev/raid/mly/mly_cam.c [deleted file]
sys/dev/raid/mly/mly_pci.c [deleted file]
sys/dev/raid/mly/mly_tables.h
sys/dev/raid/mly/mlyio.h
sys/dev/raid/mly/mlyreg.h
sys/dev/raid/mly/mlyvar.h

index 6bc2a4b..dc987d9 100644 (file)
@@ -1,5 +1,5 @@
 # $FreeBSD: src/sys/conf/files,v 1.340.2.137 2003/06/04 17:10:30 sam Exp $
-# $DragonFly: src/sys/conf/files,v 1.199 2008/01/18 19:06:29 dillon Exp $
+# $DragonFly: src/sys/conf/files,v 1.200 2008/01/21 02:27:37 pavalos Exp $
 #
 # The long compile-with and dependency lines are required because of
 # limitations in config: backslash-newline doesn't work in strings, and
@@ -310,8 +310,6 @@ dev/raid/mlx/mlx_disk.c             optional mlx
 dev/raid/mlx/mlx_pci.c         optional mlx
 dev/raid/mlx/mlx.c             optional mlx
 dev/raid/mly/mly.c             optional mly
-dev/raid/mly/mly_cam.c         optional mly
-dev/raid/mly/mly_pci.c         optional mly
 dev/disk/mpt/mpt.c             optional mpt
 dev/disk/mpt/mpt_debug.c       optional mpt
 dev/disk/mpt/mpt_freebsd.c     optional mpt
index b9ef39a..0cbb0d8 100644 (file)
@@ -1,12 +1,12 @@
 # $FreeBSD: src/sys/modules/mly/Makefile,v 1.1.2.1 2000/10/27 10:07:02 msmith Exp $
-# $DragonFly: src/sys/dev/raid/mly/Makefile,v 1.2 2003/06/17 04:28:45 dillon Exp $
+# $DragonFly: src/sys/dev/raid/mly/Makefile,v 1.3 2008/01/21 02:27:37 pavalos Exp $
 
 .PATH:         ${.CURDIR}/../../dev/mly
 KMOD           =  mly
-SRCS           =  mly.c mly_pci.c mly_cam.c
+SRCS           =  mly.c
 SRCS           += opt_scsi.h opt_cam.h
 SRCS           += device_if.h bus_if.h pci_if.h
 
-#CFLAGS+=      -DMLY_DEBUG=1
+CFLAGS+=       -DMLY_MODULE
 
 .include <bsd.kmod.mk>
index 214b883..f7ac366 100644 (file)
@@ -25,7 +25,7 @@
  * SUCH DAMAGE.
  *
  *     $FreeBSD: src/sys/dev/mly/mly.c,v 1.3.2.3 2001/03/05 20:17:24 msmith Exp $
- *     $DragonFly: src/sys/dev/raid/mly/mly.c,v 1.18 2007/04/13 02:51:34 y0netan1 Exp $
+ *     $DragonFly: src/sys/dev/raid/mly/mly.c,v 1.19 2008/01/21 02:27:37 pavalos Exp $
  */
 
 #include <sys/param.h>
 #include <sys/rman.h>
 #include <sys/thread2.h>
 
+#include <bus/cam/cam.h>
+#include <bus/cam/cam_ccb.h>
+#include <bus/cam/cam_periph.h>
+#include <bus/cam/cam_sim.h>
+#include <bus/cam/cam_xpt_sim.h>
 #include <bus/cam/scsi/scsi_all.h>
+#include <bus/cam/scsi/scsi_message.h>
+
+#include <bus/pci/pcireg.h>
+#include <bus/pci/pcivar.h>
 
 #include "mlyreg.h"
 #include "mlyio.h"
 #include "mlyvar.h"
-#define MLY_DEFINE_TABLES
 #include "mly_tables.h"
 
+static int     mly_probe(device_t dev);
+static int     mly_attach(device_t dev);
+static int     mly_pci_attach(struct mly_softc *sc);
+static int     mly_detach(device_t dev);
+static int     mly_shutdown(device_t dev);
+static void    mly_intr(void *arg);
+
+static int     mly_sg_map(struct mly_softc *sc);
+static void    mly_sg_map_helper(void *arg, bus_dma_segment_t *segs, int nseg, int error);
+static int     mly_mmbox_map(struct mly_softc *sc);
+static void    mly_mmbox_map_helper(void *arg, bus_dma_segment_t *segs, int nseg, int error);
+static void    mly_free(struct mly_softc *sc);
+
 static int     mly_get_controllerinfo(struct mly_softc *sc);
 static void    mly_scan_devices(struct mly_softc *sc);
 static void    mly_rescan_btl(struct mly_softc *sc, int bus, int target);
@@ -58,6 +79,7 @@ static int    mly_enable_mmbox(struct mly_softc *sc);
 static int     mly_flush(struct mly_softc *sc);
 static int     mly_ioctl(struct mly_softc *sc, struct mly_command_ioctl *ioctl, void **data, 
                          size_t datasize, u_int8_t *status, void *sense_buffer, size_t *sense_length);
+static void    mly_check_event(struct mly_softc *sc);
 static void    mly_fetch_event(struct mly_softc *sc);
 static void    mly_complete_event(struct mly_command *mc);
 static void    mly_process_event(struct mly_softc *sc, struct mly_event *me);
@@ -65,13 +87,28 @@ static void mly_periodic(void *data);
 
 static int     mly_immediate_command(struct mly_command *mc);
 static int     mly_start(struct mly_command *mc);
+static void    mly_done(struct mly_softc *sc);
 static void    mly_complete(void *context, int pending);
 
+static int     mly_alloc_command(struct mly_softc *sc, struct mly_command **mcp);
+static void    mly_release_command(struct mly_command *mc);
 static void    mly_alloc_commands_map(void *arg, bus_dma_segment_t *segs, int nseg, int error);
 static int     mly_alloc_commands(struct mly_softc *sc);
+static void    mly_release_commands(struct mly_softc *sc);
 static void    mly_map_command(struct mly_command *mc);
 static void    mly_unmap_command(struct mly_command *mc);
 
+static int     mly_cam_attach(struct mly_softc *sc);
+static void    mly_cam_detach(struct mly_softc *sc);
+static void    mly_cam_rescan_btl(struct mly_softc *sc, int bus, int target);
+static void    mly_cam_rescan_callback(struct cam_periph *periph, union ccb *ccb);
+static void    mly_cam_action(struct cam_sim *sim, union ccb *ccb);
+static int     mly_cam_action_io(struct cam_sim *sim, struct ccb_scsiio *csio);
+static void    mly_cam_poll(struct cam_sim *sim);
+static void    mly_cam_complete(struct mly_command *mc);
+static struct cam_periph *mly_find_periph(struct mly_softc *sc, int bus, int target);
+static int     mly_name_device(struct mly_softc *sc, int bus, int target);
+
 static int     mly_fwhandshake(struct mly_softc *sc);
 
 static void    mly_describe_controller(struct mly_softc *sc);
@@ -82,6 +119,8 @@ static void  mly_print_packet(struct mly_command *mc);
 static void    mly_panic(struct mly_softc *sc, char *reason);
 #endif
 void           mly_print_controller(int controller);
+static int     mly_timeout(struct mly_softc *sc);
+
 
 static d_open_t                mly_user_open;
 static d_close_t       mly_user_close;
@@ -89,6 +128,26 @@ static d_ioctl_t    mly_user_ioctl;
 static int     mly_user_command(struct mly_softc *sc, struct mly_user_command *uc);
 static int     mly_user_health(struct mly_softc *sc, struct mly_user_health *uh);
 
+#define MLY_CMD_TIMEOUT                20
+
+static device_method_t mly_methods[] = {
+    /* Device interface */
+    DEVMETHOD(device_probe,    mly_probe),
+    DEVMETHOD(device_attach,   mly_attach),
+    DEVMETHOD(device_detach,   mly_detach),
+    DEVMETHOD(device_shutdown, mly_shutdown),
+    { 0, 0 }
+};
+
+static driver_t mly_pci_driver = {
+       "mly",
+       mly_methods,
+       sizeof(struct mly_softc)
+};
+
+static devclass_t      mly_devclass;
+DRIVER_MODULE(mly, pci, mly_pci_driver, mly_devclass, 0, 0);
+
 #define MLY_CDEV_MAJOR  158
 
 static struct dev_ops mly_ops = {
@@ -104,32 +163,84 @@ static struct dev_ops mly_ops = {
  ********************************************************************************
  ********************************************************************************/
 
+static struct mly_ident
+{
+    u_int16_t          vendor;
+    u_int16_t          device;
+    u_int16_t          subvendor;
+    u_int16_t          subdevice;
+    int                        hwif;
+    char               *desc;
+} mly_identifiers[] = {
+    {0x1069, 0xba56, 0x1069, 0x0040, MLY_HWIF_STRONGARM, "Mylex eXtremeRAID 2000"},
+    {0x1069, 0xba56, 0x1069, 0x0030, MLY_HWIF_STRONGARM, "Mylex eXtremeRAID 3000"},
+    {0x1069, 0x0050, 0x1069, 0x0050, MLY_HWIF_I960RX,    "Mylex AcceleRAID 352"},
+    {0x1069, 0x0050, 0x1069, 0x0052, MLY_HWIF_I960RX,    "Mylex AcceleRAID 170"},
+    {0x1069, 0x0050, 0x1069, 0x0054, MLY_HWIF_I960RX,    "Mylex AcceleRAID 160"},
+    {0, 0, 0, 0, 0, 0}
+};
+
+/********************************************************************************
+ * Compare the provided PCI device with the list we support.
+ */
+static int
+mly_probe(device_t dev)
+{
+    struct mly_ident   *m;
+
+    debug_called(1);
+
+    for (m = mly_identifiers; m->vendor != 0; m++) {
+       if ((m->vendor == pci_get_vendor(dev)) &&
+           (m->device == pci_get_device(dev)) &&
+           ((m->subvendor == 0) || ((m->subvendor == pci_get_subvendor(dev)) &&
+                                    (m->subdevice == pci_get_subdevice(dev))))) {
+           
+           device_set_desc(dev, m->desc);
+           return(BUS_PROBE_DEFAULT);  /* allow room to be overridden */
+       }
+    }
+    return(ENXIO);
+}
+
 /********************************************************************************
  * Initialise the controller and softc
  */
-int
-mly_attach(struct mly_softc *sc)
+static int
+mly_attach(device_t dev)
 {
-    int                error;
+    struct mly_softc   *sc = device_get_softc(dev);
+    int                        error;
 
     debug_called(1);
 
+    sc->mly_dev = dev;
+
+#ifdef MLY_DEBUG
+    if (device_get_unit(sc->mly_dev) == 0)
+       mly_softc0 = sc;
+#endif    
+
+    /*
+     * Do PCI-specific initialisation.
+     */
+    if ((error = mly_pci_attach(sc)) != 0)
+       goto out;
+
     callout_init(&sc->mly_periodic);
+    callout_init(&sc->mly_timeout);
 
     /*
      * Initialise per-controller queues.
      */
     mly_initq_free(sc);
-    mly_initq_ready(sc);
     mly_initq_busy(sc);
     mly_initq_complete(sc);
 
-#if defined(__FreeBSD__) && __FreeBSD_version >= 500005
     /*
      * Initialise command-completion task.
      */
     TASK_INIT(&sc->mly_task_complete, 0, mly_complete, sc);
-#endif
 
     /* disable interrupts before we start talking to the controller */
     MLY_MASK_INTERRUPTS(sc);
@@ -140,38 +251,45 @@ mly_attach(struct mly_softc *sc)
      * run.
      */
     if ((error = mly_fwhandshake(sc)))
-       return(error);
+       goto out;
 
     /*
-     * Allocate command buffers
+     * Allocate initial command buffers.
      */
     if ((error = mly_alloc_commands(sc)))
-       return(error);
+       goto out;
 
     /* 
      * Obtain controller feature information
      */
     if ((error = mly_get_controllerinfo(sc)))
-       return(error);
+       goto out;
+
+    /*
+     * Reallocate command buffers now we know how many we want.
+     */
+    mly_release_commands(sc);
+    if ((error = mly_alloc_commands(sc)))
+       goto out;
 
     /*
      * Get the current event counter for health purposes, populate the initial
      * health status buffer.
      */
     if ((error = mly_get_eventstatus(sc)))
-       return(error);
+       goto out;
 
     /*
-     * Enable memory-mailbox mode
+     * Enable memory-mailbox mode.
      */
     if ((error = mly_enable_mmbox(sc)))
-       return(error);
+       goto out;
 
     /*
      * Attach to CAM.
      */
     if ((error = mly_cam_attach(sc)))
-       return(error);
+       goto out;
 
     /* 
      * Print a little information about the controller 
@@ -179,15 +297,14 @@ mly_attach(struct mly_softc *sc)
     mly_describe_controller(sc);
 
     /*
-     * Mark all attached devices for rescan
+     * Mark all attached devices for rescan.
      */
     mly_scan_devices(sc);
 
     /*
      * Instigate the first status poll immediately.  Rescan completions won't
      * happen until interrupts are enabled, which should still be before
-     * the SCSI subsystem gets to us. (XXX assuming CAM and interrupt-driven
-     * discovery here...)
+     * the SCSI subsystem gets to us, courtesy of the "SCSI settling delay".
      */
     mly_periodic((void *)sc);
 
@@ -203,28 +320,423 @@ mly_attach(struct mly_softc *sc)
     /* enable interrupts now */
     MLY_UNMASK_INTERRUPTS(sc);
 
+#ifdef MLY_DEBUG
+    callout_reset(&sc->mly_timeout, MLY_CMD_TIMEOUT * hz,
+                 (timeout_t *)mly_timeout, sc);
+#endif
+
+ out:
+    if (error != 0)
+       mly_free(sc);
+    return(error);
+}
+
+/********************************************************************************
+ * Perform PCI-specific initialisation.
+ */
+static int
+mly_pci_attach(struct mly_softc *sc)
+{
+    int                        i, error;
+    u_int32_t          command;
+
+    debug_called(1);
+
+    /* assume failure is 'not configured' */
+    error = ENXIO;
+
+    /* 
+     * Verify that the adapter is correctly set up in PCI space.
+     * 
+     * XXX we shouldn't do this; the PCI code should.
+     */
+    command = pci_read_config(sc->mly_dev, PCIR_COMMAND, 2);
+    command |= PCIM_CMD_BUSMASTEREN;
+    pci_write_config(sc->mly_dev, PCIR_COMMAND, command, 2);
+    command = pci_read_config(sc->mly_dev, PCIR_COMMAND, 2);
+    if (!(command & PCIM_CMD_BUSMASTEREN)) {
+       mly_printf(sc, "can't enable busmaster feature\n");
+       goto fail;
+    }
+    if ((command & PCIM_CMD_MEMEN) == 0) {
+       mly_printf(sc, "memory window not available\n");
+       goto fail;
+    }
+
+    /*
+     * Allocate the PCI register window.
+     */
+    sc->mly_regs_rid = PCIR_BAR(0);    /* first base address register */
+    if ((sc->mly_regs_resource = bus_alloc_resource_any(sc->mly_dev, 
+           SYS_RES_MEMORY, &sc->mly_regs_rid, RF_ACTIVE)) == NULL) {
+       mly_printf(sc, "can't allocate register window\n");
+       goto fail;
+    }
+    sc->mly_btag = rman_get_bustag(sc->mly_regs_resource);
+    sc->mly_bhandle = rman_get_bushandle(sc->mly_regs_resource);
+
+    /* 
+     * Allocate and connect our interrupt.
+     */
+    sc->mly_irq_rid = 0;
+    if ((sc->mly_irq = bus_alloc_resource_any(sc->mly_dev, SYS_RES_IRQ, 
+                   &sc->mly_irq_rid, RF_SHAREABLE | RF_ACTIVE)) == NULL) {
+       mly_printf(sc, "can't allocate interrupt\n");
+       goto fail;
+    }
+    error = bus_setup_intr(sc->mly_dev, sc->mly_irq, 0,
+                          mly_intr, sc, &sc->mly_intr, NULL);
+    if (error) {
+       mly_printf(sc, "can't set up interrupt\n");
+       goto fail;
+    }
+
+    /* assume failure is 'out of memory' */
+    error = ENOMEM;
+
+    /*
+     * Allocate the parent bus DMA tag appropriate for our PCI interface.
+     * 
+     * Note that all of these controllers are 64-bit capable.
+     */
+    if (bus_dma_tag_create(NULL,                       /* parent */
+                          1, 0,                        /* alignment, boundary */
+                          BUS_SPACE_MAXADDR_32BIT,     /* lowaddr */
+                          BUS_SPACE_MAXADDR,           /* highaddr */
+                          NULL, NULL,                  /* filter, filterarg */
+                          MAXBSIZE, MLY_MAX_SGENTRIES, /* maxsize, nsegments */
+                          BUS_SPACE_MAXSIZE_32BIT,     /* maxsegsize */
+                          BUS_DMA_ALLOCNOW,            /* flags */
+                          &sc->mly_parent_dmat)) {
+       mly_printf(sc, "can't allocate parent DMA tag\n");
+       goto fail;
+    }
+
+    /*
+     * Create DMA tag for mapping buffers into controller-addressable space.
+     */
+    if (bus_dma_tag_create(sc->mly_parent_dmat,        /* parent */
+                          1, 0,                        /* alignment, boundary */
+                          BUS_SPACE_MAXADDR,           /* lowaddr */
+                          BUS_SPACE_MAXADDR,           /* highaddr */
+                          NULL, NULL,                  /* filter, filterarg */
+                          MAXBSIZE, MLY_MAX_SGENTRIES, /* maxsize, nsegments */
+                          BUS_SPACE_MAXSIZE_32BIT,     /* maxsegsize */
+                          0,                           /* flags */
+                          &sc->mly_buffer_dmat)) {
+       mly_printf(sc, "can't allocate buffer DMA tag\n");
+       goto fail;
+    }
+
+    /*
+     * Initialise the DMA tag for command packets.
+     */
+    if (bus_dma_tag_create(sc->mly_parent_dmat,                /* parent */
+                          1, 0,                        /* alignment, boundary */
+                          BUS_SPACE_MAXADDR,           /* lowaddr */
+                          BUS_SPACE_MAXADDR,           /* highaddr */
+                          NULL, NULL,                  /* filter, filterarg */
+                          sizeof(union mly_command_packet) * MLY_MAX_COMMANDS, 1,      /* maxsize, nsegments */
+                          BUS_SPACE_MAXSIZE_32BIT,     /* maxsegsize */
+                          BUS_DMA_ALLOCNOW,            /* flags */
+                          &sc->mly_packet_dmat)) {
+       mly_printf(sc, "can't allocate command packet DMA tag\n");
+       goto fail;
+    }
+
+    /* 
+     * Detect the hardware interface version 
+     */
+    for (i = 0; mly_identifiers[i].vendor != 0; i++) {
+       if ((mly_identifiers[i].vendor == pci_get_vendor(sc->mly_dev)) &&
+           (mly_identifiers[i].device == pci_get_device(sc->mly_dev))) {
+           sc->mly_hwif = mly_identifiers[i].hwif;
+           switch(sc->mly_hwif) {
+           case MLY_HWIF_I960RX:
+               debug(1, "set hardware up for i960RX");
+               sc->mly_doorbell_true = 0x00;
+               sc->mly_command_mailbox =  MLY_I960RX_COMMAND_MAILBOX;
+               sc->mly_status_mailbox =   MLY_I960RX_STATUS_MAILBOX;
+               sc->mly_idbr =             MLY_I960RX_IDBR;
+               sc->mly_odbr =             MLY_I960RX_ODBR;
+               sc->mly_error_status =     MLY_I960RX_ERROR_STATUS;
+               sc->mly_interrupt_status = MLY_I960RX_INTERRUPT_STATUS;
+               sc->mly_interrupt_mask =   MLY_I960RX_INTERRUPT_MASK;
+               break;
+           case MLY_HWIF_STRONGARM:
+               debug(1, "set hardware up for StrongARM");
+               sc->mly_doorbell_true = 0xff;           /* doorbell 'true' is 0 */
+               sc->mly_command_mailbox =  MLY_STRONGARM_COMMAND_MAILBOX;
+               sc->mly_status_mailbox =   MLY_STRONGARM_STATUS_MAILBOX;
+               sc->mly_idbr =             MLY_STRONGARM_IDBR;
+               sc->mly_odbr =             MLY_STRONGARM_ODBR;
+               sc->mly_error_status =     MLY_STRONGARM_ERROR_STATUS;
+               sc->mly_interrupt_status = MLY_STRONGARM_INTERRUPT_STATUS;
+               sc->mly_interrupt_mask =   MLY_STRONGARM_INTERRUPT_MASK;
+               break;
+           }
+           break;
+       }
+    }
+
+    /*
+     * Create the scatter/gather mappings.
+     */
+    if ((error = mly_sg_map(sc)))
+       goto fail;
+
+    /*
+     * Allocate and map the memory mailbox
+     */
+    if ((error = mly_mmbox_map(sc)))
+       goto fail;
+
+    error = 0;
+           
+fail:
+    return(error);
+}
+
+/********************************************************************************
+ * Shut the controller down and detach all our resources.
+ */
+static int
+mly_detach(device_t dev)
+{
+    int                        error;
+
+    if ((error = mly_shutdown(dev)) != 0)
+       return(error);
+    
+    mly_free(device_get_softc(dev));
     return(0);
 }
 
 /********************************************************************************
  * Bring the controller to a state where it can be safely left alone.
+ *
+ * Note that it should not be necessary to wait for any outstanding commands,
+ * as they should be completed prior to calling here.
+ *
+ * XXX this applies for I/O, but not status polls; we should beware of
+ *     the case where a status command is running while we detach.
  */
-void
-mly_detach(struct mly_softc *sc)
+static int
+mly_shutdown(device_t dev)
 {
+    struct mly_softc   *sc = device_get_softc(dev);
 
     debug_called(1);
+    
+    if (sc->mly_state & MLY_STATE_OPEN)
+       return(EBUSY);
 
     /* kill the periodic event */
     callout_stop(&sc->mly_periodic);
 
-    sc->mly_state |= MLY_STATE_SUSPEND;
-
     /* flush controller */
     mly_printf(sc, "flushing cache...");
     kprintf("%s\n", mly_flush(sc) ? "failed" : "done");
 
     MLY_MASK_INTERRUPTS(sc);
+
+    return(0);
+}
+
+/*******************************************************************************
+ * Take an interrupt, or be poked by other code to look for interrupt-worthy
+ * status.
+ */
+static void
+mly_intr(void *arg)
+{
+    struct mly_softc   *sc = (struct mly_softc *)arg;
+
+    debug_called(2);
+
+    mly_done(sc);
+};
+
+/********************************************************************************
+ ********************************************************************************
+                                                Bus-dependant Resource Management
+ ********************************************************************************
+ ********************************************************************************/
+
+/********************************************************************************
+ * Allocate memory for the scatter/gather tables
+ */
+static int
+mly_sg_map(struct mly_softc *sc)
+{
+    size_t     segsize;
+
+    debug_called(1);
+
+    /*
+     * Create a single tag describing a region large enough to hold all of
+     * the s/g lists we will need.
+     */
+    segsize = sizeof(struct mly_sg_entry) * MLY_MAX_COMMANDS *MLY_MAX_SGENTRIES;
+    if (bus_dma_tag_create(sc->mly_parent_dmat,                /* parent */
+                          1, 0,                        /* alignment,boundary */
+                          BUS_SPACE_MAXADDR,           /* lowaddr */
+                          BUS_SPACE_MAXADDR,           /* highaddr */
+                          NULL, NULL,                  /* filter, filterarg */
+                          segsize, 1,                  /* maxsize, nsegments */
+                          BUS_SPACE_MAXSIZE_32BIT,     /* maxsegsize */
+                          BUS_DMA_ALLOCNOW,            /* flags */
+                          &sc->mly_sg_dmat)) {
+       mly_printf(sc, "can't allocate scatter/gather DMA tag\n");
+       return(ENOMEM);
+    }
+
+    /*
+     * Allocate enough s/g maps for all commands and permanently map them into
+     * controller-visible space.
+     * 
+     * XXX this assumes we can get enough space for all the s/g maps in one 
+     * contiguous slab.
+     */
+    if (bus_dmamem_alloc(sc->mly_sg_dmat, (void **)&sc->mly_sg_table,
+                        BUS_DMA_NOWAIT, &sc->mly_sg_dmamap)) {
+       mly_printf(sc, "can't allocate s/g table\n");
+       return(ENOMEM);
+    }
+    if (bus_dmamap_load(sc->mly_sg_dmat, sc->mly_sg_dmamap, sc->mly_sg_table,
+                       segsize, mly_sg_map_helper, sc, BUS_DMA_NOWAIT) != 0)
+       return (ENOMEM);
+    return(0);
+}
+
+/********************************************************************************
+ * Save the physical address of the base of the s/g table.
+ */
+static void
+mly_sg_map_helper(void *arg, bus_dma_segment_t *segs, int nseg, int error)
+{
+    struct mly_softc   *sc = (struct mly_softc *)arg;
+
+    debug_called(1);
+
+    /* save base of s/g table's address in bus space */
+    sc->mly_sg_busaddr = segs->ds_addr;
+}
+
+/********************************************************************************
+ * Allocate memory for the memory-mailbox interface
+ */
+static int
+mly_mmbox_map(struct mly_softc *sc)
+{
+
+    /*
+     * Create a DMA tag for a single contiguous region large enough for the
+     * memory mailbox structure.
+     */
+    if (bus_dma_tag_create(sc->mly_parent_dmat,                /* parent */
+                          1, 0,                        /* alignment,boundary */
+                          BUS_SPACE_MAXADDR,           /* lowaddr */
+                          BUS_SPACE_MAXADDR,           /* highaddr */
+                          NULL, NULL,                  /* filter, filterarg */
+                          sizeof(struct mly_mmbox), 1, /* maxsize, nsegments */
+                          BUS_SPACE_MAXSIZE_32BIT,     /* maxsegsize */
+                          BUS_DMA_ALLOCNOW,            /* flags */
+                          &sc->mly_mmbox_dmat)) {
+       mly_printf(sc, "can't allocate memory mailbox DMA tag\n");
+       return(ENOMEM);
+    }
+
+    /*
+     * Allocate the buffer
+     */
+    if (bus_dmamem_alloc(sc->mly_mmbox_dmat, (void **)&sc->mly_mmbox, BUS_DMA_NOWAIT, &sc->mly_mmbox_dmamap)) {
+       mly_printf(sc, "can't allocate memory mailbox\n");
+       return(ENOMEM);
+    }
+    if (bus_dmamap_load(sc->mly_mmbox_dmat, sc->mly_mmbox_dmamap, sc->mly_mmbox,
+                       sizeof(struct mly_mmbox), mly_mmbox_map_helper, sc, 
+                       BUS_DMA_NOWAIT) != 0)
+       return (ENOMEM);
+    bzero(sc->mly_mmbox, sizeof(*sc->mly_mmbox));
+    return(0);
+
+}
+
+/********************************************************************************
+ * Save the physical address of the memory mailbox 
+ */
+static void
+mly_mmbox_map_helper(void *arg, bus_dma_segment_t *segs, int nseg, int error)
+{
+    struct mly_softc   *sc = (struct mly_softc *)arg;
+
+    debug_called(1);
+
+    sc->mly_mmbox_busaddr = segs->ds_addr;
+}
+
+/********************************************************************************
+ * Free all of the resources associated with (sc)
+ *
+ * Should not be called if the controller is active.
+ */
+static void
+mly_free(struct mly_softc *sc)
+{
+    
+    debug_called(1);
+
+    /* Remove the management device */
+    destroy_dev(sc->mly_dev_t);
+
+    /* detach from CAM */
+    mly_cam_detach(sc);
+
+    /* release command memory */
+    mly_release_commands(sc);
+    
+    /* throw away the controllerinfo structure */
+    if (sc->mly_controllerinfo != NULL)
+       kfree(sc->mly_controllerinfo, M_DEVBUF);
+
+    /* throw away the controllerparam structure */
+    if (sc->mly_controllerparam != NULL)
+       kfree(sc->mly_controllerparam, M_DEVBUF);
+
+    /* destroy data-transfer DMA tag */
+    if (sc->mly_buffer_dmat)
+       bus_dma_tag_destroy(sc->mly_buffer_dmat);
+
+    /* free and destroy DMA memory and tag for s/g lists */
+    if (sc->mly_sg_table) {
+       bus_dmamap_unload(sc->mly_sg_dmat, sc->mly_sg_dmamap);
+       bus_dmamem_free(sc->mly_sg_dmat, sc->mly_sg_table, sc->mly_sg_dmamap);
+    }
+    if (sc->mly_sg_dmat)
+       bus_dma_tag_destroy(sc->mly_sg_dmat);
+
+    /* free and destroy DMA memory and tag for memory mailbox */
+    if (sc->mly_mmbox) {
+       bus_dmamap_unload(sc->mly_mmbox_dmat, sc->mly_mmbox_dmamap);
+       bus_dmamem_free(sc->mly_mmbox_dmat, sc->mly_mmbox, sc->mly_mmbox_dmamap);
+    }
+    if (sc->mly_mmbox_dmat)
+       bus_dma_tag_destroy(sc->mly_mmbox_dmat);
+
+    /* disconnect the interrupt handler */
+    if (sc->mly_intr)
+       bus_teardown_intr(sc->mly_dev, sc->mly_irq, sc->mly_intr);
+    if (sc->mly_irq != NULL)
+       bus_release_resource(sc->mly_dev, SYS_RES_IRQ, sc->mly_irq_rid, sc->mly_irq);
+
+    /* destroy the parent DMA tag */
+    if (sc->mly_parent_dmat)
+       bus_dma_tag_destroy(sc->mly_parent_dmat);
+
+    /* release the register window mapping */
+    if (sc->mly_regs_resource != NULL)
+       bus_release_resource(sc->mly_dev, SYS_RES_MEMORY, sc->mly_regs_rid, sc->mly_regs_resource);
 }
 
 /********************************************************************************
@@ -281,7 +793,7 @@ mly_get_controllerinfo(struct mly_softc *sc)
 static void
 mly_scan_devices(struct mly_softc *sc)
 {
-    int                bus, target, nchn;
+    int                bus, target;
 
     debug_called(1);
 
@@ -291,19 +803,22 @@ mly_scan_devices(struct mly_softc *sc)
     bzero(&sc->mly_btl, sizeof(sc->mly_btl));
 
     /*
-     * Mark all devices as requiring a rescan, and let the early periodic scan collect them.
+     * Mark all devices as requiring a rescan, and let the next
+     * periodic scan collect them. 
      */
-    nchn = sc->mly_controllerinfo->physical_channels_present +
-       sc->mly_controllerinfo->virtual_channels_present;
-    for (bus = 0; bus < nchn; bus++)
-       for (target = 0; target < MLY_MAX_TARGETS; target++)
-           sc->mly_btl[bus][target].mb_flags = MLY_BTL_RESCAN;
+    for (bus = 0; bus < sc->mly_cam_channels; bus++)
+       if (MLY_BUS_IS_VALID(sc, bus)) 
+           for (target = 0; target < MLY_MAX_TARGETS; target++)
+               sc->mly_btl[bus][target].mb_flags = MLY_BTL_RESCAN;
 
 }
 
 /********************************************************************************
  * Rescan a device, possibly as a consequence of getting an event which suggests
  * that it may have changed.
+ *
+ * If we suffer resource starvation, we can abandon the rescan as we'll be
+ * retried.
  */
 static void
 mly_rescan_btl(struct mly_softc *sc, int bus, int target)
@@ -311,51 +826,52 @@ mly_rescan_btl(struct mly_softc *sc, int bus, int target)
     struct mly_command         *mc;
     struct mly_command_ioctl   *mci;
 
-    debug_called(2);
+    debug_called(1);
+
+    /* check that this bus is valid */
+    if (!MLY_BUS_IS_VALID(sc, bus))
+       return;
 
     /* get a command */
-    mc = NULL;
     if (mly_alloc_command(sc, &mc))
-       return;                         /* we'll be retried soon */
+       return;
 
     /* set up the data buffer */
     mc->mc_data = kmalloc(sizeof(union mly_devinfo), M_DEVBUF, M_INTWAIT | M_ZERO);
     mc->mc_flags |= MLY_CMD_DATAIN;
     mc->mc_complete = mly_complete_rescan;
 
-    sc->mly_btl[bus][target].mb_flags &= ~MLY_BTL_RESCAN;
-
     /* 
      * Build the ioctl.
-     *
-     * At this point we are committed to sending this request, as it
-     * will be the only one constructed for this particular update.
      */
     mci = (struct mly_command_ioctl *)&mc->mc_packet->ioctl;
     mci->opcode = MDACMD_IOCTL;
     mci->addr.phys.controller = 0;
     mci->timeout.value = 30;
     mci->timeout.scale = MLY_TIMEOUT_SECONDS;
-    if (bus >= sc->mly_controllerinfo->physical_channels_present) {
+    if (MLY_BUS_IS_VIRTUAL(sc, bus)) {
        mc->mc_length = mci->data_size = sizeof(struct mly_ioctl_getlogdevinfovalid);
        mci->sub_ioctl = MDACIOCTL_GETLOGDEVINFOVALID;
-       mci->addr.log.logdev = ((bus - sc->mly_controllerinfo->physical_channels_present) * MLY_MAX_TARGETS) 
-           + target;
-       debug(2, "logical device %d", mci->addr.log.logdev);
+       mci->addr.log.logdev = MLY_LOGDEV_ID(sc, bus, target);
+       debug(1, "logical device %d", mci->addr.log.logdev);
     } else {
        mc->mc_length = mci->data_size = sizeof(struct mly_ioctl_getphysdevinfovalid);
        mci->sub_ioctl = MDACIOCTL_GETPHYSDEVINFOVALID;
        mci->addr.phys.lun = 0;
        mci->addr.phys.target = target;
        mci->addr.phys.channel = bus;
-       debug(2, "physical device %d:%d", mci->addr.phys.channel, mci->addr.phys.target);
+       debug(1, "physical device %d:%d", mci->addr.phys.channel, mci->addr.phys.target);
     }
     
     /*
-     * Use the ready queue to get this command dispatched.
+     * Dispatch the command.  If we successfully send the command, clear the rescan
+     * bit.
      */
-    mly_enqueue_ready(mc);
-    mly_startio(sc);
+    if (mly_start(mc) != 0) {
+       mly_release_command(mc);
+    } else {
+       sc->mly_btl[bus][target].mb_flags &= ~MLY_BTL_RESCAN;   /* success */   
+    }
 }
 
 /********************************************************************************
@@ -367,46 +883,94 @@ mly_complete_rescan(struct mly_command *mc)
     struct mly_softc                           *sc = mc->mc_sc;
     struct mly_ioctl_getlogdevinfovalid                *ldi;
     struct mly_ioctl_getphysdevinfovalid       *pdi;
-    int                                                bus, target;
+    struct mly_command_ioctl                   *mci;
+    struct mly_btl                             btl, *btlp;
+    int                                                bus, target, rescan;
 
-    debug_called(2);
+    debug_called(1);
 
-    /* iff the command completed OK, we should use the result to update our data */
+    /*
+     * Recover the bus and target from the command.  We need these even in
+     * the case where we don't have a useful response.
+     */
+    mci = (struct mly_command_ioctl *)&mc->mc_packet->ioctl;
+    if (mci->sub_ioctl == MDACIOCTL_GETLOGDEVINFOVALID) {
+       bus = MLY_LOGDEV_BUS(sc, mci->addr.log.logdev);
+       target = MLY_LOGDEV_TARGET(sc, mci->addr.log.logdev);
+    } else {
+       bus = mci->addr.phys.channel;
+       target = mci->addr.phys.target;
+    }
+    /* XXX validate bus/target? */
+    
+    /* the default result is 'no device' */
+    bzero(&btl, sizeof(btl));
+
+    /* if the rescan completed OK, we have possibly-new BTL data */
     if (mc->mc_status == 0) {
        if (mc->mc_length == sizeof(*ldi)) {
            ldi = (struct mly_ioctl_getlogdevinfovalid *)mc->mc_data;
-           bus = MLY_LOGDEV_BUS(sc, ldi->logical_device_number);
-           target = MLY_LOGDEV_TARGET(ldi->logical_device_number);
-           sc->mly_btl[bus][target].mb_flags = MLY_BTL_LOGICAL;        /* clears all other flags */
-           sc->mly_btl[bus][target].mb_type = ldi->raid_level;
-           sc->mly_btl[bus][target].mb_state = ldi->state;
-           debug(2, "BTL rescan for %d returns %s, %s", ldi->logical_device_number, 
+           if ((MLY_LOGDEV_BUS(sc, ldi->logical_device_number) != bus) ||
+               (MLY_LOGDEV_TARGET(sc, ldi->logical_device_number) != target)) {
+               mly_printf(sc, "WARNING: BTL rescan for %d:%d returned data for %d:%d instead\n",
+                          bus, target, MLY_LOGDEV_BUS(sc, ldi->logical_device_number),
+                          MLY_LOGDEV_TARGET(sc, ldi->logical_device_number));
+               /* XXX what can we do about this? */
+           }
+           btl.mb_flags = MLY_BTL_LOGICAL;
+           btl.mb_type = ldi->raid_level;
+           btl.mb_state = ldi->state;
+           debug(1, "BTL rescan for %d returns %s, %s", ldi->logical_device_number, 
                  mly_describe_code(mly_table_device_type, ldi->raid_level),
                  mly_describe_code(mly_table_device_state, ldi->state));
        } else if (mc->mc_length == sizeof(*pdi)) {
            pdi = (struct mly_ioctl_getphysdevinfovalid *)mc->mc_data;
-           bus = pdi->channel;
-           target = pdi->target;
-           sc->mly_btl[bus][target].mb_flags = MLY_BTL_PHYSICAL;       /* clears all other flags */
-           sc->mly_btl[bus][target].mb_type = MLY_DEVICE_TYPE_PHYSICAL;
-           sc->mly_btl[bus][target].mb_state = pdi->state;
-           sc->mly_btl[bus][target].mb_speed = pdi->speed;
-           sc->mly_btl[bus][target].mb_width = pdi->width;
+           if ((pdi->channel != bus) || (pdi->target != target)) {
+               mly_printf(sc, "WARNING: BTL rescan for %d:%d returned data for %d:%d instead\n",
+                          bus, target, pdi->channel, pdi->target);
+               /* XXX what can we do about this? */
+           }
+           btl.mb_flags = MLY_BTL_PHYSICAL;
+           btl.mb_type = MLY_DEVICE_TYPE_PHYSICAL;
+           btl.mb_state = pdi->state;
+           btl.mb_speed = pdi->speed;
+           btl.mb_width = pdi->width;
            if (pdi->state != MLY_DEVICE_STATE_UNCONFIGURED)
                sc->mly_btl[bus][target].mb_flags |= MLY_BTL_PROTECTED;
-           debug(2, "BTL rescan for %d:%d returns %s", bus, target, 
+           debug(1, "BTL rescan for %d:%d returns %s", bus, target, 
                  mly_describe_code(mly_table_device_state, pdi->state));
        } else {
-           mly_printf(sc, "BTL rescan result corrupted\n");
+           mly_printf(sc, "BTL rescan result invalid\n");
        }
-    } else {
-       /*
-        * A request sent for a device beyond the last device present will fail.
-        * We don't care about this, so we do nothing about it.
-        */
     }
+
     kfree(mc->mc_data, M_DEVBUF);
     mly_release_command(mc);
+
+    /*
+     * Decide whether we need to rescan the device.
+     */
+    rescan = 0;
+
+    /* device type changes (usually between 'nothing' and 'something') */
+    btlp = &sc->mly_btl[bus][target];
+    if (btl.mb_flags != btlp->mb_flags) {
+       debug(1, "flags changed, rescanning");
+       rescan = 1;
+    }
+    
+    /* XXX other reasons? */
+
+    /*
+     * Update BTL information.
+     */
+    *btlp = btl;
+
+    /*
+     * Perform CAM rescan if required.
+     */
+    if (rescan)
+       mly_cam_rescan_btl(sc, bus, target);
 }
 
 /********************************************************************************
@@ -592,8 +1156,37 @@ out:
     return(error);
 }
 
+/********************************************************************************
+ * Check for event(s) outstanding in the controller.
+ */
+static void
+mly_check_event(struct mly_softc *sc)
+{
+    
+    /*
+     * The controller may have updated the health status information,
+     * so check for it here.  Note that the counters are all in host memory,
+     * so this check is very cheap.  Also note that we depend on checking on
+     * completion 
+     */
+    if (sc->mly_mmbox->mmm_health.status.change_counter != sc->mly_event_change) {
+       sc->mly_event_change = sc->mly_mmbox->mmm_health.status.change_counter;
+       debug(1, "event change %d, event status update, %d -> %d", sc->mly_event_change,
+             sc->mly_event_waiting, sc->mly_mmbox->mmm_health.status.next_event);
+       sc->mly_event_waiting = sc->mly_mmbox->mmm_health.status.next_event;
+
+       /* wake up anyone that might be interested in this */
+       wakeup(&sc->mly_event_change);
+    }
+    if (sc->mly_event_counter != sc->mly_event_waiting)
+    mly_fetch_event(sc);
+}
+
 /********************************************************************************
  * Fetch one event from the controller.
+ *
+ * If we fail due to resource starvation, we'll be retried the next time a 
+ * command completes.
  */
 static void
 mly_fetch_event(struct mly_softc *sc)
@@ -602,12 +1195,11 @@ mly_fetch_event(struct mly_softc *sc)
     struct mly_command_ioctl   *mci;
     u_int32_t                  event;
 
-    debug_called(2);
+    debug_called(1);
 
     /* get a command */
-    mc = NULL;
     if (mly_alloc_command(sc, &mc))
-       return;                         /* we'll get retried the next time a command completes */
+       return;
 
     /* set up the data buffer */
     mc->mc_data = kmalloc(sizeof(struct mly_event), M_DEVBUF, M_INTWAIT|M_ZERO);
@@ -646,20 +1238,22 @@ mly_fetch_event(struct mly_softc *sc)
     mci->sub_ioctl = MDACIOCTL_GETEVENT;
     mci->param.getevent.sequence_number_low = event & 0xffff;
 
-    debug(2, "fetch event %u", event);
+    debug(1, "fetch event %u", event);
 
     /*
-     * Use the ready queue to get this command dispatched.
+     * Submit the command.
+     *
+     * Note that failure of mly_start() will result in this event never being
+     * fetched.
      */
-    mly_enqueue_ready(mc);
-    mly_startio(sc);
+    if (mly_start(mc) != 0) {
+       mly_printf(sc, "couldn't fetch event %u\n", event);
+       mly_release_command(mc);
+    }
 }
 
 /********************************************************************************
  * Handle the completion of an event poll.
- *
- * Note that we don't actually have to instigate another poll; the completion of
- * this command will trigger that if there are any more events to poll for.
  */
 static void
 mly_complete_event(struct mly_command *mc)
@@ -667,7 +1261,7 @@ mly_complete_event(struct mly_command *mc)
     struct mly_softc   *sc = mc->mc_sc;
     struct mly_event   *me = (struct mly_event *)mc->mc_data;
 
-    debug_called(2);
+    debug_called(1);
 
     /* 
      * If the event was successfully fetched, process it.
@@ -677,6 +1271,11 @@ mly_complete_event(struct mly_command *mc)
        kfree(me, M_DEVBUF);
     }
     mly_release_command(mc);
+
+    /*
+     * Check for another event.
+     */
+    mly_check_event(sc);
 }
 
 /********************************************************************************
@@ -707,7 +1306,7 @@ mly_process_event(struct mly_softc *sc, struct mly_event *me)
     /* look up event, get codes */
     fp = mly_describe_code(mly_table_event, event);
 
-    debug(2, "Event %d  code 0x%x", me->sequence_number, me->code);
+    debug(1, "Event %d  code 0x%x", me->sequence_number, me->code);
 
     /* quiet event? */
     class = fp[0];
@@ -733,7 +1332,7 @@ mly_process_event(struct mly_softc *sc, struct mly_event *me)
     case 'l':          /* error on logical unit */
     case 'm':          /* message about logical unit */
        bus = MLY_LOGDEV_BUS(sc, me->lun);
-       target = MLY_LOGDEV_TARGET(me->lun);
+       target = MLY_LOGDEV_TARGET(sc, me->lun);
        mly_name_device(sc, bus, target);
        mly_printf(sc, "logical device %d (%s) %s\n", me->lun, sc->mly_btl[bus][target].mb_name, tp);
        if (action == 'r')
@@ -756,6 +1355,7 @@ mly_process_event(struct mly_softc *sc, struct mly_event *me)
        break;
     case 'e':
        mly_printf(sc, tp, me->target, me->lun);
+       kprintf("\n");
        break;
     case 'c':
        mly_printf(sc, "controller %s\n", tp);
@@ -775,29 +1375,33 @@ static void
 mly_periodic(void *data)
 {
     struct mly_softc   *sc = (struct mly_softc *)data;
-    int                        nchn, bus, target;
+    int                        bus, target;
 
     debug_called(2);
 
     /*
      * Scan devices.
      */
-    nchn = sc->mly_controllerinfo->physical_channels_present +
-       sc->mly_controllerinfo->virtual_channels_present;
-    for (bus = 0; bus < nchn; bus++) {
-       for (target = 0; target < MLY_MAX_TARGETS; target++) {
+    for (bus = 0; bus < sc->mly_cam_channels; bus++) {
+       if (MLY_BUS_IS_VALID(sc, bus)) {
+           for (target = 0; target < MLY_MAX_TARGETS; target++) {
 
-           /* ignore the controller in this scan */
-           if (target == sc->mly_controllerparam->initiator_id)
-               continue;
+               /* ignore the controller in this scan */
+               if (target == sc->mly_controllerparam->initiator_id)
+                   continue;
 
-           /* perform device rescan? */
-           if (sc->mly_btl[bus][target].mb_flags & MLY_BTL_RESCAN)
-               mly_rescan_btl(sc, bus, target);
+               /* perform device rescan? */
+               if (sc->mly_btl[bus][target].mb_flags & MLY_BTL_RESCAN)
+                   mly_rescan_btl(sc, bus, target);
+           }
        }
     }
+    
+    /* check for controller events */
+    mly_check_event(sc);
 
-    callout_reset(&sc->mly_periodic, hz, mly_periodic, sc);
+    /* reschedule ourselves */
+    callout_reset(&sc->mly_periodic, MLY_PERIODIC_INTERVAL * hz, mly_periodic, sc);
 }
 
 /********************************************************************************
@@ -816,7 +1420,7 @@ mly_immediate_command(struct mly_command *mc)
     struct mly_softc   *sc = mc->mc_sc;
     int                        error;
 
-    debug_called(2);
+    debug_called(1);
 
     /* spinning at splcam is ugly, but we're only used during controller init */
     crit_enter();
@@ -841,40 +1445,11 @@ mly_immediate_command(struct mly_command *mc)
 }
 
 /********************************************************************************
- * Start as much queued I/O as possible on the controller
- */
-void
-mly_startio(struct mly_softc *sc)
-{
-    struct mly_command *mc;
-
-    debug_called(2);
-
-    for (;;) {
-
-       /* try for a ready command */
-       mc = mly_dequeue_ready(sc);
-
-       /* try to build a command from a queued ccb */
-       if (!mc)
-           mly_cam_command(sc, &mc);
-
-       /* no command == nothing to do */
-       if (!mc)
-           break;
-
-       /* try to post the command */
-       if (mly_start(mc)) {
-           /* controller busy, or no resources - defer for later */
-           mly_requeue_ready(mc);
-           break;
-       }
-    }
-}
-
-/********************************************************************************
- * Deliver a command to the controller; allocate controller resources at the
- * last moment.
+ * Deliver a command to the controller.
+ *
+ * XXX it would be good to just queue commands that we can't submit immediately
+ *     and send them later, but we probably want a wrapper for that so that
+ *     we don't hang on a failed submission for an immediate command.
  */
 static int
 mly_start(struct mly_command *mc)
@@ -890,6 +1465,10 @@ mly_start(struct mly_command *mc)
     mly_map_command(mc);
     mc->mc_packet->generic.command_id = mc->mc_slot;
 
+#ifdef MLY_DEBUG
+    mc->mc_timestamp = time_second;
+#endif
+
     crit_enter();
 
     /*
@@ -925,11 +1504,13 @@ mly_start(struct mly_command *mc)
        /* copy in new command */
        bcopy(mc->mc_packet->mmbox.data, pkt->mmbox.data, sizeof(pkt->mmbox.data));
        /* barrier to ensure completion of previous write before we write the flag */
-       bus_space_barrier(NULL, NULL, 0, 0, BUS_SPACE_BARRIER_WRITE);   /* tag/handle? */
+       bus_space_barrier(sc->mly_btag, sc->mly_bhandle, 0, 0,
+           BUS_SPACE_BARRIER_WRITE);
        /* copy flag last */
        pkt->mmbox.flag = mc->mc_packet->mmbox.flag;
        /* barrier to ensure completion of previous write before we notify the controller */
-       bus_space_barrier(NULL, NULL, 0, 0, BUS_SPACE_BARRIER_WRITE);   /* tag/handle */
+       bus_space_barrier(sc->mly_btag, sc->mly_bhandle, 0, 0,
+           BUS_SPACE_BARRIER_WRITE);
 
        /* signal controller, update index */
        MLY_SET_REG(sc, sc->mly_idbr, MLY_AM_CMDSENT);
@@ -944,7 +1525,7 @@ mly_start(struct mly_command *mc)
 /********************************************************************************
  * Pick up command status from the controller, schedule a completion event
  */
-void
+static void
 mly_done(struct mly_softc *sc) 
 {
     struct mly_command         *mc;
@@ -1012,11 +1593,9 @@ mly_done(struct mly_softc *sc)
 
     crit_exit();
     if (worked) {
-#if defined(__FreeBSD__) && __FreeBSD_version >= 500005
        if (sc->mly_state & MLY_STATE_INTERRUPTS_ON)
            taskqueue_enqueue(taskqueue_swi, &sc->mly_task_complete);
        else
-#endif
            mly_complete(sc, 0);
     }
 }
@@ -1060,33 +1639,11 @@ mly_complete(void *context, int pending)
            wakeup(mc);
        }
     }
-
+    
     /*
-     * We may have freed up controller resources which would allow us
-     * to push more commands onto the controller, so we check here.
+     * XXX if we are deferring commands due to controller-busy status, we should
+     *     retry submitting them here.
      */
-    mly_startio(sc);
-
-    /*
-     * The controller may have updated the health status information,
-     * so check for it here.
-     *
-     * Note that we only check for health status after a completed command.  It
-     * might be wise to ping the controller occasionally if it's been idle for
-     * a while just to check up on it.  While a filesystem is mounted, or I/O is
-     * active this isn't really an issue.
-     */
-    if (sc->mly_mmbox->mmm_health.status.change_counter != sc->mly_event_change) {
-       sc->mly_event_change = sc->mly_mmbox->mmm_health.status.change_counter;
-       debug(1, "event change %d, event status update, %d -> %d", sc->mly_event_change,
-             sc->mly_event_waiting, sc->mly_mmbox->mmm_health.status.next_event);
-       sc->mly_event_waiting = sc->mly_mmbox->mmm_health.status.next_event;
-
-       /* wake up anyone that might be interested in this */
-       wakeup(&sc->mly_event_change);
-    }
-    if (sc->mly_event_counter != sc->mly_event_waiting)
-       mly_fetch_event(sc);
 }
 
 /********************************************************************************
@@ -1098,7 +1655,7 @@ mly_complete(void *context, int pending)
 /********************************************************************************
  * Allocate a command.
  */
-int
+static int
 mly_alloc_command(struct mly_softc *sc, struct mly_command **mcp)
 {
     struct mly_command *mc;
@@ -1115,7 +1672,7 @@ mly_alloc_command(struct mly_softc *sc, struct mly_command **mcp)
 /********************************************************************************
  * Release a command back to the freelist.
  */
-void
+static void
 mly_release_command(struct mly_command *mc)
 {
     debug_called(3);
@@ -1145,22 +1702,33 @@ mly_release_command(struct mly_command *mc)
 static void
 mly_alloc_commands_map(void *arg, bus_dma_segment_t *segs, int nseg, int error)
 {
-    struct mly_softc   *sc = (struct mly_softc *)arg
+    struct mly_softc   *sc = (struct mly_softc *)arg;
 
-    debug_called(2);
+    debug_called(1);
 
     sc->mly_packetphys = segs[0].ds_addr;
 }
 
 /********************************************************************************
  * Allocate and initialise command and packet structures.
+ *
+ * If the controller supports fewer than MLY_MAX_COMMANDS commands, limit our
+ * allocation to that number.  If we don't yet know how many commands the
+ * controller supports, allocate a very small set (suitable for initialisation
+ * purposes only).
  */
 static int
 mly_alloc_commands(struct mly_softc *sc)
 {
     struct mly_command         *mc;
-    int                                i;
+    int                                i, ncmd;
  
+    if (sc->mly_controllerinfo == NULL) {
+       ncmd = 4;
+    } else {
+       ncmd = min(MLY_MAX_COMMANDS, sc->mly_controllerinfo->maximum_parallel_commands);
+    }
+
     /*
      * Allocate enough space for all the command packets in one chunk and
      * map them permanently into controller-visible space.
@@ -1169,11 +1737,12 @@ mly_alloc_commands(struct mly_softc *sc)
                         BUS_DMA_NOWAIT, &sc->mly_packetmap)) {
        return(ENOMEM);
     }
-    bus_dmamap_load(sc->mly_packet_dmat, sc->mly_packetmap, sc->mly_packet, 
-                   MLY_MAXCOMMANDS * sizeof(union mly_command_packet), 
-                   mly_alloc_commands_map, sc, 0);
+    if (bus_dmamap_load(sc->mly_packet_dmat, sc->mly_packetmap, sc->mly_packet, 
+                       ncmd * sizeof(union mly_command_packet), 
+                       mly_alloc_commands_map, sc, BUS_DMA_NOWAIT) != 0)
+       return (ENOMEM);
 
-    for (i = 0; i < MLY_MAXCOMMANDS; i++) {
+    for (i = 0; i < ncmd; i++) {
        mc = &sc->mly_command[i];
        bzero(mc, sizeof(*mc));
        mc->mc_sc = sc;
@@ -1183,9 +1752,32 @@ mly_alloc_commands(struct mly_softc *sc)
        if (!bus_dmamap_create(sc->mly_buffer_dmat, 0, &mc->mc_datamap))
            mly_release_command(mc);
     }
-    return(0);
+    return(0);
+}
+
+/********************************************************************************
+ * Free all the storage held by commands.
+ *
+ * Must be called with all commands on the free list.
+ */
+static void
+mly_release_commands(struct mly_softc *sc)
+{
+    struct mly_command *mc;
+
+    /* throw away command buffer DMA maps */
+    while (mly_alloc_command(sc, &mc) == 0)
+       bus_dmamap_destroy(sc->mly_buffer_dmat, mc->mc_datamap);
+
+    /* release the packet storage */
+    if (sc->mly_packet != NULL) {
+       bus_dmamap_unload(sc->mly_packet_dmat, sc->mly_packetmap);
+       bus_dmamem_free(sc->mly_packet_dmat, sc->mly_packet, sc->mly_packetmap);
+       sc->mly_packet = NULL;
+    }
 }
 
+
 /********************************************************************************
  * Command-mapping helper function - populate this command's s/g table
  * with the s/g entries for its data.
@@ -1199,14 +1791,14 @@ mly_map_command_sg(void *arg, bus_dma_segment_t *segs, int nseg, int error)
     struct mly_sg_entry                *sg;
     int                                i, tabofs;
 
-    debug_called(3);
+    debug_called(2);
 
     /* can we use the transfer structure directly? */
     if (nseg <= 2) {
        sg = &gen->transfer.direct.sg[0];
        gen->command_control.extended_sg_table = 0;
     } else {
-       tabofs = ((mc->mc_slot - MLY_SLOT_START) * MLY_MAXSGENTRIES);
+       tabofs = ((mc->mc_slot - MLY_SLOT_START) * MLY_MAX_SGENTRIES);
        sg = sc->mly_sg_table + tabofs;
        gen->transfer.indirect.entries[0] = nseg;
        gen->transfer.indirect.table_physaddr[0] = sc->mly_sg_busaddr + (tabofs * sizeof(struct mly_sg_entry));
@@ -1232,7 +1824,7 @@ mly_map_command_cdb(void *arg, bus_dma_segment_t *segs, int nseg, int error)
 {
     struct mly_command                 *mc = (struct mly_command *)arg;
 
-    debug_called(3);
+    debug_called(2);
 
     /* XXX can we safely assume that a CDB will never cross a page boundary? */
     if ((segs[0].ds_addr % PAGE_SIZE) > 
@@ -1259,15 +1851,15 @@ mly_map_command(struct mly_command *mc)
        return;
 
     /* does the command have a data buffer? */
-    if (mc->mc_data != NULL)
+    if (mc->mc_data != NULL) {
        bus_dmamap_load(sc->mly_buffer_dmat, mc->mc_datamap, mc->mc_data, mc->mc_length, 
                        mly_map_command_sg, mc, 0);
        
-    if (mc->mc_flags & MLY_CMD_DATAIN)
-       bus_dmamap_sync(sc->mly_buffer_dmat, mc->mc_datamap, BUS_DMASYNC_PREREAD);
-    if (mc->mc_flags & MLY_CMD_DATAOUT)
-       bus_dmamap_sync(sc->mly_buffer_dmat, mc->mc_datamap, BUS_DMASYNC_PREWRITE);
-
+       if (mc->mc_flags & MLY_CMD_DATAIN)
+           bus_dmamap_sync(sc->mly_buffer_dmat, mc->mc_datamap, BUS_DMASYNC_PREREAD);
+       if (mc->mc_flags & MLY_CMD_DATAOUT)
+           bus_dmamap_sync(sc->mly_buffer_dmat, mc->mc_datamap, BUS_DMASYNC_PREWRITE);
+    }
     mc->mc_flags |= MLY_CMD_MAPPED;
 }
 
@@ -1284,18 +1876,602 @@ mly_unmap_command(struct mly_command *mc)
     if (!(mc->mc_flags & MLY_CMD_MAPPED))
        return;
 
-    if (mc->mc_flags & MLY_CMD_DATAIN)
-       bus_dmamap_sync(sc->mly_buffer_dmat, mc->mc_datamap, BUS_DMASYNC_POSTREAD);
-    if (mc->mc_flags & MLY_CMD_DATAOUT)
-       bus_dmamap_sync(sc->mly_buffer_dmat, mc->mc_datamap, BUS_DMASYNC_POSTWRITE);
-
     /* does the command have a data buffer? */
-    if (mc->mc_data != NULL)
-       bus_dmamap_unload(sc->mly_buffer_dmat, mc->mc_datamap);
+    if (mc->mc_data != NULL) {
+       if (mc->mc_flags & MLY_CMD_DATAIN)
+           bus_dmamap_sync(sc->mly_buffer_dmat, mc->mc_datamap, BUS_DMASYNC_POSTREAD);
+       if (mc->mc_flags & MLY_CMD_DATAOUT)
+           bus_dmamap_sync(sc->mly_buffer_dmat, mc->mc_datamap, BUS_DMASYNC_POSTWRITE);
 
+       bus_dmamap_unload(sc->mly_buffer_dmat, mc->mc_datamap);
+    }
     mc->mc_flags &= ~MLY_CMD_MAPPED;
 }
 
+
+/********************************************************************************
+ ********************************************************************************
+                                                                    CAM interface
+ ********************************************************************************
+ ********************************************************************************/
+
+/********************************************************************************
+ * Attach the physical and virtual SCSI busses to CAM.
+ *
+ * Physical bus numbering starts from 0, virtual bus numbering from one greater
+ * than the highest physical bus.  Physical busses are only registered if
+ * the kernel environment variable "hw.mly.register_physical_channels" is set.
+ *
+ * When we refer to a "bus", we are referring to the bus number registered with
+ * the SIM, wheras a "channel" is a channel number given to the adapter.  In order
+ * to keep things simple, we map these 1:1, so "bus" and "channel" may be used
+ * interchangeably.
+ */
+static int
+mly_cam_attach(struct mly_softc *sc)
+{
+    struct cam_devq    *devq;
+    int                        chn, i;
+
+    debug_called(1);
+
+    /*
+     * Allocate a devq for all our channels combined.
+     */
+    if ((devq = cam_simq_alloc(sc->mly_controllerinfo->maximum_parallel_commands)) == NULL) {
+       mly_printf(sc, "can't allocate CAM SIM queue\n");
+       return(ENOMEM);
+    }
+
+    /*
+     * If physical channel registration has been requested, register these first.
+     * Note that we enable tagged command queueing for physical channels.
+     */
+    if (ktestenv("hw.mly.register_physical_channels")) {
+       chn = 0;
+       for (i = 0; i < sc->mly_controllerinfo->physical_channels_present; i++, chn++) {
+
+           if ((sc->mly_cam_sim[chn] = cam_sim_alloc(mly_cam_action, mly_cam_poll, "mly", sc,
+                                                     device_get_unit(sc->mly_dev),
+                                                     sc->mly_controllerinfo->maximum_parallel_commands,
+                                                     1, devq)) == NULL) {
+               return(ENOMEM);
+           }
+           if (xpt_bus_register(sc->mly_cam_sim[chn], chn)) {
+               mly_printf(sc, "CAM XPT phsyical channel registration failed\n");
+               return(ENXIO);
+           }
+           debug(1, "registered physical channel %d", chn);
+       }
+    }
+
+    /*
+     * Register our virtual channels, with bus numbers matching channel numbers.
+     */
+    chn = sc->mly_controllerinfo->physical_channels_present;
+    for (i = 0; i < sc->mly_controllerinfo->virtual_channels_present; i++, chn++) {
+       if ((sc->mly_cam_sim[chn] = cam_sim_alloc(mly_cam_action, mly_cam_poll, "mly", sc,
+                                                 device_get_unit(sc->mly_dev),
+                                                 sc->mly_controllerinfo->maximum_parallel_commands,
+                                                 0, devq)) == NULL) {
+           return(ENOMEM);
+       }
+       if (xpt_bus_register(sc->mly_cam_sim[chn], chn)) {
+           mly_printf(sc, "CAM XPT virtual channel registration failed\n");
+           return(ENXIO);
+       }
+       debug(1, "registered virtual channel %d", chn);
+    }
+
+    /*
+     * This is the total number of channels that (might have been) registered with
+     * CAM.  Some may not have been; check the mly_cam_sim array to be certain.
+     */
+    sc->mly_cam_channels = sc->mly_controllerinfo->physical_channels_present +
+       sc->mly_controllerinfo->virtual_channels_present;
+
+    return(0);
+}
+
+/********************************************************************************
+ * Detach from CAM
+ */
+static void
+mly_cam_detach(struct mly_softc *sc)
+{
+    int                i;
+    
+    debug_called(1);
+
+    for (i = 0; i < sc->mly_cam_channels; i++) {
+       if (sc->mly_cam_sim[i] != NULL) {
+           xpt_bus_deregister(cam_sim_path(sc->mly_cam_sim[i]));
+           cam_sim_free(sc->mly_cam_sim[i]);
+       }
+    }
+    if (sc->mly_cam_devq != NULL)
+       cam_simq_release(sc->mly_cam_devq);
+}
+
+/************************************************************************
+ * Rescan a device.
+ */ 
+static void
+mly_cam_rescan_btl(struct mly_softc *sc, int bus, int target)
+{
+    union ccb  *ccb;
+
+    debug_called(1);
+
+    ccb = kmalloc(sizeof(union ccb), M_TEMP, M_WAITOK | M_ZERO);
+    
+    if (xpt_create_path(&sc->mly_cam_path, xpt_periph, 
+                       cam_sim_path(sc->mly_cam_sim[bus]), target, 0) != CAM_REQ_CMP) {
+       mly_printf(sc, "rescan failed (can't create path)\n");
+       kfree(ccb, M_TEMP);
+       return;
+    }
+    xpt_setup_ccb(&ccb->ccb_h, sc->mly_cam_path, 5/*priority (low)*/);
+    ccb->ccb_h.func_code = XPT_SCAN_LUN;
+    ccb->ccb_h.cbfcnp = mly_cam_rescan_callback;
+    ccb->crcn.flags = CAM_FLAG_NONE;
+    debug(1, "rescan target %d:%d", bus, target);
+    xpt_action(ccb);
+}
+
+static void
+mly_cam_rescan_callback(struct cam_periph *periph, union ccb *ccb)
+{
+    kfree(ccb, M_TEMP);
+}
+
+/********************************************************************************
+ * Handle an action requested by CAM
+ */
+static void
+mly_cam_action(struct cam_sim *sim, union ccb *ccb)
+{
+    struct mly_softc   *sc = cam_sim_softc(sim);
+
+    debug_called(2);
+
+    switch (ccb->ccb_h.func_code) {
+
+       /* perform SCSI I/O */
+    case XPT_SCSI_IO:
+       if (!mly_cam_action_io(sim, (struct ccb_scsiio *)&ccb->csio))
+           return;
+       break;
+
+       /* perform geometry calculations */
+    case XPT_CALC_GEOMETRY:
+    {
+       struct ccb_calc_geometry        *ccg = &ccb->ccg;
+        u_int32_t                      secs_per_cylinder;
+
+       debug(2, "XPT_CALC_GEOMETRY %d:%d:%d", cam_sim_bus(sim), ccb->ccb_h.target_id, ccb->ccb_h.target_lun);
+
+       if (sc->mly_controllerparam->bios_geometry == MLY_BIOSGEOM_8G) {
+           ccg->heads = 255;
+            ccg->secs_per_track = 63;
+       } else {                                /* MLY_BIOSGEOM_2G */
+           ccg->heads = 128;
+            ccg->secs_per_track = 32;
+       }
+       secs_per_cylinder = ccg->heads * ccg->secs_per_track;
+        ccg->cylinders = ccg->volume_size / secs_per_cylinder;
+        ccb->ccb_h.status = CAM_REQ_CMP;
+        break;
+    }
+
+       /* handle path attribute inquiry */
+    case XPT_PATH_INQ:
+    {
+       struct ccb_pathinq      *cpi = &ccb->cpi;
+
+       debug(2, "XPT_PATH_INQ %d:%d:%d", cam_sim_bus(sim), ccb->ccb_h.target_id, ccb->ccb_h.target_lun);
+
+       cpi->version_num = 1;
+       cpi->hba_inquiry = PI_TAG_ABLE;         /* XXX extra flags for physical channels? */
+       cpi->target_sprt = 0;
+       cpi->hba_misc = 0;
+       cpi->max_target = MLY_MAX_TARGETS - 1;
+       cpi->max_lun = MLY_MAX_LUNS - 1;
+       cpi->initiator_id = sc->mly_controllerparam->initiator_id;
+       strncpy(cpi->sim_vid, "FreeBSD", SIM_IDLEN);
+        strncpy(cpi->hba_vid, "FreeBSD", HBA_IDLEN);
+        strncpy(cpi->dev_name, cam_sim_name(sim), DEV_IDLEN);
+        cpi->unit_number = cam_sim_unit(sim);
+        cpi->bus_id = cam_sim_bus(sim);
+       cpi->base_transfer_speed = 132 * 1024;  /* XXX what to set this to? */
+#ifdef CAM_NEW_TRAN_CODE
+       cpi->transport = XPORT_SPI;
+       cpi->transport_version = 2;
+       cpi->protocol = PROTO_SCSI;
+       cpi->protocol_version = SCSI_REV_2;
+#endif
+       ccb->ccb_h.status = CAM_REQ_CMP;
+       break;
+    }
+
+    case XPT_GET_TRAN_SETTINGS:
+    {
+       struct ccb_trans_settings       *cts = &ccb->cts;
+       int                             bus, target;
+#ifdef CAM_NEW_TRAN_CODE
+       struct ccb_trans_settings_scsi *scsi = &cts->proto_specific.scsi;
+       struct ccb_trans_settings_spi *spi = &cts->xport_specific.spi;
+
+       cts->protocol = PROTO_SCSI;
+       cts->protocol_version = SCSI_REV_2;
+       cts->transport = XPORT_SPI;
+       cts->transport_version = 2;
+
+       scsi->flags = 0;
+       scsi->valid = 0;
+       spi->flags = 0;
+       spi->valid = 0;
+
+       bus = cam_sim_bus(sim);
+       target = cts->ccb_h.target_id;
+       debug(2, "XPT_GET_TRAN_SETTINGS %d:%d", bus, target);
+       /* logical device? */
+       if (sc->mly_btl[bus][target].mb_flags & MLY_BTL_LOGICAL) {
+           /* nothing special for these */
+       /* physical device? */
+       } else if (sc->mly_btl[bus][target].mb_flags & MLY_BTL_PHYSICAL) {
+           /* allow CAM to try tagged transactions */
+           scsi->flags |= CTS_SCSI_FLAGS_TAG_ENB;
+           scsi->valid |= CTS_SCSI_VALID_TQ;
+
+           /* convert speed (MHz) to usec */
+           if (sc->mly_btl[bus][target].mb_speed == 0) {
+               spi->sync_period = 1000000 / 5;
+           } else {
+               spi->sync_period = 1000000 / sc->mly_btl[bus][target].mb_speed;
+           }
+
+           /* convert bus width to CAM internal encoding */
+           switch (sc->mly_btl[bus][target].mb_width) {
+           case 32:
+               spi->bus_width = MSG_EXT_WDTR_BUS_32_BIT;
+               break;
+           case 16:
+               spi->bus_width = MSG_EXT_WDTR_BUS_16_BIT;
+               break;
+           case 8:
+           default:
+               spi->bus_width = MSG_EXT_WDTR_BUS_8_BIT;
+               break;
+           }
+           spi->valid |= CTS_SPI_VALID_SYNC_RATE | CTS_SPI_VALID_BUS_WIDTH;
+
+           /* not a device, bail out */
+       } else {
+           cts->ccb_h.status = CAM_REQ_CMP_ERR;
+           break;
+       }
+
+       /* disconnect always OK */
+       spi->flags |= CTS_SPI_FLAGS_DISC_ENB;
+       spi->valid |= CTS_SPI_VALID_DISC;
+#else
+       cts->valid = 0;
+
+       bus = cam_sim_bus(sim);
+       target = cts->ccb_h.target_id;
+       /* XXX validate bus/target? */
+
+       debug(2, "XPT_GET_TRAN_SETTINGS %d:%d", bus, target);
+
+       /* logical device? */
+       if (sc->mly_btl[bus][target].mb_flags & MLY_BTL_LOGICAL) {
+           /* nothing special for these */
+
+       /* physical device? */
+       } else if (sc->mly_btl[bus][target].mb_flags & MLY_BTL_PHYSICAL) {
+           /* allow CAM to try tagged transactions */
+           cts->flags |= CCB_TRANS_TAG_ENB;
+           cts->valid |= CCB_TRANS_TQ_VALID;
+
+           /* convert speed (MHz) to usec */
+           if (sc->mly_btl[bus][target].mb_speed == 0) {
+               cts->sync_period = 1000000 / 5;
+           } else {
+               cts->sync_period = 1000000 / sc->mly_btl[bus][target].mb_speed;
+           }
+
+           /* convert bus width to CAM internal encoding */
+           switch (sc->mly_btl[bus][target].mb_width) {
+           case 32:
+               cts->bus_width = MSG_EXT_WDTR_BUS_32_BIT;
+               break;
+           case 16:
+               cts->bus_width = MSG_EXT_WDTR_BUS_16_BIT;
+               break;
+           case 8:
+           default:
+               cts->bus_width = MSG_EXT_WDTR_BUS_8_BIT;
+               break;
+           }
+           cts->valid |= CCB_TRANS_SYNC_RATE_VALID | CCB_TRANS_BUS_WIDTH_VALID;
+
+           /* not a device, bail out */
+       } else {
+           cts->ccb_h.status = CAM_REQ_CMP_ERR;
+           break;
+       }
+
+       /* disconnect always OK */
+       cts->flags |= CCB_TRANS_DISC_ENB;
+       cts->valid |= CCB_TRANS_DISC_VALID;
+#endif
+
+       cts->ccb_h.status = CAM_REQ_CMP;
+       break;
+    }
+
+    default:           /* we can't do this */
+       debug(2, "unsupported func_code = 0x%x", ccb->ccb_h.func_code);
+       ccb->ccb_h.status = CAM_REQ_INVALID;
+       break;
+    }
+
+    xpt_done(ccb);
+}
+
+/********************************************************************************
+ * Handle an I/O operation requested by CAM
+ */
+static int
+mly_cam_action_io(struct cam_sim *sim, struct ccb_scsiio *csio)
+{
+    struct mly_softc                   *sc = cam_sim_softc(sim);
+    struct mly_command                 *mc;
+    struct mly_command_scsi_small      *ss;
+    int                                        bus, target;
+    int                                        error;
+
+    bus = cam_sim_bus(sim);
+    target = csio->ccb_h.target_id;
+
+    debug(2, "XPT_SCSI_IO %d:%d:%d", bus, target, csio->ccb_h.target_lun);
+
+    /* validate bus number */
+    if (!MLY_BUS_IS_VALID(sc, bus)) {
+       debug(0, " invalid bus %d", bus);
+       csio->ccb_h.status = CAM_REQ_CMP_ERR;
+    }
+
+    /*  check for I/O attempt to a protected device */
+    if (sc->mly_btl[bus][target].mb_flags & MLY_BTL_PROTECTED) {
+       debug(2, "  device protected");
+       csio->ccb_h.status = CAM_REQ_CMP_ERR;
+    }
+
+    /* check for I/O attempt to nonexistent device */
+    if (!(sc->mly_btl[bus][target].mb_flags & (MLY_BTL_LOGICAL | MLY_BTL_PHYSICAL))) {
+       debug(2, "  device %d:%d does not exist", bus, target);
+       csio->ccb_h.status = CAM_REQ_CMP_ERR;
+    }
+
+    /* XXX increase if/when we support large SCSI commands */
+    if (csio->cdb_len > MLY_CMD_SCSI_SMALL_CDB) {
+       debug(0, "  command too large (%d > %d)", csio->cdb_len, MLY_CMD_SCSI_SMALL_CDB);
+       csio->ccb_h.status = CAM_REQ_CMP_ERR;
+    }
+
+    /* check that the CDB pointer is not to a physical address */
+    if ((csio->ccb_h.flags & CAM_CDB_POINTER) && (csio->ccb_h.flags & CAM_CDB_PHYS)) {
+       debug(0, "  CDB pointer is to physical address");
+       csio->ccb_h.status = CAM_REQ_CMP_ERR;
+    }
+
+    /* if there is data transfer, it must be to/from a virtual address */
+    if ((csio->ccb_h.flags & CAM_DIR_MASK) != CAM_DIR_NONE) {
+       if (csio->ccb_h.flags & CAM_DATA_PHYS) {                /* we can't map it */
+           debug(0, "  data pointer is to physical address");
+           csio->ccb_h.status = CAM_REQ_CMP_ERR;
+       }
+       if (csio->ccb_h.flags & CAM_SCATTER_VALID) {    /* we want to do the s/g setup */
+           debug(0, "  data has premature s/g setup");
+           csio->ccb_h.status = CAM_REQ_CMP_ERR;
+       }
+    }
+
+    /* abandon aborted ccbs or those that have failed validation */
+    if ((csio->ccb_h.status & CAM_STATUS_MASK) != CAM_REQ_INPROG) {
+       debug(2, "abandoning CCB due to abort/validation failure");
+       return(EINVAL);
+    }
+
+    /*
+     * Get a command, or push the ccb back to CAM and freeze the queue.
+     */
+    if ((error = mly_alloc_command(sc, &mc))) {
+       crit_enter();
+       xpt_freeze_simq(sim, 1);
+       csio->ccb_h.status |= CAM_REQUEUE_REQ;
+       sc->mly_qfrzn_cnt++;
+       crit_exit();
+       return(error);
+    }
+    
+    /* build the command */
+    mc->mc_data = csio->data_ptr;
+    mc->mc_length = csio->dxfer_len;
+    mc->mc_complete = mly_cam_complete;
+    mc->mc_private = csio;
+
+    /* save the bus number in the ccb for later recovery XXX should be a better way */
+     csio->ccb_h.sim_priv.entries[0].field = bus;
+
+    /* build the packet for the controller */
+    ss = &mc->mc_packet->scsi_small;
+    ss->opcode = MDACMD_SCSI;
+    if (csio->ccb_h.flags & CAM_DIS_DISCONNECT)
+       ss->command_control.disable_disconnect = 1;
+    if ((csio->ccb_h.flags & CAM_DIR_MASK) == CAM_DIR_OUT)
+       ss->command_control.data_direction = MLY_CCB_WRITE;
+    ss->data_size = csio->dxfer_len;
+    ss->addr.phys.lun = csio->ccb_h.target_lun;
+    ss->addr.phys.target = csio->ccb_h.target_id;
+    ss->addr.phys.channel = bus;
+    if (csio->ccb_h.timeout < (60 * 1000)) {
+       ss->timeout.value = csio->ccb_h.timeout / 1000;
+       ss->timeout.scale = MLY_TIMEOUT_SECONDS;
+    } else if (csio->ccb_h.timeout < (60 * 60 * 1000)) {
+       ss->timeout.value = csio->ccb_h.timeout / (60 * 1000);
+       ss->timeout.scale = MLY_TIMEOUT_MINUTES;
+    } else {
+       ss->timeout.value = csio->ccb_h.timeout / (60 * 60 * 1000);     /* overflow? */
+       ss->timeout.scale = MLY_TIMEOUT_HOURS;
+    }
+    ss->maximum_sense_size = csio->sense_len;
+    ss->cdb_length = csio->cdb_len;
+    if (csio->ccb_h.flags & CAM_CDB_POINTER) {
+       bcopy(csio->cdb_io.cdb_ptr, ss->cdb, csio->cdb_len);
+    } else {
+       bcopy(csio->cdb_io.cdb_bytes, ss->cdb, csio->cdb_len);
+    }
+
+    /* give the command to the controller */
+    if ((error = mly_start(mc))) {
+       crit_enter();
+       xpt_freeze_simq(sim, 1);
+       csio->ccb_h.status |= CAM_REQUEUE_REQ;
+       sc->mly_qfrzn_cnt++;
+       crit_exit();
+       return(error);
+    }
+
+    return(0);
+}
+
+/********************************************************************************
+ * Check for possibly-completed commands.
+ */
+static void
+mly_cam_poll(struct cam_sim *sim)
+{
+    struct mly_softc   *sc = cam_sim_softc(sim);
+
+    debug_called(2);
+
+    mly_done(sc);
+}
+
+/********************************************************************************
+ * Handle completion of a command - pass results back through the CCB
+ */
+static void
+mly_cam_complete(struct mly_command *mc)
+{
+    struct mly_softc           *sc = mc->mc_sc;
+    struct ccb_scsiio          *csio = (struct ccb_scsiio *)mc->mc_private;
+    struct scsi_inquiry_data   *inq = (struct scsi_inquiry_data *)csio->data_ptr;
+    struct mly_btl             *btl;
+    u_int8_t                   cmd;
+    int                                bus, target;
+
+    debug_called(2);
+
+    csio->scsi_status = mc->mc_status;
+    switch(mc->mc_status) {
+    case SCSI_STATUS_OK:
+       /*
+        * In order to report logical device type and status, we overwrite
+        * the result of the INQUIRY command to logical devices.
+        */
+       bus = csio->ccb_h.sim_priv.entries[0].field;
+       target = csio->ccb_h.target_id;
+       /* XXX validate bus/target? */
+       if (sc->mly_btl[bus][target].mb_flags & MLY_BTL_LOGICAL) {
+           if (csio->ccb_h.flags & CAM_CDB_POINTER) {
+               cmd = *csio->cdb_io.cdb_ptr;
+           } else {
+               cmd = csio->cdb_io.cdb_bytes[0];
+           }
+           if (cmd == INQUIRY) {
+               btl = &sc->mly_btl[bus][target];
+               padstr(inq->vendor, mly_describe_code(mly_table_device_type, btl->mb_type), 8);
+               padstr(inq->product, mly_describe_code(mly_table_device_state, btl->mb_state), 16);
+               padstr(inq->revision, "", 4);
+           }
+       }
+
+       debug(2, "SCSI_STATUS_OK");
+       csio->ccb_h.status = CAM_REQ_CMP;
+       break;
+
+    case SCSI_STATUS_CHECK_COND:
+       debug(1, "SCSI_STATUS_CHECK_COND  sense %d  resid %d", mc->mc_sense, mc->mc_resid);
+       csio->ccb_h.status = CAM_SCSI_STATUS_ERROR;
+       bzero(&csio->sense_data, SSD_FULL_SIZE);
+       bcopy(mc->mc_packet, &csio->sense_data, mc->mc_sense);
+       csio->sense_len = mc->mc_sense;
+       csio->ccb_h.status |= CAM_AUTOSNS_VALID;
+       csio->resid = mc->mc_resid;     /* XXX this is a signed value... */
+       break;
+
+    case SCSI_STATUS_BUSY:
+       debug(1, "SCSI_STATUS_BUSY");
+       csio->ccb_h.status = CAM_SCSI_BUSY;
+       break;
+
+    default:
+       debug(1, "unknown status 0x%x", csio->scsi_status);
+       csio->ccb_h.status = CAM_REQ_CMP_ERR;
+       break;
+    }
+
+    crit_enter();
+    if (sc->mly_qfrzn_cnt) {
+       csio->ccb_h.status |= CAM_RELEASE_SIMQ;
+       sc->mly_qfrzn_cnt--;
+    }
+    crit_exit();
+
+    xpt_done((union ccb *)csio);
+    mly_release_command(mc);
+}
+
+/********************************************************************************
+ * Find a peripheral attahed at (bus),(target)
+ */
+static struct cam_periph *
+mly_find_periph(struct mly_softc *sc, int bus, int target)
+{
+    struct cam_periph  *periph;
+    struct cam_path    *path;
+    int                        status;
+
+    status = xpt_create_path(&path, NULL, cam_sim_path(sc->mly_cam_sim[bus]), target, 0);
+    if (status == CAM_REQ_CMP) {
+       periph = cam_periph_find(path, NULL);
+       xpt_free_path(path);
+    } else {
+       periph = NULL;
+    }
+    return(periph);
+}
+
+/********************************************************************************
+ * Name the device at (bus)(target)
+ */
+static int
+mly_name_device(struct mly_softc *sc, int bus, int target)
+{
+    struct cam_periph  *periph;
+
+    if ((periph = mly_find_periph(sc, bus, target)) != NULL) {
+       ksprintf(sc->mly_btl[bus][target].mb_name, "%s%d", periph->periph_name, periph->unit_number);
+       return(0);
+    }
+    sc->mly_btl[bus][target].mb_name[0] = 0;
+    return(ENOENT);
+}
+
 /********************************************************************************
  ********************************************************************************
                                                                  Hardware Control
@@ -1657,7 +2833,6 @@ mly_panic(struct mly_softc *sc, char *reason)
     mly_printstate(sc);
     panic(reason);
 }
-#endif
 
 /********************************************************************************
  * Print queue statistics, callable from DDB.
@@ -1673,14 +2848,13 @@ mly_print_controller(int controller)
        device_printf(sc->mly_dev, "queue    curr max\n");
        device_printf(sc->mly_dev, "free     %04d/%04d\n", 
                      sc->mly_qstat[MLYQ_FREE].q_length, sc->mly_qstat[MLYQ_FREE].q_max);
-       device_printf(sc->mly_dev, "ready    %04d/%04d\n", 
-                     sc->mly_qstat[MLYQ_READY].q_length, sc->mly_qstat[MLYQ_READY].q_max);
        device_printf(sc->mly_dev, "busy     %04d/%04d\n", 
                      sc->mly_qstat[MLYQ_BUSY].q_length, sc->mly_qstat[MLYQ_BUSY].q_max);
        device_printf(sc->mly_dev, "complete %04d/%04d\n", 
                      sc->mly_qstat[MLYQ_COMPLETE].q_length, sc->mly_qstat[MLYQ_COMPLETE].q_max);
     }
 }
+#endif
 
 
 /********************************************************************************
@@ -1750,8 +2924,8 @@ mly_user_ioctl(struct dev_ioctl_args *ap)
 static int
 mly_user_command(struct mly_softc *sc, struct mly_user_command *uc)
 {
-    struct mly_command                 *mc;
-    int                                        error;
+    struct mly_command *mc;
+    int                        error;
 
     /* allocate a command */
     if (mly_alloc_command(sc, &mc)) {
@@ -1780,9 +2954,9 @@ mly_user_command(struct mly_softc *sc, struct mly_user_command *uc)
     mc->mc_complete = NULL;
 
     /* execute the command */
+    if ((error = mly_start(mc)) != 0)
+       goto out;
     crit_enter();
-    mly_requeue_ready(mc);
-    mly_startio(sc);
     while (!(mc->mc_flags & MLY_CMD_COMPLETE))
        tsleep(mc, 0, "mlyioctl", 0);
     crit_exit();
@@ -1841,3 +3015,24 @@ mly_user_health(struct mly_softc *sc, struct mly_user_health *uh)
                    sizeof(uh->HealthStatusBuffer));
     return(error);
 }
+
+static int
+mly_timeout(struct mly_softc *sc)
+{
+       struct mly_command *mc;
+       int deadline;
+
+       deadline = time_second - MLY_CMD_TIMEOUT;
+       TAILQ_FOREACH(mc, &sc->mly_busy, mc_link) {
+               if ((mc->mc_timestamp < deadline)) {
+                       device_printf(sc->mly_dev,
+                           "COMMAND %p TIMEOUT AFTER %d SECONDS\n", mc,
+                           (int)(time_second - mc->mc_timestamp));
+               }
+       }
+
+       callout_reset(&sc->mly_timeout, MLY_CMD_TIMEOUT * hz,
+                     (timeout_t *)mly_timeout, sc);
+
+       return (0);
+}
diff --git a/sys/dev/raid/mly/mly_cam.c b/sys/dev/raid/mly/mly_cam.c
deleted file mode 100644 (file)
index 8ece100..0000000
+++ /dev/null
@@ -1,590 +0,0 @@
-/*-
- * Copyright (c) 2000, 2001 Michael Smith
- * Copyright (c) 2000 BSDi
- * All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY THE AUTHOR AND CONTRIBUTORS ``AS IS'' AND
- * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
- * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
- * ARE DISCLAIMED.  IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE LIABLE
- * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
- * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
- * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
- * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
- * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
- * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
- * SUCH DAMAGE.
- *
- *     $FreeBSD: src/sys/dev/mly/mly_cam.c,v 1.1.2.3 2001/04/21 04:09:06 msmith Exp $
- *     $DragonFly: src/sys/dev/raid/mly/Attic/mly_cam.c,v 1.8 2007/05/13 18:33:57 swildner Exp $
- */
-/*
- * CAM interface for FreeBSD
- */
-
-#include <sys/param.h>
-#include <sys/systm.h>
-#include <sys/bus.h>
-#include <sys/devicestat.h>
-#include <sys/thread2.h>
-
-#include <bus/cam/cam.h>
-#include <bus/cam/cam_ccb.h>
-#include <bus/cam/cam_periph.h>
-#include <bus/cam/cam_sim.h>
-#include <bus/cam/cam_xpt_sim.h>
-#include <bus/cam/scsi/scsi_all.h>
-#include <bus/cam/scsi/scsi_message.h>
-
-#include "mlyreg.h"
-#include "mlyio.h"
-#include "mlyvar.h"
-#include "mly_tables.h"
-
-static void                    mly_cam_poll(struct cam_sim *sim);
-static void                    mly_cam_action(struct cam_sim *sim, union ccb *ccb);
-static void                    mly_cam_complete(struct mly_command *mc);
-static struct cam_periph       *mly_find_periph(struct mly_softc *sc, int bus, int target);
-
-/********************************************************************************
- * CAM-specific queue primitives
- */
-static __inline void
-mly_initq_ccb(struct mly_softc *sc)
-{
-    TAILQ_INIT(&sc->mly_cam_ccbq);
-    MLYQ_INIT(sc, MLYQ_CCB);
-}
-
-static __inline void
-mly_enqueue_ccb(struct mly_softc *sc, union ccb *ccb)
-{
-    crit_enter();
-    TAILQ_INSERT_TAIL(&sc->mly_cam_ccbq, &ccb->ccb_h, sim_links.tqe);
-    MLYQ_ADD(sc, MLYQ_CCB);
-    crit_exit();
-}
-
-static __inline void
-mly_requeue_ccb(struct mly_softc *sc, union ccb *ccb)
-{
-    crit_enter();
-    TAILQ_INSERT_HEAD(&sc->mly_cam_ccbq, &ccb->ccb_h, sim_links.tqe);
-    MLYQ_ADD(sc, MLYQ_CCB);
-    crit_exit();
-}
-
-static __inline union ccb *
-mly_dequeue_ccb(struct mly_softc *sc)
-{
-    union ccb  *ccb;
-
-    crit_enter();
-    if ((ccb = (union ccb *)TAILQ_FIRST(&sc->mly_cam_ccbq)) != NULL) {
-       TAILQ_REMOVE(&sc->mly_cam_ccbq, &ccb->ccb_h, sim_links.tqe);
-       MLYQ_REMOVE(sc, MLYQ_CCB);
-    }
-    crit_exit();
-    return(ccb);
-}
-
-/********************************************************************************
- * space-fill a character string
- */
-static __inline void
-padstr(char *targ, char *src, int len)
-{
-    while (len-- > 0) {
-       if (*src != 0) {
-           *targ++ = *src++;
-       } else {
-           *targ++ = ' ';
-       }
-    }
-}
-
-/********************************************************************************
- * Attach the real and virtual SCSI busses to CAM
- */
-int
-mly_cam_attach(struct mly_softc *sc)
-{
-    struct cam_devq    *devq;
-    int                        chn, i;
-
-    debug_called(1);
-
-    /* initialise the CCB queue */
-    mly_initq_ccb(sc);
-
-    /*
-     * Allocate a devq for all our channels combined.
-     */
-    if ((devq = cam_simq_alloc(sc->mly_controllerinfo->maximum_parallel_commands)) == NULL) {
-       mly_printf(sc, "can't allocate CAM SIM\n");
-       return(ENOMEM);
-    }
-
-    /*
-     * Iterate over channels, registering them with CAM.
-     *
-     * Physical channels are set up to support tagged commands and only a single
-     * untagged command.  Virtual channels do not support tags, and don't need them.
-     */
-    for (i = 0, chn = 0; i < sc->mly_controllerinfo->physical_channels_present; i++, chn++) {
-       /* allocate a sim */
-       if ((sc->mly_cam_sim[chn] = cam_sim_alloc(mly_cam_action, 
-                                                 mly_cam_poll, 
-                                                 "mly", 
-                                                 sc,
-                                                 device_get_unit(sc->mly_dev), 
-                                                 1,
-                                                 sc->mly_controllerinfo->maximum_parallel_commands,
-                                                 devq)) ==  NULL) {
-           mly_printf(sc, "CAM SIM attach failed\n");
-           return(ENOMEM);
-       }
-    }
-    for (i = 0; i < sc->mly_controllerinfo->virtual_channels_present; i++, chn++) {
-       /* allocate a sim */
-       if ((sc->mly_cam_sim[chn] = cam_sim_alloc(mly_cam_action, 
-                                                 mly_cam_poll, 
-                                                 "mly", 
-                                                 sc,
-                                                 device_get_unit(sc->mly_dev), 
-                                                 sc->mly_controllerinfo->maximum_parallel_commands,
-                                                 0,
-                                                 devq)) ==  NULL) {
-           mly_printf(sc, "CAM SIM attach failed\n");
-           return(ENOMEM);
-       }
-    }
-    cam_simq_release(devq);
-
-    for (i = 0; i < chn; i++) {
-       /* register the bus IDs so we can get them later */
-       if (xpt_bus_register(sc->mly_cam_sim[i], i)) {
-           mly_printf(sc, "CAM XPT bus registration failed\n");
-           return(ENXIO);
-       }
-       debug(1, "registered sim %p bus %d", sc->mly_cam_sim[i], i);
-    }
-
-    return(0);
-}
-
-/********************************************************************************
- * Detach from CAM
- */
-void
-mly_cam_detach(struct mly_softc *sc)
-{
-    int                chn, nchn;
-
-    debug_called(1);
-
-    nchn = sc->mly_controllerinfo->physical_channels_present +
-       sc->mly_controllerinfo->virtual_channels_present;
-
-    /*
-     * Iterate over channels, deregistering as we go.
-     */
-    nchn = sc->mly_controllerinfo->physical_channels_present +
-       sc->mly_controllerinfo->virtual_channels_present;
-    for (chn = 0; chn < nchn; chn++) {
-
-       /*
-        * If a sim was registered for this channel, free it.
-        */
-       if (sc->mly_cam_sim[chn] != NULL) {
-           debug(1, "deregister bus %d", chn);
-           xpt_bus_deregister(cam_sim_path(sc->mly_cam_sim[chn]));
-           debug(1, "free sim for channel %d", chn);
-           cam_sim_free(sc->mly_cam_sim[chn]);
-       }
-    }
-}
-
-/********************************************************************************
- * Handle an action requested by CAM
- */
-static void
-mly_cam_action(struct cam_sim *sim, union ccb *ccb)
-{
-    struct mly_softc   *sc = cam_sim_softc(sim);
-
-    debug_called(2);
-
-    switch (ccb->ccb_h.func_code) {
-
-       /* perform SCSI I/O */
-    case XPT_SCSI_IO:
-    {
-       struct ccb_scsiio       *csio = &ccb->csio;
-       int                     bus, target;
-
-       bus = cam_sim_bus(sim);
-       target = csio->ccb_h.target_id;
-
-       debug(2, "XPT_SCSI_IO %d:%d:%d", bus, target, ccb->ccb_h.target_lun);
-
-       /*  check for I/O attempt to a protected device */
-       if (sc->mly_btl[bus][target].mb_flags & MLY_BTL_PROTECTED) {
-           debug(2, "  device protected");
-           csio->ccb_h.status = CAM_REQ_CMP_ERR;
-       }
-
-       /* check for I/O attempt to nonexistent device */
-       if (!(sc->mly_btl[bus][target].mb_flags & (MLY_BTL_LOGICAL | MLY_BTL_PHYSICAL))) {
-           debug(2, "  device does not exist");
-           csio->ccb_h.status = CAM_REQ_CMP_ERR;
-       }
-
-       /* XXX increase if/when we support large SCSI commands */
-       if (csio->cdb_len > MLY_CMD_SCSI_SMALL_CDB) {
-           debug(2, "  command too large (%d > %d)", csio->cdb_len, MLY_CMD_SCSI_SMALL_CDB);
-           csio->ccb_h.status = CAM_REQ_CMP_ERR;
-       }
-
-        /* check that the CDB pointer is not to a physical address */
-        if ((csio->ccb_h.flags & CAM_CDB_POINTER) && (csio->ccb_h.flags & CAM_CDB_PHYS)) {
-           debug(2, "  CDB pointer is to physical address");
-            csio->ccb_h.status = CAM_REQ_CMP_ERR;
-       }
-
-        /* if there is data transfer, it must be to/from a virtual address */
-        if ((csio->ccb_h.flags & CAM_DIR_MASK) != CAM_DIR_NONE) {
-            if (csio->ccb_h.flags & CAM_DATA_PHYS) {           /* we can't map it */
-               debug(2, "  data pointer is to physical address");
-                csio->ccb_h.status = CAM_REQ_CMP_ERR;
-           }
-            if (csio->ccb_h.flags & CAM_SCATTER_VALID) {       /* we want to do the s/g setup */
-               debug(2, "  data has premature s/g setup");
-                csio->ccb_h.status = CAM_REQ_CMP_ERR;
-           }
-        }
-
-       /* abandon aborted ccbs or those that have failed validation */
-       if ((csio->ccb_h.status & CAM_STATUS_MASK) != CAM_REQ_INPROG) {
-           debug(2, "abandoning CCB due to abort/validation failure");
-           break;
-       }
-
-       /* save the channel number in the ccb */
-       csio->ccb_h.sim_priv.entries[0].field = bus;
-
-       /* enqueue the ccb and start I/O */
-       mly_enqueue_ccb(sc, ccb);
-       mly_startio(sc);
-       return;
-    }
-
-       /* perform geometry calculations */
-    case XPT_CALC_GEOMETRY:
-    {
-       struct ccb_calc_geometry        *ccg = &ccb->ccg;
-        u_int32_t                      secs_per_cylinder;
-
-       debug(2, "XPT_CALC_GEOMETRY %d:%d:%d", cam_sim_bus(sim), ccb->ccb_h.target_id, ccb->ccb_h.target_lun);
-
-       if (sc->mly_controllerparam->bios_geometry == MLY_BIOSGEOM_8G) {
-           ccg->heads = 255;
-            ccg->secs_per_track = 63;
-       } else {                                /* MLY_BIOSGEOM_2G */
-           ccg->heads = 128;
-            ccg->secs_per_track = 32;
-       }
-       secs_per_cylinder = ccg->heads * ccg->secs_per_track;
-        ccg->cylinders = ccg->volume_size / secs_per_cylinder;
-        ccb->ccb_h.status = CAM_REQ_CMP;
-        break;
-    }
-
-       /* handle path attribute inquiry */
-    case XPT_PATH_INQ:
-    {
-       struct ccb_pathinq      *cpi = &ccb->cpi;
-
-       debug(2, "XPT_PATH_INQ %d:%d:%d", cam_sim_bus(sim), ccb->ccb_h.target_id, ccb->ccb_h.target_lun);
-
-       cpi->version_num = 1;
-       cpi->hba_inquiry = PI_TAG_ABLE;         /* XXX extra flags for physical channels? */
-       cpi->target_sprt = 0;
-       cpi->hba_misc = 0;
-       cpi->max_target = MLY_MAX_TARGETS - 1;
-       cpi->max_lun = MLY_MAX_LUNS - 1;
-       cpi->initiator_id = sc->mly_controllerparam->initiator_id;
-       strncpy(cpi->sim_vid, "FreeBSD", SIM_IDLEN);
-        strncpy(cpi->hba_vid, "BSDi", HBA_IDLEN);
-        strncpy(cpi->dev_name, cam_sim_name(sim), DEV_IDLEN);
-        cpi->unit_number = cam_sim_unit(sim);
-        cpi->bus_id = cam_sim_bus(sim);
-       cpi->base_transfer_speed = 132 * 1024;  /* XXX what to set this to? */
-       ccb->ccb_h.status = CAM_REQ_CMP;
-       break;
-    }
-
-    case XPT_GET_TRAN_SETTINGS:
-    {
-       struct ccb_trans_settings       *cts = &ccb->cts;
-       int                             bus, target;
-
-       bus = cam_sim_bus(sim);
-       target = cts->ccb_h.target_id;
-
-       debug(2, "XPT_GET_TRAN_SETTINGS %d:%d", bus, target);
-       cts->valid = 0;
-
-       /* logical device? */
-       if (sc->mly_btl[bus][target].mb_flags & MLY_BTL_LOGICAL) {
-           /* nothing special for these */
-
-       /* physical device? */
-       } else if (sc->mly_btl[bus][target].mb_flags & MLY_BTL_PHYSICAL) {
-           /* allow CAM to try tagged transactions */
-           cts->flags |= CCB_TRANS_TAG_ENB;
-           cts->valid |= CCB_TRANS_TQ_VALID;
-
-           /* convert speed (MHz) to usec */
-           if (sc->mly_btl[bus][target].mb_speed == 0) {
-               cts->sync_period = 1000000 / 5;
-           } else {
-               cts->sync_period = 1000000 / sc->mly_btl[bus][target].mb_speed;
-           }
-
-           /* convert bus width to CAM internal encoding */
-           switch (sc->mly_btl[bus][target].mb_width) {
-           case 32:
-               cts->bus_width = MSG_EXT_WDTR_BUS_32_BIT;
-               break;
-           case 16:
-               cts->bus_width = MSG_EXT_WDTR_BUS_16_BIT;
-               break;
-           case 8:
-           default:
-               cts->bus_width = MSG_EXT_WDTR_BUS_8_BIT;
-               break;
-           }
-           cts->valid |= CCB_TRANS_SYNC_RATE_VALID | CCB_TRANS_BUS_WIDTH_VALID;
-
-           /* not a device, bail out */
-       } else {
-           cts->ccb_h.status = CAM_REQ_CMP_ERR;
-           break;
-       }
-
-       /* disconnect always OK */
-       cts->flags |= CCB_TRANS_DISC_ENB;
-       cts->valid |= CCB_TRANS_DISC_VALID;
-
-       cts->ccb_h.status = CAM_REQ_CMP;
-       break;
-    }
-
-    default:           /* we can't do this */
-       debug(2, "unsupported func_code = 0x%x", ccb->ccb_h.func_code);
-       ccb->ccb_h.status = CAM_REQ_INVALID;
-       break;
-    }
-
-    xpt_done(ccb);
-}
-
-/********************************************************************************
- * Check for possibly-completed commands.
- */
-static void
-mly_cam_poll(struct cam_sim *sim)
-{
-    struct mly_softc   *sc = cam_sim_softc(sim);
-
-    debug_called(2);
-
-    mly_done(sc);
-}
-
-/********************************************************************************
- * Pull a CCB off the work queue and turn it into a command.
- */
-int
-mly_cam_command(struct mly_softc *sc, struct mly_command **mcp)
-{
-    struct mly_command                 *mc;
-    struct mly_command_scsi_small      *ss;
-    struct ccb_scsiio                  *csio;
-    int                                        error;
-
-    debug_called(2);
-
-    error = 0;
-    mc = NULL;
-    csio = NULL;
-
-    /* check for a CCB */
-    if (!(csio = (struct ccb_scsiio *)mly_dequeue_ccb(sc)))
-       goto out;
-
-    /* get a command to back it */
-    if (mly_alloc_command(sc, &mc)) {
-       error = ENOMEM;
-       goto out;
-    }
-
-    /* build the command */
-    mc->mc_data = csio->data_ptr;
-    mc->mc_length = csio->dxfer_len;
-    mc->mc_complete = mly_cam_complete;
-    mc->mc_private = csio;
-
-    /* build the packet for the controller */
-    ss = &mc->mc_packet->scsi_small;
-    ss->opcode = MDACMD_SCSI;
-    if (csio->ccb_h.flags * CAM_DIS_DISCONNECT)
-       ss->command_control.disable_disconnect = 1;
-    if ((csio->ccb_h.flags & CAM_DIR_MASK) == CAM_DIR_OUT)
-       ss->command_control.data_direction = MLY_CCB_WRITE;
-    ss->data_size = csio->dxfer_len;
-    ss->addr.phys.lun = csio->ccb_h.target_lun;
-    ss->addr.phys.target = csio->ccb_h.target_id;
-    ss->addr.phys.channel = csio->ccb_h.sim_priv.entries[0].field;
-    if (csio->ccb_h.timeout < (60 * 1000)) {
-       ss->timeout.value = csio->ccb_h.timeout / 1000;
-       ss->timeout.scale = MLY_TIMEOUT_SECONDS;
-    } else if (csio->ccb_h.timeout < (60 * 60 * 1000)) {
-       ss->timeout.value = csio->ccb_h.timeout / (60 * 1000);
-       ss->timeout.scale = MLY_TIMEOUT_MINUTES;
-    } else {
-       ss->timeout.value = csio->ccb_h.timeout / (60 * 60 * 1000);     /* overflow? */
-       ss->timeout.scale = MLY_TIMEOUT_HOURS;
-    }
-    ss->maximum_sense_size = csio->sense_len;
-    ss->cdb_length = csio->cdb_len;
-    if (csio->ccb_h.flags & CAM_CDB_POINTER) {
-       bcopy(csio->cdb_io.cdb_ptr, ss->cdb, csio->cdb_len);
-    } else {
-       bcopy(csio->cdb_io.cdb_bytes, ss->cdb, csio->cdb_len);
-    }
-
-out:
-    if (error != 0) {
-       if (mc != NULL) {
-           mly_release_command(mc);
-           mc = NULL;
-       }
-       if (csio != NULL)
-           mly_requeue_ccb(sc, (union ccb *)csio);
-    }
-    *mcp = mc;
-    return(error);
-}
-
-/********************************************************************************
- * Handle completion of a command - pass results back through the CCB
- */
-static void
-mly_cam_complete(struct mly_command *mc)
-{
-    struct mly_softc           *sc = mc->mc_sc;
-    struct ccb_scsiio          *csio = (struct ccb_scsiio *)mc->mc_private;
-    struct scsi_inquiry_data   *inq = (struct scsi_inquiry_data *)csio->data_ptr;
-    struct mly_btl             *btl;
-    u_int8_t                   cmd;
-    int                                bus, target;
-
-    debug_called(2);
-
-    csio->scsi_status = mc->mc_status;
-    switch(mc->mc_status) {
-    case SCSI_STATUS_OK:
-       /*
-        * In order to report logical device type and status, we overwrite
-        * the result of the INQUIRY command to logical devices.
-        */
-       bus = csio->ccb_h.sim_priv.entries[0].field;
-       if (bus >= sc->mly_controllerinfo->physical_channels_present) {
-           if (csio->ccb_h.flags & CAM_CDB_POINTER) {
-               cmd = *csio->cdb_io.cdb_ptr;
-           } else {
-               cmd = csio->cdb_io.cdb_bytes[0];
-           }
-           if (cmd == INQUIRY) {
-               target = csio->ccb_h.target_id;
-               btl = &sc->mly_btl[bus][target];
-               padstr(inq->vendor, mly_describe_code(mly_table_device_type, btl->mb_type), 8);
-               padstr(inq->product, mly_describe_code(mly_table_device_state, btl->mb_state), 16);
-               padstr(inq->revision, "", 4);
-           }
-       }
-
-       debug(2, "SCSI_STATUS_OK");
-       csio->ccb_h.status = CAM_REQ_CMP;
-       break;
-
-    case SCSI_STATUS_CHECK_COND:
-       debug(2, "SCSI_STATUS_CHECK_COND  sense %d  resid %d", mc->mc_sense, mc->mc_resid);
-       csio->ccb_h.status = CAM_SCSI_STATUS_ERROR;
-       bzero(&csio->sense_data, SSD_FULL_SIZE);
-       bcopy(mc->mc_packet, &csio->sense_data, mc->mc_sense);
-       csio->sense_len = mc->mc_sense;
-       csio->ccb_h.status |= CAM_AUTOSNS_VALID;
-       csio->resid = mc->mc_resid;     /* XXX this is a signed value... */
-       break;
-
-    case SCSI_STATUS_BUSY:
-       debug(2, "SCSI_STATUS_BUSY");
-       csio->ccb_h.status = CAM_SCSI_BUSY;
-       break;
-
-    default:
-       debug(2, "unknown status 0x%x", csio->scsi_status);
-       csio->ccb_h.status = CAM_REQ_CMP_ERR;
-       break;
-    }
-    xpt_done((union ccb *)csio);
-    mly_release_command(mc);
-}
-
-/********************************************************************************
- * Find a peripheral attahed at (bus),(target)
- */
-static struct cam_periph *
-mly_find_periph(struct mly_softc *sc, int bus, int target)
-{
-    struct cam_periph  *periph;
-    struct cam_path    *path;
-    int                        status;
-
-    status = xpt_create_path(&path, NULL, cam_sim_path(sc->mly_cam_sim[bus]), target, 0);
-    if (status == CAM_REQ_CMP) {
-       periph = cam_periph_find(path, NULL);
-       xpt_free_path(path);
-    } else {
-       periph = NULL;
-    }
-    return(periph);
-}
-
-/********************************************************************************
- * Name the device at (bus)(target)
- */
-int
-mly_name_device(struct mly_softc *sc, int bus, int target)
-{
-    struct cam_periph  *periph;
-
-    if ((periph = mly_find_periph(sc, bus, target)) != NULL) {
-       ksprintf(sc->mly_btl[bus][target].mb_name, "%s%d", periph->periph_name, periph->unit_number);
-       return(0);
-    }
-    sc->mly_btl[bus][target].mb_name[0] = 0;
-    return(ENOENT);
-}
diff --git a/sys/dev/raid/mly/mly_pci.c b/sys/dev/raid/mly/mly_pci.c
deleted file mode 100644 (file)
index 1c15391..0000000
+++ /dev/null
@@ -1,575 +0,0 @@
-/*-
- * Copyright (c) 2000, 2001 Michael Smith
- * Copyright (c) 2000 BSDi
- * All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY THE AUTHOR AND CONTRIBUTORS ``AS IS'' AND
- * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
- * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
- * ARE DISCLAIMED.  IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE LIABLE
- * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
- * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
- * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
- * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
- * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
- * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
- * SUCH DAMAGE.
- *
- *     $FreeBSD: src/sys/dev/mly/mly_pci.c,v 1.1.2.2 2001/03/05 20:17:24 msmith Exp $
- *     $DragonFly: src/sys/dev/raid/mly/Attic/mly_pci.c,v 1.8 2006/10/25 20:56:01 dillon Exp $
- */
-
-#include <sys/param.h>
-#include <sys/systm.h>
-#include <sys/malloc.h>
-#include <sys/kernel.h>
-#include <sys/bus.h>
-#include <sys/conf.h>
-#include <sys/devicestat.h>
-#include <sys/disk.h>
-#include <sys/rman.h>
-#include <sys/thread2.h>
-
-#include <bus/pci/pcireg.h>
-#include <bus/pci/pcivar.h>
-
-#include "mlyreg.h"
-#include "mlyio.h"
-#include "mlyvar.h"
-
-static int     mly_pci_probe(device_t dev);
-static int     mly_pci_attach(device_t dev);
-static int     mly_pci_detach(device_t dev);
-static int     mly_pci_shutdown(device_t dev);
-static int     mly_pci_suspend(device_t dev); 
-static int     mly_pci_resume(device_t dev);
-static void    mly_pci_intr(void *arg);
-
-static int     mly_sg_map(struct mly_softc *sc);
-static void    mly_sg_map_helper(void *arg, bus_dma_segment_t *segs, int nseg, int error);
-static int     mly_mmbox_map(struct mly_softc *sc);
-static void    mly_mmbox_map_helper(void *arg, bus_dma_segment_t *segs, int nseg, int error);
-
-static device_method_t mly_methods[] = {
-    /* Device interface */
-    DEVMETHOD(device_probe,    mly_pci_probe),
-    DEVMETHOD(device_attach,   mly_pci_attach),
-    DEVMETHOD(device_detach,   mly_pci_detach),
-    DEVMETHOD(device_shutdown, mly_pci_shutdown),
-    DEVMETHOD(device_suspend,  mly_pci_suspend),
-    DEVMETHOD(device_resume,   mly_pci_resume),
-    { 0, 0 }
-};
-
-static driver_t mly_pci_driver = {
-       "mly",
-       mly_methods,
-       sizeof(struct mly_softc)
-};
-
-static devclass_t      mly_devclass;
-DRIVER_MODULE(mly, pci, mly_pci_driver, mly_devclass, 0, 0);
-
-struct mly_ident
-{
-    u_int16_t          vendor;
-    u_int16_t          device;
-    u_int16_t          subvendor;
-    u_int16_t          subdevice;
-    int                        hwif;
-    char               *desc;
-} mly_identifiers[] = {
-    {0x1069, 0xba56, 0x1069, 0x0040, MLY_HWIF_STRONGARM, "Mylex eXtremeRAID 2000"},
-    {0x1069, 0xba56, 0x1069, 0x0030, MLY_HWIF_STRONGARM, "Mylex eXtremeRAID 3000"},
-    {0x1069, 0x0050, 0x1069, 0x0050, MLY_HWIF_I960RX,    "Mylex AcceleRAID 352"},
-    {0x1069, 0x0050, 0x1069, 0x0052, MLY_HWIF_I960RX,    "Mylex AcceleRAID 170"},
-    {0x1069, 0x0050, 0x1069, 0x0054, MLY_HWIF_I960RX,    "Mylex AcceleRAID 160"},
-    {0, 0, 0, 0, 0, 0}
-};
-
-/********************************************************************************
- ********************************************************************************
-                                                                    Bus Interface
- ********************************************************************************
- ********************************************************************************/
-
-static int
-mly_pci_probe(device_t dev)
-{
-    struct mly_ident   *m;
-
-    debug_called(1);
-
-    for (m = mly_identifiers; m->vendor != 0; m++) {
-       if ((m->vendor == pci_get_vendor(dev)) &&
-           (m->device == pci_get_device(dev)) &&
-           ((m->subvendor == 0) || ((m->subvendor == pci_get_subvendor(dev)) &&
-                                    (m->subdevice == pci_get_subdevice(dev))))) {
-           
-           device_set_desc(dev, m->desc);
-           return(-10);        /* allow room to be overridden */
-       }
-    }
-    return(ENXIO);
-}
-
-static int
-mly_pci_attach(device_t dev)
-{
-    struct mly_softc   *sc;
-    int                        i, error;
-    u_int32_t          command;
-
-    debug_called(1);
-
-    /*
-     * Initialise softc.
-     */
-    sc = device_get_softc(dev);
-    bzero(sc, sizeof(*sc));
-    sc->mly_dev = dev;
-
-#ifdef MLY_DEBUG
-    if (device_get_unit(sc->mly_dev) == 0)
-       mly_softc0 = sc;
-#endif    
-
-    /* assume failure is 'not configured' */
-    error = ENXIO;
-
-    /* 
-     * Verify that the adapter is correctly set up in PCI space.
-     */
-    command = pci_read_config(sc->mly_dev, PCIR_COMMAND, 2);
-    command |= PCIM_CMD_BUSMASTEREN;
-    pci_write_config(dev, PCIR_COMMAND, command, 2);
-    command = pci_read_config(sc->mly_dev, PCIR_COMMAND, 2);
-    if (!(command & PCIM_CMD_BUSMASTEREN)) {
-       mly_printf(sc, "can't enable busmaster feature\n");
-       goto fail;
-    }
-    if ((command & PCIM_CMD_MEMEN) == 0) {
-       mly_printf(sc, "memory window not available\n");
-       goto fail;
-    }
-
-    /*
-     * Allocate the PCI register window.
-     */
-    sc->mly_regs_rid = PCIR_MAPS;      /* first base address register */
-    if ((sc->mly_regs_resource = bus_alloc_resource(sc->mly_dev, SYS_RES_MEMORY, &sc->mly_regs_rid, 
-                                                   0, ~0, 1, RF_ACTIVE)) == NULL) {
-       mly_printf(sc, "can't allocate register window\n");
-       goto fail;
-    }
-    sc->mly_btag = rman_get_bustag(sc->mly_regs_resource);
-    sc->mly_bhandle = rman_get_bushandle(sc->mly_regs_resource);
-
-    /* 
-     * Allocate and connect our interrupt.
-     */
-    sc->mly_irq_rid = 0;
-    if ((sc->mly_irq = bus_alloc_resource(sc->mly_dev, SYS_RES_IRQ, &sc->mly_irq_rid, 
-                                         0, ~0, 1, RF_SHAREABLE | RF_ACTIVE)) == NULL) {
-       mly_printf(sc, "can't allocate interrupt\n");
-       goto fail;
-    }
-    error = bus_setup_intr(sc->mly_dev, sc->mly_irq, 0, 
-                          mly_pci_intr, sc, &sc->mly_intr, NULL);
-    if (error) {
-       mly_printf(sc, "can't set up interrupt\n");
-       goto fail;
-    }
-
-    /* assume failure is 'out of memory' */
-    error = ENOMEM;
-
-    /*
-     * Allocate the parent bus DMA tag appropriate for our PCI interface.
-     * 
-     * Note that all of these controllers are 64-bit capable.
-     */
-    if (bus_dma_tag_create(NULL,                       /* parent */
-                          1, 0,                        /* alignment, boundary */
-                          BUS_SPACE_MAXADDR_32BIT,     /* lowaddr */
-                          BUS_SPACE_MAXADDR,           /* highaddr */
-                          NULL, NULL,                  /* filter, filterarg */
-                          MAXBSIZE, MLY_MAXSGENTRIES,  /* maxsize, nsegments */
-                          BUS_SPACE_MAXSIZE_32BIT,     /* maxsegsize */
-                          BUS_DMA_ALLOCNOW,            /* flags */
-                          &sc->mly_parent_dmat)) {
-       mly_printf(sc, "can't allocate parent DMA tag\n");
-       goto fail;
-    }
-
-    /*
-     * Create DMA tag for mapping buffers into controller-addressable space.
-     */
-    if (bus_dma_tag_create(sc->mly_parent_dmat,        /* parent */
-                          1, 0,                        /* alignment, boundary */
-                          BUS_SPACE_MAXADDR,           /* lowaddr */
-                          BUS_SPACE_MAXADDR,           /* highaddr */
-                          NULL, NULL,                  /* filter, filterarg */
-                          MAXBSIZE, MLY_MAXSGENTRIES,  /* maxsize, nsegments */
-                          BUS_SPACE_MAXSIZE_32BIT,     /* maxsegsize */
-                          0,                           /* flags */
-                          &sc->mly_buffer_dmat)) {
-       mly_printf(sc, "can't allocate buffer DMA tag\n");
-       goto fail;
-    }
-
-    /*
-     * Initialise the DMA tag for command packets.
-     */
-    if (bus_dma_tag_create(sc->mly_parent_dmat,                /* parent */
-                          1, 0,                        /* alignment, boundary */
-                          BUS_SPACE_MAXADDR,           /* lowaddr */
-                          BUS_SPACE_MAXADDR,           /* highaddr */
-                          NULL, NULL,                  /* filter, filterarg */
-                          sizeof(union mly_command_packet) * MLY_MAXCOMMANDS, 1,       /* maxsize, nsegments */
-                          BUS_SPACE_MAXSIZE_32BIT,     /* maxsegsize */
-                          0,                           /* flags */
-                          &sc->mly_packet_dmat)) {
-       mly_printf(sc, "can't allocate command packet DMA tag\n");
-       goto fail;
-    }
-
-    /* 
-     * Detect the hardware interface version 
-     */
-    for (i = 0; mly_identifiers[i].vendor != 0; i++) {
-       if ((mly_identifiers[i].vendor == pci_get_vendor(dev)) &&
-           (mly_identifiers[i].device == pci_get_device(dev))) {
-           sc->mly_hwif = mly_identifiers[i].hwif;
-           switch(sc->mly_hwif) {
-           case MLY_HWIF_I960RX:
-               debug(2, "set hardware up for i960RX");
-               sc->mly_doorbell_true = 0x00;
-               sc->mly_command_mailbox =  MLY_I960RX_COMMAND_MAILBOX;
-               sc->mly_status_mailbox =   MLY_I960RX_STATUS_MAILBOX;
-               sc->mly_idbr =             MLY_I960RX_IDBR;
-               sc->mly_odbr =             MLY_I960RX_ODBR;
-               sc->mly_error_status =     MLY_I960RX_ERROR_STATUS;
-               sc->mly_interrupt_status = MLY_I960RX_INTERRUPT_STATUS;
-               sc->mly_interrupt_mask =   MLY_I960RX_INTERRUPT_MASK;
-               break;
-           case MLY_HWIF_STRONGARM:
-               debug(2, "set hardware up for StrongARM");
-               sc->mly_doorbell_true = 0xff;           /* doorbell 'true' is 0 */
-               sc->mly_command_mailbox =  MLY_STRONGARM_COMMAND_MAILBOX;
-               sc->mly_status_mailbox =   MLY_STRONGARM_STATUS_MAILBOX;
-               sc->mly_idbr =             MLY_STRONGARM_IDBR;
-               sc->mly_odbr =             MLY_STRONGARM_ODBR;
-               sc->mly_error_status =     MLY_STRONGARM_ERROR_STATUS;
-               sc->mly_interrupt_status = MLY_STRONGARM_INTERRUPT_STATUS;
-               sc->mly_interrupt_mask =   MLY_STRONGARM_INTERRUPT_MASK;
-               break;
-           }
-           break;
-       }
-    }
-
-    /*
-     * Create the scatter/gather mappings.
-     */
-    if ((error = mly_sg_map(sc)))
-       goto fail;
-
-    /*
-     * Allocate and map the memory mailbox
-     */
-    if ((error = mly_mmbox_map(sc)))
-       goto fail;
-
-    /*
-     * Do bus-independent initialisation.
-     */
-    if ((error = mly_attach(sc)))
-       goto fail;
-    
-    return(0);
-           
-fail:
-    mly_free(sc);
-    return(error);
-}
-
-/********************************************************************************
- * Disconnect from the controller completely, in preparation for unload.
- */
-static int
-mly_pci_detach(device_t dev)
-{
-    struct mly_softc   *sc = device_get_softc(dev);
-    int                        error;
-
-    debug_called(1);
-
-    if (sc->mly_state & MLY_STATE_OPEN)
-       return(EBUSY);
-
-    if ((error = mly_pci_shutdown(dev)))
-       return(error);
-
-    mly_free(sc);
-
-    return(0);
-}
-
-/********************************************************************************
- * Bring the controller down to a dormant state and detach all child devices.
- *
- * This function is called before detach or system shutdown.
- *
- * Note that we can assume that the camq on the controller is empty, as we won't
- * allow shutdown if any device is open.
- */
-static int
-mly_pci_shutdown(device_t dev)
-{
-    struct mly_softc   *sc = device_get_softc(dev);
-
-    debug_called(1);
-
-    mly_detach(sc);
-    return(0);
-}
-
-/********************************************************************************
- * Bring the controller to a quiescent state, ready for system suspend.
- *
- * We can't assume that the controller is not active at this point, so we need
- * to mask interrupts.
- */
-static int
-mly_pci_suspend(device_t dev)
-{
-    struct mly_softc   *sc = device_get_softc(dev);
-
-    debug_called(1);
-    crit_enter();
-    mly_detach(sc);
-    crit_exit();
-    return(0);
-}
-
-/********************************************************************************
- * Bring the controller back to a state ready for operation.
- */
-static int
-mly_pci_resume(device_t dev)
-{
-    struct mly_softc   *sc = device_get_softc(dev);
-
-    debug_called(1);
-    sc->mly_state &= ~MLY_STATE_SUSPEND;
-    MLY_UNMASK_INTERRUPTS(sc);
-    return(0);
-}
-
-/*******************************************************************************
- * Take an interrupt, or be poked by other code to look for interrupt-worthy
- * status.
- */
-static void
-mly_pci_intr(void *arg)
-{
-    struct mly_softc   *sc = (struct mly_softc *)arg;
-
-    debug_called(3);
-
-    /* collect finished commands, queue anything waiting */
-    mly_done(sc);
-};
-
-/********************************************************************************
- ********************************************************************************
-                                                Bus-dependant Resource Management
- ********************************************************************************
- ********************************************************************************/
-
-/********************************************************************************
- * Allocate memory for the scatter/gather tables
- */
-static int
-mly_sg_map(struct mly_softc *sc)
-{
-    size_t     segsize;
-
-    debug_called(1);
-
-    /*
-     * Create a single tag describing a region large enough to hold all of
-     * the s/g lists we will need.
-     */
-    segsize = sizeof(struct mly_sg_entry) * MLY_MAXCOMMANDS * MLY_MAXSGENTRIES;
-    if (bus_dma_tag_create(sc->mly_parent_dmat,                /* parent */
-                          1, 0,                        /* alignment, boundary */
-                          BUS_SPACE_MAXADDR,           /* lowaddr */
-                          BUS_SPACE_MAXADDR,           /* highaddr */
-                          NULL, NULL,                  /* filter, filterarg */
-                          segsize, 1,                  /* maxsize, nsegments */
-                          BUS_SPACE_MAXSIZE_32BIT,     /* maxsegsize */
-                          0,                           /* flags */
-                          &sc->mly_sg_dmat)) {
-       mly_printf(sc, "can't allocate scatter/gather DMA tag\n");
-       return(ENOMEM);
-    }
-
-    /*
-     * Allocate enough s/g maps for all commands and permanently map them into
-     * controller-visible space.
-     * 
-     * XXX this assumes we can get enough space for all the s/g maps in one 
-     * contiguous slab.
-     */
-    if (bus_dmamem_alloc(sc->mly_sg_dmat, (void **)&sc->mly_sg_table, BUS_DMA_NOWAIT, &sc->mly_sg_dmamap)) {
-       mly_printf(sc, "can't allocate s/g table\n");
-       return(ENOMEM);
-    }
-    bus_dmamap_load(sc->mly_sg_dmat, sc->mly_sg_dmamap, sc->mly_sg_table, segsize, mly_sg_map_helper, sc, 0);
-    return(0);
-}
-
-/********************************************************************************
- * Save the physical address of the base of the s/g table.
- */
-static void
-mly_sg_map_helper(void *arg, bus_dma_segment_t *segs, int nseg, int error)
-{
-    struct mly_softc   *sc = (struct mly_softc *)arg;
-
-    debug_called(2);
-
-    /* save base of s/g table's address in bus space */
-    sc->mly_sg_busaddr = segs->ds_addr;
-}
-
-/********************************************************************************
- * Allocate memory for the memory-mailbox interface
- */
-static int
-mly_mmbox_map(struct mly_softc *sc)
-{
-
-    /*
-     * Create a DMA tag for a single contiguous region large enough for the
-     * memory mailbox structure.
-     */
-    if (bus_dma_tag_create(sc->mly_parent_dmat,                /* parent */
-                          1, 0,                        /* alignment, boundary */
-                          BUS_SPACE_MAXADDR,           /* lowaddr */
-                          BUS_SPACE_MAXADDR,           /* highaddr */
-                          NULL, NULL,                  /* filter, filterarg */
-                          sizeof(struct mly_mmbox), 1, /* maxsize, nsegments */
-                          BUS_SPACE_MAXSIZE_32BIT,     /* maxsegsize */
-                          0,                           /* flags */
-                          &sc->mly_mmbox_dmat)) {
-       mly_printf(sc, "can't allocate memory mailbox DMA tag\n");
-       return(ENOMEM);
-    }
-
-    /*
-     * Allocate the buffer
-     */
-    if (bus_dmamem_alloc(sc->mly_mmbox_dmat, (void **)&sc->mly_mmbox, BUS_DMA_NOWAIT, &sc->mly_mmbox_dmamap)) {
-       mly_printf(sc, "can't allocate memory mailbox\n");
-       return(ENOMEM);
-    }
-    bus_dmamap_load(sc->mly_mmbox_dmat, sc->mly_mmbox_dmamap, sc->mly_mmbox, sizeof(struct mly_mmbox), 
-                   mly_mmbox_map_helper, sc, 0);
-    bzero(sc->mly_mmbox, sizeof(*sc->mly_mmbox));
-    return(0);
-
-}
-
-/********************************************************************************
- * Save the physical address of the memory mailbox 
- */
-static void
-mly_mmbox_map_helper(void *arg, bus_dma_segment_t *segs, int nseg, int error)
-{
-    struct mly_softc   *sc = (struct mly_softc *)arg;
-
-    debug_called(2);
-
-    sc->mly_mmbox_busaddr = segs->ds_addr;
-}
-
-/********************************************************************************
- * Free all of the resources associated with (sc)
- *
- * Should not be called if the controller is active.
- */
-void
-mly_free(struct mly_softc *sc)
-{
-    struct mly_command *mc;
-    
-    debug_called(1);
-
-    /* detach from CAM */
-    mly_cam_detach(sc);
-
-    /* throw away command buffer DMA maps */
-    while (mly_alloc_command(sc, &mc) == 0)
-       bus_dmamap_destroy(sc->mly_buffer_dmat, mc->mc_datamap);
-
-    /* release the packet storage */
-    if (sc->mly_packet != NULL) {
-       bus_dmamap_unload(sc->mly_packet_dmat, sc->mly_packetmap);
-       bus_dmamem_free(sc->mly_packet_dmat, sc->mly_packet, sc->mly_packetmap);
-    }
-
-    /* throw away the controllerinfo structure */
-    if (sc->mly_controllerinfo != NULL)
-       kfree(sc->mly_controllerinfo, M_DEVBUF);
-
-    /* throw away the controllerparam structure */
-    if (sc->mly_controllerparam != NULL)
-       kfree(sc->mly_controllerparam, M_DEVBUF);
-
-    /* destroy data-transfer DMA tag */
-    if (sc->mly_buffer_dmat)
-       bus_dma_tag_destroy(sc->mly_buffer_dmat);
-
-    /* free and destroy DMA memory and tag for s/g lists */
-    if (sc->mly_sg_table) {
-       bus_dmamap_unload(sc->mly_sg_dmat, sc->mly_sg_dmamap);
-       bus_dmamem_free(sc->mly_sg_dmat, sc->mly_sg_table, sc->mly_sg_dmamap);
-    }
-    if (sc->mly_sg_dmat)
-       bus_dma_tag_destroy(sc->mly_sg_dmat);
-
-    /* free and destroy DMA memory and tag for memory mailbox */
-    if (sc->mly_mmbox) {
-       bus_dmamap_unload(sc->mly_mmbox_dmat, sc->mly_mmbox_dmamap);
-       bus_dmamem_free(sc->mly_mmbox_dmat, sc->mly_mmbox, sc->mly_mmbox_dmamap);
-    }
-    if (sc->mly_mmbox_dmat)
-       bus_dma_tag_destroy(sc->mly_mmbox_dmat);
-
-    /* disconnect the interrupt handler */
-    if (sc->mly_intr)
-       bus_teardown_intr(sc->mly_dev, sc->mly_irq, sc->mly_intr);
-    if (sc->mly_irq != NULL)
-       bus_release_resource(sc->mly_dev, SYS_RES_IRQ, sc->mly_irq_rid, sc->mly_irq);
-
-    /* destroy the parent DMA tag */
-    if (sc->mly_parent_dmat)
-       bus_dma_tag_destroy(sc->mly_parent_dmat);
-
-    /* release the register window mapping */
-    if (sc->mly_regs_resource != NULL)
-       bus_release_resource(sc->mly_dev, SYS_RES_MEMORY, sc->mly_regs_rid, sc->mly_regs_resource);
-}
-
index 22a94ea..ba0578f 100644 (file)
@@ -25,7 +25,7 @@
  * SUCH DAMAGE.
  *
  *     $FreeBSD: src/sys/dev/mly/mly_tables.h,v 1.1.2.1 2000/10/27 10:06:56 msmith Exp $
- *     $DragonFly: src/sys/dev/raid/mly/mly_tables.h,v 1.2 2003/06/17 04:28:28 dillon Exp $
+ *     $DragonFly: src/sys/dev/raid/mly/mly_tables.h,v 1.3 2008/01/21 02:27:37 pavalos Exp $
  */
 
 /*
@@ -36,26 +36,12 @@ struct mly_code_lookup {
     u_int32_t  code;
 };
 
-extern char    *mly_describe_code(struct mly_code_lookup *table, u_int32_t code);
-
-#ifndef MLY_DEFINE_TABLES
-extern struct mly_code_lookup mly_table_bustype[];
-extern struct mly_code_lookup mly_table_controllertype[];
-extern struct mly_code_lookup mly_table_oemname[];
-extern struct mly_code_lookup mly_table_memorytype[];
-extern struct mly_code_lookup mly_table_cputype[];
-extern struct mly_code_lookup mly_table_event[];
-extern struct mly_code_lookup mly_table_device_state[];
-extern struct mly_code_lookup mly_table_device_type[];
-extern struct mly_code_lookup mly_table_stripe_size[];
-extern struct mly_code_lookup mly_table_cacheline_size[];
-
-#else /* MLY_DEFINE_TABLES */
+static char    *mly_describe_code(struct mly_code_lookup *table, u_int32_t code);
 
 /********************************************************************************
  * Look up a text description of a numeric code and return a pointer to same.
  */
-char *
+static char *
 mly_describe_code(struct mly_code_lookup *table, u_int32_t code)
 {
     int                i;
@@ -66,15 +52,17 @@ mly_describe_code(struct mly_code_lookup *table, u_int32_t code)
     return(table[i+1].string);
 }
 
-struct mly_code_lookup mly_table_bustype[] = {
+#if 0
+static struct mly_code_lookup mly_table_bustype[] = {
     {"SCSI",           0x00},
     {"FC-AL",          0x01},
     {"PCI",            0x03},
     {NULL,             0},
     {"unknown bus",    0}
 };
+#endif
 
-struct mly_code_lookup mly_table_controllertype[] = {
+static struct mly_code_lookup mly_table_controllertype[] = {
 #if 0  /* not supported by this driver */
     {"DAC960E",                0x01},  /* EISA */
     {"DAC960M",                0x08},  /* MCA */
@@ -100,7 +88,7 @@ struct mly_code_lookup mly_table_controllertype[] = {
     {"unknown adapter",        0}
 };    
 
-struct mly_code_lookup mly_table_oemname[] = {
+static struct mly_code_lookup mly_table_oemname[] = {
     {"Mylex",          MLY_OEM_MYLEX},
     {"IBM",            MLY_OEM_IBM},
     {"Hewlett-Packard",        MLY_OEM_HP},
@@ -111,7 +99,7 @@ struct mly_code_lookup mly_table_oemname[] = {
     {"unknown OEM",    0}
 };
 
-struct mly_code_lookup mly_table_memorytype[] = {
+static struct mly_code_lookup mly_table_memorytype[] = {
     {"DRAM",           0x01},
     {"EDRAM",          0x02},
     {"EDO RAM",                0x03},
@@ -120,7 +108,7 @@ struct mly_code_lookup mly_table_memorytype[] = {
     {"unknown memory", 0}
 };
 
-struct mly_code_lookup mly_table_cputype[] = {
+static struct mly_code_lookup mly_table_cputype[] = {
     {"i960CA",         0x01},
     {"i960RD",         0x02},
     {"i960RN",         0x03},
@@ -154,7 +142,7 @@ struct mly_code_lookup mly_table_cputype[] = {
  * r   rescan the device for possible state change
  *
  */
-struct mly_code_lookup mly_table_event[] = {
+static struct mly_code_lookup mly_table_event[] = {
     /* physical device events (0x0000 - 0x007f) */
     {"pr online",                                                      0x0001},
     {"pr standby",                                                     0x0002},
@@ -267,7 +255,7 @@ struct mly_code_lookup mly_table_event[] = {
  * Values here must be 16 characters or less, as they are packed into
  * the 'product' field in the SCSI inquiry data.
  */
-struct mly_code_lookup mly_table_device_state[] = {
+static struct mly_code_lookup mly_table_device_state[] = {
     {"offline",                MLY_DEVICE_STATE_OFFLINE},
     {"unconfigured",   MLY_DEVICE_STATE_UNCONFIGURED},
     {"online",         MLY_DEVICE_STATE_ONLINE},
@@ -283,7 +271,7 @@ struct mly_code_lookup mly_table_device_state[] = {
  * Values here must be 8 characters or less, as they are packed into
  * the 'vendor' field in the SCSI inquiry data.
  */
-struct mly_code_lookup mly_table_device_type[] = {
+static struct mly_code_lookup mly_table_device_type[] = {
     {"RAID 0",         MLY_DEVICE_TYPE_RAID0},
     {"RAID 1",         MLY_DEVICE_TYPE_RAID1},
     {"RAID 3",         MLY_DEVICE_TYPE_RAID3},         /* right asymmetric parity */
@@ -301,7 +289,8 @@ struct mly_code_lookup mly_table_device_type[] = {
     {"UNKNOWN",                0}
 };
 
-struct mly_code_lookup mly_table_stripe_size[] = {
+#if 0
+static struct mly_code_lookup mly_table_stripe_size[] = {
     {"NONE",           MLY_STRIPE_ZERO},
     {"512B",           MLY_STRIPE_512b},
     {"1k",             MLY_STRIPE_1k},
@@ -319,7 +308,7 @@ struct mly_code_lookup mly_table_stripe_size[] = {
     {"unknown",                0}
 };
 
-struct mly_code_lookup mly_table_cacheline_size[] = {
+static struct mly_code_lookup mly_table_cacheline_size[] = {
     {"NONE",           MLY_CACHELINE_ZERO},
     {"512B",           MLY_CACHELINE_512b},
     {"1k",             MLY_CACHELINE_1k},
@@ -332,5 +321,4 @@ struct mly_code_lookup mly_table_cacheline_size[] = {
     {NULL, 0},
     {"unknown",                0}
 };
-
-#endif /* MLY_DEFINE_TABLES */
+#endif
index bd1aae7..d6d9a11 100644 (file)
@@ -24,7 +24,7 @@
  * SUCH DAMAGE.
  *
  *     $FreeBSD: src/sys/dev/mly/mlyio.h,v 1.1.2.1 2001/03/05 20:17:24 msmith Exp $
- *     $DragonFly: src/sys/dev/raid/mly/mlyio.h,v 1.2 2003/06/17 04:28:28 dillon Exp $
+ *     $DragonFly: src/sys/dev/raid/mly/mlyio.h,v 1.3 2008/01/21 02:27:37 pavalos Exp $
  */
 
 /********************************************************************************
@@ -60,11 +60,9 @@ struct mly_user_health
  */
 
 #define MLYQ_FREE      0
-#define MLYQ_CCB       1
-#define MLYQ_READY     2
-#define MLYQ_BUSY      3
-#define MLYQ_COMPLETE  4
-#define MLYQ_COUNT     5
+#define MLYQ_BUSY      1
+#define MLYQ_COMPLETE  2
+#define MLYQ_COUNT     3
 
 struct mly_qstat 
 {
index 85bddaa..650e430 100644 (file)
@@ -25,7 +25,7 @@
  * SUCH DAMAGE.
  *
  *     $FreeBSD: src/sys/dev/mly/mlyreg.h,v 1.1.2.1 2000/10/27 10:06:56 msmith Exp $
- *     $DragonFly: src/sys/dev/raid/mly/mlyreg.h,v 1.2 2003/06/17 04:28:28 dillon Exp $
+ *     $DragonFly: src/sys/dev/raid/mly/mlyreg.h,v 1.3 2008/01/21 02:27:37 pavalos Exp $
  */
 
 /*
@@ -49,7 +49,7 @@
 struct mly_sg_entry {
     u_int64_t  physaddr;
     u_int64_t  length;
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * 5.2 System Device Access
@@ -147,7 +147,7 @@ struct mly_lun_map {
     u_int8_t   lun;                    /* LUN */
     u_int8_t   tid;                    /* TID */
     u_int8_t   hid[32];                /* HID (one bit for each host) */
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * 10.1 Controller Parameters
@@ -253,7 +253,7 @@ struct mly_param_controller {
 #define MLY_STARTUP_ALWAYS             0x5
 
     u_int8_t   res15[62];
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * 10.2 Physical Device Parameters
@@ -265,7 +265,7 @@ struct mly_param_physical_device {
     u_int8_t   combing:1;
     u_int8_t   res1:7;
     u_int8_t   res2[3];
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * 10.3 Logical Device Parameters
@@ -279,7 +279,7 @@ struct mly_param_logical_device {
     u_int8_t   stripe_size;            /* see 8.3 */
     u_int8_t   read_write_control;     /* see 8.5 */
     u_int8_t   res2[8];
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * 12.3 Health Status Buffer
@@ -299,7 +299,7 @@ struct mly_health_status {
     u_int32_t  profiler_page;                          /* N/A */
     u_int32_t  next_event;
     u_int8_t   res3[4 + 16 + 64];                      /* N/A */
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * 14.2 Timeout Bit Format
@@ -310,7 +310,7 @@ struct mly_timeout {
 #define MLY_TIMEOUT_SECONDS    0x0
 #define MLY_TIMEOUT_MINUTES    0x1
 #define MLY_TIMEOUT_HOURS      0x2
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * 14.3 Operation Device
@@ -336,7 +336,7 @@ struct mly_status {
     u_int8_t   status;
     u_int8_t   sense_length;
     int32_t    residue;
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * 14.5 Command Control Bit (CCB) format
@@ -354,7 +354,7 @@ struct mly_command_control {
     u_int8_t   res2:1;
     u_int8_t   no_auto_sense:1;
     u_int8_t   disable_disconnect:1;
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * 15.0 Commands
@@ -437,7 +437,7 @@ struct mly_command_control {
  */
 struct mly_short_transfer {
     struct mly_sg_entry        sg[2];
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * 17.1.5 Data Transfer Memory Address With SG List
@@ -448,7 +448,7 @@ struct mly_sg_transfer {
     u_int16_t  entries[3];
     u_int16_t  res1;
     u_int64_t  table_physaddr[3];
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * 17.1.3 Data Transfer Memory Address Format
@@ -485,7 +485,7 @@ union mly_command_transfer {
 struct mly_ioctl_param_data {
     u_int8_t                   param[10];
     union mly_command_transfer transfer;
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * 21.2 MDACIOCTL_SETMEMORYMAILBOX
@@ -497,7 +497,7 @@ struct mly_ioctl_param_setmemorymailbox {
     u_int64_t  command_mailbox_physaddr;
     u_int64_t  status_mailbox_physaddr;
     u_int64_t  res2[2];
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * 21.8.2 MDACIOCTL_GETCONTROLLERINFO: Data Format
@@ -610,7 +610,7 @@ struct mly_ioctl_getcontrollerinfo {
        u_int8_t        number;
        u_int8_t        res1[12];                               /* N/A */
        char            name[16];                               /* N/A */
-    } cpu[2] __attribute__ ((packed));
+    } cpu[2] __packed;
     /* debugging/profiling/command time tracing information */
     u_int16_t  profiling_page;                                 /* N/A */
     u_int16_t  profiling_programs;                             /* N/A */
@@ -675,7 +675,7 @@ struct mly_ioctl_getcontrollerinfo {
     u_int8_t   res26:6;                                        /* N/A */
     u_int8_t   res27[3];                                       /* N/A */
     u_int8_t   res28[32 + 512];                                /* N/A */
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * 21.9.2 MDACIOCTL_GETLOGDEVINFOVALID
@@ -725,7 +725,7 @@ struct mly_ioctl_getlogdevinfovalid {
     u_int64_t  migration_block;
     u_int64_t  patrol_block;                                   /* N/A */
     u_int8_t   res7[64];                                       /* N/A */
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * 21.10.2 MDACIOCTL_GETPHYSDEVINFOVALID: Data Format
@@ -785,7 +785,7 @@ struct mly_ioctl_getphysdevinfovalid {
     u_int64_t  migration_block;                                /* N/A */
     u_int64_t  patrol_block;                                   /* N/A */
     u_int8_t   res11[256];
-} __attribute__ ((packed));
+} __packed;
 
 union mly_devinfo {
     struct mly_ioctl_getlogdevinfovalid                logdev;
@@ -819,7 +819,7 @@ struct mly_ioctl_getdevstatistics {
     u_int16_t  active_commands;                                /* N/A */
     u_int16_t  waiting_commands;                               /* N/A */
     u_int8_t   res3[8];                                        /* N/A */
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * 21.13.2 MDACIOCTL_GETCONTROLLERSTATISTICS: Data Format
@@ -861,14 +861,14 @@ struct mly_ioctl_getcontrollerstatistics {
     u_int16_t  host_system_commands_active;                    /* N/A */
     u_int16_t  host_system_commands_waiting;                   /* N/A */
     u_int8_t   res4[48 + 64];                                  /* N/A */
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * 21.2 MDACIOCTL_SETRAIDDEVSTATE
  */
 struct mly_ioctl_param_setraiddevstate {
     u_int8_t   state;
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * 21.27.2 MDACIOCTL_GETBDT_FOR_SYSDRIVE: Data Format
@@ -877,7 +877,7 @@ struct mly_ioctl_param_setraiddevstate {
 struct mly_ioctl_getbdt_for_sysdrive {
     u_int32_t  num_of_bdt_entries;
     u_int32_t  bad_data_block_address[MLY_MAX_BDT_ENTRIES];
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * 22.1 Physical Device Definition (PDD)
@@ -892,7 +892,7 @@ struct mly_pdd {
     u_int8_t   target;
     u_int8_t   lun;
     u_int32_t  start_address;
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * 22.2 RAID Device Use Definition (UDD)
@@ -902,7 +902,7 @@ struct mly_udd {
     u_int8_t   state;                          /* see 8.1 */
     u_int16_t  raid_device;
     u_int32_t  start_address;
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * RAID Device Definition (LDD)
@@ -918,7 +918,7 @@ struct mly_ldd {
     u_int8_t   read_write_control;             /* see 8.5 */
     u_int32_t  devices_used_size;              /* XXX "block or MB" Huh? */
     u_int16_t  devices_used[32];               /* XXX actual size of this field unknown! */
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * Define a datastructure giving the smallest allocation that will hold
@@ -944,7 +944,7 @@ union mly_ioctl_devconfinfo {
  */
 struct mly_ioctl_param_renameraiddev {
     u_int8_t   new_raid_device;
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * 23.6.2 MDACIOCTL_XLATEPHYSDEVTORAIDDEV
@@ -958,7 +958,7 @@ struct mly_ioctl_param_xlatephysdevtoraiddev {
     u_int8_t   channel;
     u_int8_t   target;
     u_int8_t   lun;
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * 23.7 MDACIOCTL_GETGROUPCONFINFO
@@ -967,7 +967,7 @@ struct mly_ioctl_param_getgroupconfinfo {
     u_int16_t                  group;
     u_int8_t                   res1[8];
     union mly_command_transfer transfer;
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * 23.9.2 MDACIOCTL_GETFREESPACELIST: Data Format
@@ -980,14 +980,14 @@ struct mly_ioctl_getfreespacelist_entry {
     u_int8_t   res1[6];
     u_int32_t  address;                /* XXX "blocks or MB" Huh? */
     u_int32_t  size;                   /* XXX "blocks or MB" Huh? */
-} __attribute__ ((packed));
+} __packed;
 
 struct mly_ioctl_getfrespacelist {
     u_int16_t  returned_entries;
     u_int16_t  total_entries;
     u_int8_t   res1[12];
     struct mly_ioctl_getfreespacelist_entry space[0];  /* expand to suit */
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * 27.1 MDACIOCTL_GETSUBSYSTEMDATA
@@ -1004,7 +1004,7 @@ struct mly_ioctl_param_subsystemdata {
     u_int8_t   subsystem:4;
 #define MLY_SUBSYSTEM_BBU      0x01
     u_int      parameter[3];           /* only for SETSUBSYSTEMDATA */
-} __attribute__ ((packed));
+} __packed;
 
 struct mly_ioctl_getsubsystemdata_bbustatus {
     u_int16_t  current_power;
@@ -1026,7 +1026,7 @@ struct mly_ioctl_getsubsystemdata_bbustatus {
 #define MLY_BBU_STATUS_DISCHARGING     0x20
 #define MLY_BBU_STATUS_FASTCHARGING    0x40
     u_int8_t   res2;
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * 28.9  MDACIOCTL_RESETDEVICE
@@ -1036,7 +1036,7 @@ struct mly_ioctl_getsubsystemdata_bbustatus {
  */
 struct mly_ioctl_param_deviceoperation {
     u_int8_t   operation_device;               /* see 14.3 */
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * 31.1 Event Data Format
@@ -1052,7 +1052,7 @@ struct mly_event {
     u_int8_t           res1[4];
     u_int32_t  param;
     u_int8_t   sense[40];
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * 31.2 MDACIOCTL_GETEVENT
@@ -1061,7 +1061,7 @@ struct mly_ioctl_param_getevent {
     u_int16_t                  sequence_number_low;
     u_int8_t                   res1[8];
     union mly_command_transfer transfer;
-} __attribute__ ((packed));
+} __packed;
 
 union mly_ioctl_param {
     struct mly_ioctl_param_data                                data;
@@ -1083,13 +1083,13 @@ struct mly_command_address_physical {
     u_int8_t                   target;
     u_int8_t                   channel:3;
     u_int8_t                   controller:5;
-} __attribute__ ((packed));
+} __packed;
 
 struct mly_command_address_logical {
     u_int16_t                  logdev;
     u_int8_t                   res1:3;
     u_int8_t                   controller:5;
-} __attribute__ ((packed));
+} __packed;
 
 union mly_command_address {
     struct mly_command_address_physical        phys;
@@ -1107,7 +1107,7 @@ struct mly_command_generic {
     u_int8_t                   maximum_sense_size;
     u_int8_t                   res1[11];
     union mly_command_transfer transfer;
-} __attribute__ ((packed));
+} __packed;
     
 
 /*
@@ -1126,7 +1126,7 @@ struct mly_command_scsi_small {
     u_int8_t                   cdb_length;
     u_int8_t                   cdb[MLY_CMD_SCSI_SMALL_CDB];
     union mly_command_transfer transfer;
-} __attribute__ ((packed));
+} __packed;
     
 /*
  * 19.2 MDACMD_SCSILC & MDACMD_SCSILCPT
@@ -1144,7 +1144,7 @@ struct mly_command_scsi_large {
     u_int16_t                  res1;
     u_int64_t                  cdb_physaddr;
     union mly_command_transfer transfer;
-} __attribute__ ((packed));
+} __packed;
     
 /*
  * 20.1 IOCTL Command Format: Internal Bus
@@ -1160,7 +1160,7 @@ struct mly_command_ioctl {
     u_int8_t                   maximum_sense_size;
     u_int8_t                   sub_ioctl;
     union mly_ioctl_param      param;
-} __attribute__ ((packed));
+} __packed;
 
 /*
  * PG6: 8.2.2
@@ -1168,7 +1168,7 @@ struct mly_command_ioctl {
 struct mly_command_mmbox {
     u_int32_t                  flag;
     u_int8_t                   data[60];
-} __attribute__ ((packed));
+} __packed;
 
 union mly_command_packet {
     struct mly_command_generic         generic;
@@ -1255,7 +1255,7 @@ union mly_status_packet {
      struct {
         u_int32_t              flag;
         u_int8_t               data[4];
-     } __attribute__ ((packed)) mmbox;
+     } __packed mmbox;
 };
 union mly_health_region {
     struct mly_health_status   status;
@@ -1268,4 +1268,4 @@ struct mly_mmbox {
     union mly_command_packet   mmm_command[MLY_MMBOX_COMMANDS];
     union mly_status_packet    mmm_status[MLY_MMBOX_STATUS];
     union mly_health_region    mmm_health;
-} __attribute__ ((packed));
+} __packed;
index b5d104b..e97efc1 100644 (file)
@@ -25,7 +25,7 @@
  * SUCH DAMAGE.
  *
  *     $FreeBSD: src/sys/dev/mly/mlyvar.h,v 1.1.2.2 2001/03/05 20:17:24 msmith Exp $
- *     $DragonFly: src/sys/dev/raid/mly/mlyvar.h,v 1.8 2006/12/22 23:26:24 swildner Exp $
+ *     $DragonFly: src/sys/dev/raid/mly/mlyvar.h,v 1.9 2008/01/21 02:27:37 pavalos Exp $
  */
 
 #include <sys/thread2.h>
  * table this size (256k) would be too expensive, so we cap ourselves at a
  * reasonable limit.
  */
-#define MLY_MAXCOMMANDS                256     /* max commands per controller */
+#define MLY_MAX_COMMANDS       256     /* max commands per controller */
 
 /*
  * The firmware interface allows for a 16-bit s/g list length.  We limit 
  * ourselves to a reasonable maximum and ensure alignment.
  */
-#define MLY_MAXSGENTRIES       64      /* max S/G entries, limit 65535 */              
+#define MLY_MAX_SGENTRIES      64      /* max S/G entries, limit 65535 */              
+
+/*
+ * The interval at which we poke the controller for status updates (in seconds).
+ */
+#define MLY_PERIODIC_INTERVAL  1
 
 /********************************************************************************
  ********************************************************************************
  ********************************************************************************
  ********************************************************************************/
 
-#if defined(__FreeBSD__) && __FreeBSD_version >= 500005
 # include <sys/taskqueue.h>
-#endif
 
-#if defined(__DragonFly__) || __FreeBSD_version <= 500014
-# include <machine/clock.h>
-# undef offsetof
-# define offsetof(type, field) ((size_t)(&((type *)0)->field))
+#ifndef INTR_ENTROPY
+# define INTR_ENTROPY 0
 #endif
 
 /********************************************************************************
@@ -142,6 +143,7 @@ struct mly_command {
     void       (* mc_complete)(struct mly_command *mc);        /* completion handler */
     void       *mc_private;                                    /* caller-private data */
 
+    int                                mc_timestamp;
 };
 
 /*
@@ -150,7 +152,7 @@ struct mly_command {
  * We can't use slot 0 due to the memory mailbox implementation.
  */
 #define MLY_SLOT_START         1
-#define MLY_SLOT_MAX           (MLY_SLOT_START + MLY_MAXCOMMANDS)
+#define MLY_SLOT_MAX           (MLY_SLOT_START + MLY_MAX_COMMANDS)
 
 /*
  * Per-controller structure.
@@ -196,22 +198,21 @@ struct mly_softc {
 
     /* controller features, limits and status */
     int                        mly_state;
-#define MLY_STATE_SUSPEND      (1<<0)
 #define        MLY_STATE_OPEN          (1<<1)
 #define MLY_STATE_INTERRUPTS_ON        (1<<2)
 #define MLY_STATE_MMBOX_ACTIVE (1<<3)
+#define MLY_STATE_CAM_FROZEN   (1<<4)
     struct mly_ioctl_getcontrollerinfo *mly_controllerinfo;
     struct mly_param_controller                *mly_controllerparam;
     struct mly_btl                     mly_btl[MLY_MAX_CHANNELS][MLY_MAX_TARGETS];
 
     /* command management */
-    struct mly_command         mly_command[MLY_MAXCOMMANDS];   /* commands */
+    struct mly_command         mly_command[MLY_MAX_COMMANDS];  /* commands */
     union mly_command_packet   *mly_packet;            /* command packets */
     bus_dma_tag_t              mly_packet_dmat;        /* packet DMA tag */
     bus_dmamap_t               mly_packetmap;          /* packet DMA map */
     u_int64_t                  mly_packetphys;         /* packet array base address */
     TAILQ_HEAD(,mly_command)   mly_free;               /* commands available for reuse */
-    TAILQ_HEAD(,mly_command)   mly_ready;              /* commands ready to be submitted */
     TAILQ_HEAD(,mly_command)   mly_busy;
     TAILQ_HEAD(,mly_command)   mly_complete;           /* commands which have been returned by the controller */
     struct mly_qstat           mly_qstat[MLYQ_COUNT];  /* queue statistics */
@@ -221,16 +222,17 @@ struct mly_softc {
     u_int32_t                  mly_event_counter;      /* next event for which we anticpiate status */
     u_int32_t                  mly_event_waiting;      /* next event the controller will post status for */
     struct callout             mly_periodic;           /* periodic event handling */
+    struct callout             mly_timeout;            /* timeout event handling */
 
     /* CAM connection */
-    TAILQ_HEAD(,ccb_hdr)       mly_cam_ccbq;                   /* outstanding I/O from CAM */
-    struct cam_sim             *mly_cam_sim[MLY_MAX_CHANNELS];
-    int                                mly_cam_lowbus;
+    struct cam_devq            *mly_cam_devq;                  /* CAM device queue */
+    struct cam_sim             *mly_cam_sim[MLY_MAX_CHANNELS]; /* CAM SIMs */
+    struct cam_path            *mly_cam_path;                  /* rescan path */
+    int                                mly_cam_channels;               /* total channel count */
 
-#if defined(__FreeBSD__) && __FreeBSD_version >= 500005
     /* command-completion task */
-    struct task                mly_task_complete;      /* deferred-completion task */
-#endif
+    struct task                        mly_task_complete;      /* deferred-completion task */
+    int                                mly_qfrzn_cnt;          /* Track simq freezes */
 };
 
 /*
@@ -275,28 +277,15 @@ struct mly_softc {
        } while(0);
 
 /*
- * Logical device number -> bus/target translation
+ * Bus/target/logical ID-related macros.
  */
-#define MLY_LOGDEV_BUS(sc, x)  (((x) / MLY_MAX_TARGETS) + (sc)->mly_controllerinfo->physical_channels_present)
-#define MLY_LOGDEV_TARGET(x)   ((x) % MLY_MAX_TARGETS)
-
-/*
- * Public functions/variables
- */
-/* mly.c */
-extern int     mly_attach(struct mly_softc *sc);
-extern void    mly_detach(struct mly_softc *sc);
-extern void    mly_free(struct mly_softc *sc);
-extern void    mly_startio(struct mly_softc *sc);
-extern void    mly_done(struct mly_softc *sc);
-extern int     mly_alloc_command(struct mly_softc *sc, struct mly_command **mcp);
-extern void    mly_release_command(struct mly_command *mc);
-
-/* mly_cam.c */
-extern int     mly_cam_attach(struct mly_softc *sc);
-extern void    mly_cam_detach(struct mly_softc *sc);
-extern int     mly_cam_command(struct mly_softc *sc, struct mly_command **mcp);
-extern int     mly_name_device(struct mly_softc *sc, int bus, int target);
+#define MLY_LOGDEV_ID(sc, bus, target) (((bus) - (sc)->mly_controllerinfo->physical_channels_present) * \
+                                        MLY_MAX_TARGETS + (target))
+#define MLY_LOGDEV_BUS(sc, logdev)     (((logdev) / MLY_MAX_TARGETS) + \
+                                        (sc)->mly_controllerinfo->physical_channels_present)
+#define MLY_LOGDEV_TARGET(sc, logdev)  ((logdev) % MLY_MAX_TARGETS)
+#define MLY_BUS_IS_VIRTUAL(sc, bus)    ((bus) >= (sc)->mly_controllerinfo->physical_channels_present)
+#define MLY_BUS_IS_VALID(sc, bus)      (((bus) < (sc)->mly_cam_channels) && ((sc)->mly_cam_sim[(bus)] != NULL))
 
 /********************************************************************************
  * Queue primitives
@@ -366,7 +355,20 @@ mly_remove_ ## name (struct mly_command *mc)                               \
 struct hack
 
 MLYQ_COMMAND_QUEUE(free, MLYQ_FREE);
-MLYQ_COMMAND_QUEUE(ready, MLYQ_READY);
 MLYQ_COMMAND_QUEUE(busy, MLYQ_BUSY);
 MLYQ_COMMAND_QUEUE(complete, MLYQ_COMPLETE);
 
+/********************************************************************************
+ * space-fill a character string
+ */
+static __inline void
+padstr(char *targ, char *src, int len)
+{
+    while (len-- > 0) {
+       if (*src != 0) {
+           *targ++ = *src++;
+       } else {
+           *targ++ = ' ';
+       }
+    }
+}