Author: markj
Date: Wed Jul  3 23:15:30 2013
New Revision: 252643
URL: http://svnweb.freebsd.org/changeset/base/252643

Log:
  MFC r242726 (ambrisko):
  Add support for SCSI pass through devices to be attached and detached.
  
  MFC r251172 (sbruno):
  xpt_create_path() requires mfi_io_lock to be held, so do it.
  
  mfi(4) doesn't panic on host startup now.

Modified:
  stable/9/sys/dev/mfi/mfi.c
  stable/9/sys/dev/mfi/mfi_cam.c
  stable/9/sys/dev/mfi/mfivar.h
Directory Properties:
  stable/9/sys/   (props changed)
  stable/9/sys/dev/   (props changed)

Modified: stable/9/sys/dev/mfi/mfi.c
==============================================================================
--- stable/9/sys/dev/mfi/mfi.c  Wed Jul  3 23:06:58 2013        (r252642)
+++ stable/9/sys/dev/mfi/mfi.c  Wed Jul  3 23:15:30 2013        (r252643)
@@ -1628,6 +1628,11 @@ mfi_decode_evt(struct mfi_softc *sc, str
                                sx_xunlock(&sc->mfi_config_lock);
                        }
                }
+               if (sc->mfi_cam_rescan_cb != NULL &&
+                   (detail->code == MR_EVT_PD_INSERTED ||
+                   detail->code == MR_EVT_PD_REMOVED)) {
+                       sc->mfi_cam_rescan_cb(sc, detail->args.pd.device_id);
+               }
                break;
        }
 }

Modified: stable/9/sys/dev/mfi/mfi_cam.c
==============================================================================
--- stable/9/sys/dev/mfi/mfi_cam.c      Wed Jul  3 23:06:58 2013        
(r252642)
+++ stable/9/sys/dev/mfi/mfi_cam.c      Wed Jul  3 23:15:30 2013        
(r252643)
@@ -50,7 +50,9 @@ __FBSDID("$FreeBSD$");
 #include <cam/cam.h>
 #include <cam/cam_ccb.h>
 #include <cam/cam_debug.h>
+#include <cam/cam_periph.h>
 #include <cam/cam_sim.h>
+#include <cam/cam_xpt_periph.h>
 #include <cam/cam_xpt_sim.h>
 #include <cam/scsi/scsi_all.h>
 #include <cam/scsi/scsi_message.h>
@@ -63,12 +65,19 @@ __FBSDID("$FreeBSD$");
 #include <dev/mfi/mfi_ioctl.h>
 #include <dev/mfi/mfivar.h>
 
+enum mfip_state {
+       MFIP_STATE_NONE,
+       MFIP_STATE_DETACH,
+       MFIP_STATE_RESCAN
+};
+
 struct mfip_softc {
        device_t        dev;
        struct mfi_softc *mfi_sc;
        struct cam_devq *devq;
        struct cam_sim  *sim;
        struct cam_path *path;
+       enum mfip_state state;
 };
 
 static int     mfip_probe(device_t);
@@ -76,6 +85,7 @@ static int    mfip_attach(device_t);
 static int     mfip_detach(device_t);
 static void    mfip_cam_action(struct cam_sim *, union ccb *);
 static void    mfip_cam_poll(struct cam_sim *);
+static void    mfip_cam_rescan(struct mfi_softc *, uint32_t tid);
 static struct mfi_command * mfip_start(void *);
 static void    mfip_done(struct mfi_command *cm);
 
@@ -122,6 +132,7 @@ mfip_attach(device_t dev)
 
        mfisc = device_get_softc(device_get_parent(dev));
        sc->dev = dev;
+       sc->state = MFIP_STATE_NONE;
        sc->mfi_sc = mfisc;
        mfisc->mfi_cam_start = mfip_start;
 
@@ -138,6 +149,8 @@ mfip_attach(device_t dev)
                return (EINVAL);
        }
 
+       mfisc->mfi_cam_rescan_cb = mfip_cam_rescan;
+
        mtx_lock(&mfisc->mfi_io_lock);
        if (xpt_bus_register(sc->sim, dev, 0) != 0) {
                device_printf(dev, "XPT bus registration failed\n");
@@ -162,6 +175,16 @@ mfip_detach(device_t dev)
        if (sc == NULL)
                return (EINVAL);
 
+       mtx_lock(&sc->mfi_sc->mfi_io_lock);
+       if (sc->state == MFIP_STATE_RESCAN) {
+               mtx_unlock(&sc->mfi_sc->mfi_io_lock);
+               return (EBUSY);
+       }
+       sc->state = MFIP_STATE_DETACH;
+       mtx_unlock(&sc->mfi_sc->mfi_io_lock);
+
+       sc->mfi_sc->mfi_cam_rescan_cb = NULL;
+
        if (sc->sim != NULL) {
                mtx_lock(&sc->mfi_sc->mfi_io_lock);
                xpt_bus_deregister(cam_sim_path(sc->sim));
@@ -261,6 +284,57 @@ mfip_cam_action(struct cam_sim *sim, uni
        return;
 }
 
+static void
+mfip_cam_rescan(struct mfi_softc *sc, uint32_t tid)
+{
+       union ccb *ccb;
+       struct mfip_softc *camsc;
+       struct cam_sim *sim;
+       device_t mfip_dev;
+
+       mtx_lock(&Giant);
+       mfip_dev = device_find_child(sc->mfi_dev, "mfip", -1);
+       mtx_unlock(&Giant);
+       if (mfip_dev == NULL) {
+               device_printf(sc->mfi_dev, "Couldn't find mfip child 
device!\n");
+               return;
+       }
+
+       mtx_lock(&sc->mfi_io_lock);
+       camsc = device_get_softc(mfip_dev);
+       if (camsc->state == MFIP_STATE_DETACH) {
+               mtx_unlock(&sc->mfi_io_lock);
+               return;
+       }
+       camsc->state = MFIP_STATE_RESCAN;
+       mtx_unlock(&sc->mfi_io_lock);
+
+       ccb = xpt_alloc_ccb_nowait();
+       if (ccb == NULL) {
+               device_printf(sc->mfi_dev,
+                   "Cannot allocate ccb for bus rescan.\n");
+               return;
+       }
+
+       sim = camsc->sim;
+       mtx_lock(&sc->mfi_io_lock);
+       if (xpt_create_path(&ccb->ccb_h.path, xpt_periph, cam_sim_path(sim),
+           tid, CAM_LUN_WILDCARD) != CAM_REQ_CMP) {
+               xpt_free_ccb(ccb);
+               mtx_unlock(&sc->mfi_io_lock);
+               device_printf(sc->mfi_dev,
+                   "Cannot create path for bus rescan.\n");
+               return;
+       }
+       mtx_unlock(&sc->mfi_io_lock);
+
+       xpt_rescan(ccb);
+
+       mtx_lock(&sc->mfi_io_lock);
+       camsc->state = MFIP_STATE_NONE;
+       mtx_unlock(&sc->mfi_io_lock);
+}
+
 static struct mfi_command *
 mfip_start(void *data)
 {

Modified: stable/9/sys/dev/mfi/mfivar.h
==============================================================================
--- stable/9/sys/dev/mfi/mfivar.h       Wed Jul  3 23:06:58 2013        
(r252642)
+++ stable/9/sys/dev/mfi/mfivar.h       Wed Jul  3 23:15:30 2013        
(r252643)
@@ -313,6 +313,8 @@ struct mfi_softc {
 
        TAILQ_HEAD(, ccb_hdr)           mfi_cam_ccbq;
        struct mfi_command *            (* mfi_cam_start)(void *);
+       void                            (*mfi_cam_rescan_cb)(struct mfi_softc *,
+                                           uint32_t);
        struct callout                  mfi_watchdog_callout;
        struct mtx                      mfi_io_lock;
        struct sx                       mfi_config_lock;
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to