kernel/disk: Remove some unused variables and add __debugvar.
authorSascha Wildner <saw@online.de>
Sat, 5 Jan 2013 11:44:01 +0000 (12:44 +0100)
committerSascha Wildner <saw@online.de>
Sat, 5 Jan 2013 11:44:01 +0000 (12:44 +0100)
14 files changed:
sys/dev/disk/advansys/advansys.c
sys/dev/disk/advansys/advlib.c
sys/dev/disk/ahci/ahci_cam.c
sys/dev/disk/ahci/ahci_pm.c
sys/dev/disk/iscsi/initiator/isc_soc.c
sys/dev/disk/md/md.c
sys/dev/disk/nata/ata-raid.c
sys/dev/disk/nata/atapi-cam.c
sys/dev/disk/nata/atapi-cd.c
sys/dev/disk/sbp/sbp.c
sys/dev/disk/sili/sili_cam.c
sys/dev/disk/stg/tmc18c30.c
sys/dev/disk/sym/sym_hipd.c
sys/dev/disk/trm/trm.c

index 2a2af51..2632113 100644 (file)
@@ -1046,7 +1046,6 @@ adv_run_doneq(struct adv_softc *adv)
                struct adv_ccb_info *cinfo;
                u_int done_qaddr;
                u_int sg_queue_cnt;
-               int   aborted;
 
                done_qaddr = ADV_QNO_TO_QADDR(done_qno);
 
@@ -1091,8 +1090,6 @@ adv_run_doneq(struct adv_softc *adv)
 #endif         
                adv->cur_active -= sg_queue_cnt + 1;
 
-               aborted = (scsiq.q_status & QS_ABORTED) != 0;
-
                if ((scsiq.q_status != QS_DONE)
                 && (scsiq.q_status & QS_ABORTED) == 0)
                        panic("adv_qdone: completed scsiq with unknown status");
index 3c40e47..e6bfc03 100644 (file)
@@ -1700,13 +1700,9 @@ adv_send_scsi_queue(struct adv_softc *adv, struct adv_scsi_q *scsiq,
 {
        u_int8_t        free_q_head;
        u_int8_t        next_qp;
-       u_int8_t        tid_no;
-       u_int8_t        target_ix;
        int             retval;
 
        retval = 1;
-       target_ix = scsiq->q2.target_ix;
-       tid_no = ADV_TIX_TO_TID(target_ix);
        free_q_head = adv_read_lram_16(adv, ADVV_FREE_Q_HEAD_W) & 0xFF;
        if ((next_qp = adv_alloc_free_queues(adv, free_q_head, n_q_required))
            != ADV_QLINK_END) {
index 3b5d857..0a24c95 100644 (file)
@@ -206,7 +206,7 @@ ahci_cam_changed(struct ahci_port *ap, struct ata_port *atx, int found)
 void
 ahci_cam_detach(struct ahci_port *ap)
 {
-       int error;
+       int error __debugvar;
 
        if ((ap->ap_flags & AP_F_CAM_ATTACHED) == 0)
                return;
@@ -787,14 +787,12 @@ ahci_xpt_action(struct cam_sim *sim, union ccb *ccb)
        struct ahci_port *ap;
        struct ata_port  *at, *atx;
        struct ccb_hdr *ccbh;
-       int unit;
 
        /* XXX lock */
        ap = cam_sim_softc(sim);
        atx = NULL;
        KKASSERT(ap != NULL);
        ccbh = &ccb->ccb_h;
-       unit = cam_sim_unit(sim);
 
        /*
         * Early failure checks.  These checks do not apply to XPT_PATH_INQ,
index 3ebeff7..6b96b50 100644 (file)
@@ -556,7 +556,6 @@ ahci_pm_softreset(struct ahci_port *ap, int target)
        int                     count;
        int                     error;
        u_int32_t               data;
-       int                     tried_longer;
 
        error = EIO;
        at = ap->ap_ata[target];
@@ -564,7 +563,6 @@ ahci_pm_softreset(struct ahci_port *ap, int target)
        DPRINTF(AHCI_D_VERBOSE, "%s: soft reset\n", PORTNAME(ap));
 
        count = 2;
-       tried_longer = 0;
 retry:
        /*
         * Try to clear the phy so we get a good signature, otherwise
index c77e721..0225c4f 100644 (file)
@@ -431,9 +431,6 @@ so_recv(isc_session_t *sp, pduq_t *pq)
      }
      if(len) {
          int   flags;
-         struct mbuf **mpp;
-
-         mpp = &pq->mp;
 
          sbp.sb_climit = len;
          uio->uio_td = curthread; // why ...
@@ -468,7 +465,6 @@ so_recv(isc_session_t *sp, pduq_t *pq)
                                   pq->iov[1].iov_len = len - pq->pdu.ds_len;
                                   uio->uio_iovcnt++;
                              }
-                             mpp = NULL;
 
                              sdebug(4, "uio_resid=0x%zx itt=0x%x bp=%p bo=%x len=%x/%x",
                                     uio->uio_resid,
index 83f50d5..ed75f81 100644 (file)
@@ -140,12 +140,10 @@ static int
 mdclose(struct dev_close_args *ap)
 {
        cdev_t dev = ap->a_head.a_dev;
-       struct md_s *sc;
 
        if (md_debug)
                kprintf("mdclose(%s %x %x)\n",
                        devtoname(dev), ap->a_fflag, ap->a_devtype);
-       sc = dev->si_drv1;
        atomic_add_int(&refcnt, -1);
 
        return (0);
index 23f642a..1b3a96a 100644 (file)
@@ -257,9 +257,8 @@ ata_raid_flush(struct ar_softc *rdp, struct bio *bp)
 {
     struct ata_request *request;
     device_t dev;
-    int disk, error;
+    int disk;
 
-    error = 0;
     bp->bio_driver_info = NULL;
 
     for (disk = 0; disk < rdp->total_disks; disk++) {
index d019ba4..7a548aa 100644 (file)
@@ -26,7 +26,6 @@
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/dev/ata/atapi-cam.c,v 1.44 2006/03/31 08:09:05 sos Exp $
- * $DragonFly: src/sys/dev/disk/nata/atapi-cam.c,v 1.12 2008/11/19 08:12:16 hasso Exp $
  */
 
 #include "opt_ata.h"
@@ -782,14 +781,10 @@ static void
 atapi_async(void *callback_arg, u_int32_t code,
             struct cam_path* path, void *arg)
 {
-    struct atapi_xpt_softc *softc;
-    struct cam_sim *sim;
     int targ;
 
     crit_enter();
 
-    sim = (struct cam_sim *) callback_arg;
-    softc = (struct atapi_xpt_softc *) cam_sim_softc(sim);
     switch (code) {
     case AC_LOST_DEVICE:
        targ = xpt_path_target_id(path);
index 7001b2b..ed8a7f8 100644 (file)
@@ -809,13 +809,10 @@ acd_start(device_t dev, struct bio *bp)
     struct ata_device *atadev = device_get_softc(dev);
     struct acd_softc *cdp = device_get_ivars(dev);
     struct ata_request *request;
-    cdev_t cdev;
     u_int32_t lba, lastlba, count;
     int8_t ccb[16];
     int track, blocksize;
 
-    cdev = bp->bio_driver_info;
-
     /* reject all queued entries if media changed */
     if (atadev->flags & ATA_D_MEDIA_CHANGED) {
        bbp->b_flags |= B_ERROR;
index 81fd02b..a6d792b 100644 (file)
@@ -657,9 +657,7 @@ static void
 sbp_probe_target(void *arg)
 {
        struct sbp_target *target = (struct sbp_target *)arg;
-       struct sbp_softc *sbp;
        struct sbp_dev *sdev;
-       struct firewire_comm *fc;
        int i, alive;
 
        alive = SBP_FWDEV_ALIVE(target->fwdev);
@@ -669,8 +667,6 @@ SBP_DEBUG(1)
                kprintf("not alive\n");
 END_DEBUG
 
-       sbp = target->sbp;
-       fc = target->sbp->fd.fc;
        sbp_alloc_lun(target);
 
        /* XXX callout_stop mgm_ocb and dequeue */
@@ -896,7 +892,9 @@ static void
 sbp_mgm_callback(struct fw_xfer *xfer)
 {
        struct sbp_dev *sdev;
+#if 0
        int resp;
+#endif
 
        sdev = (struct sbp_dev *)xfer->sc;
 
@@ -904,7 +902,9 @@ SBP_DEBUG(1)
        sbp_show_sdev_info(sdev, 2);
        kprintf("sbp_mgm_callback\n");
 END_DEBUG
+#if 0
        resp = xfer->resp;
+#endif
        sbp_xfer_free(xfer);
 #if 0
        if (resp != 0) {
@@ -1005,11 +1005,9 @@ sbp_do_attach(struct fw_xfer *xfer)
 {
        struct sbp_dev *sdev;
        struct sbp_target *target;
-       struct sbp_softc *sbp;
 
        sdev = (struct sbp_dev *)xfer->sc;
        target = sdev->target;
-       sbp = target->sbp;
 SBP_DEBUG(0)
        sbp_show_sdev_info(sdev, 2);
        kprintf("sbp_do_attach\n");
@@ -1992,14 +1990,12 @@ sbp_free_sdev(struct sbp_dev *sdev)
 static void
 sbp_free_target(struct sbp_target *target)
 {
-       struct sbp_softc *sbp;
        struct fw_xfer *xfer, *next;
        int i;
 
        if (target->luns == NULL)
                return;
        callout_stop(&target->mgm_ocb_timeout);
-       sbp = target->sbp;
        for (i = 0; i < target->num_lun; i++)
                sbp_free_sdev(target->luns[i]);
 
@@ -2574,7 +2570,6 @@ sbp_dequeue_ocb(struct sbp_dev *sdev, struct sbp_status *sbp_status)
        struct sbp_ocb *ocb;
        struct sbp_ocb *next;
        int order = 0;
-       int flags;
 
        crit_enter();
 
@@ -2585,7 +2580,6 @@ SBP_DEBUG(1)
 END_DEBUG
        for (ocb = STAILQ_FIRST(&sdev->ocbs); ocb != NULL; ocb = next) {
                next = STAILQ_NEXT(ocb, ocb);
-               flags = ocb->flags;
                if (OCB_MATCH(ocb, sbp_status)) {
                        /* found */
                        STAILQ_REMOVE(&sdev->ocbs, ocb, sbp_ocb, ocb);
index ec4350b..cd6e69f 100644 (file)
@@ -195,7 +195,7 @@ sili_cam_changed(struct sili_port *ap, struct ata_port *atx, int found)
 void
 sili_cam_detach(struct sili_port *ap)
 {
-       int error;
+       int error __debugvar;
 
        if ((ap->ap_flags & AP_F_CAM_ATTACHED) == 0)
                return;
@@ -785,7 +785,6 @@ sili_xpt_action(struct cam_sim *sim, union ccb *ccb)
        struct sili_port *ap;
        struct ata_port  *at, *atx;
        struct ccb_hdr *ccbh;
-       int unit;
 
        /* XXX lock */
        ap = cam_sim_softc(sim);
@@ -793,7 +792,6 @@ sili_xpt_action(struct cam_sim *sim, union ccb *ccb)
        atx = NULL;
        KKASSERT(ap != NULL);
        ccbh = &ccb->ccb_h;
-       unit = cam_sim_unit(sim);
 
        /*
         * Early failure checks.  These checks do not apply to XPT_PATH_INQ,
index c1a069b..d3c49c5 100644 (file)
@@ -339,7 +339,7 @@ stg_msg(struct stg_softc *sc, struct targ_info *ti, u_int msg)
        bus_space_tag_t iot = sc->sc_iot;
        bus_space_handle_t ioh = sc->sc_ioh;
        struct stg_targ_info *sti = (void *) ti;
-       u_int period, offset;
+       u_int period;
 
        if ((msg & SCSI_LOW_MSG_WIDE) != 0)
        {
@@ -355,7 +355,6 @@ stg_msg(struct stg_softc *sc, struct targ_info *ti, u_int msg)
                return 0;
 
        period = ti->ti_maxsynch.period;
-       offset = ti->ti_maxsynch.offset;
        period = period << 2;
        if (period >= 200)
        {
index ce2b27b..1b11a49 100644 (file)
@@ -5509,7 +5509,7 @@ out_reject:
 
 static int sym_compute_residual(hcb_p np, ccb_p cp)
 {
-       int dp_sg, dp_sgmin, resid = 0;
+       int dp_sg, resid = 0;
        int dp_ofs = 0;
 
        /*
@@ -5556,7 +5556,6 @@ static int sym_compute_residual(hcb_p np, ccb_p cp)
         *  We are now full comfortable in the computation
         *  of the data residual (2's complement).
         */
-       dp_sgmin = SYM_CONF_MAX_SG - cp->segments;
        resid = -cp->ext_ofs;
        for (dp_sg = cp->ext_sg; dp_sg < SYM_CONF_MAX_SG; ++dp_sg) {
                u_int tmp = scr_to_cpu(cp->phys.data[dp_sg].size);
index 464df3e..e23f7be 100644 (file)
@@ -3546,15 +3546,10 @@ static int
 trm_attach(device_t dev)
 {
        struct  cam_devq *device_Q;
-       u_long  device_id;
        PACB    pACB = NULL;
        int     rid = 0;
        int unit = device_get_unit(dev);
        
-       device_id = pci_get_devid(dev);
-       /*
-        * These cards do not allow memory mapped accesses
-        */
        if ((pACB = trm_init((u_int16_t) unit,
            dev)) == NULL) {
                kprintf("trm%d: trm_init error!\n",unit);