Author: mav
Date: Tue Jun  5 09:45:42 2012
New Revision: 236602
URL: http://svn.freebsd.org/changeset/base/236602

Log:
  Tune and add some missing CAM_DEBUG() points for better consistency.

Modified:
  head/sys/cam/ata/ata_da.c
  head/sys/cam/ata/ata_pmp.c
  head/sys/cam/scsi/scsi_cd.c
  head/sys/cam/scsi/scsi_da.c
  head/sys/cam/scsi/scsi_pt.c

Modified: head/sys/cam/ata/ata_da.c
==============================================================================
--- head/sys/cam/ata/ata_da.c   Tue Jun  5 06:43:45 2012        (r236601)
+++ head/sys/cam/ata/ata_da.c   Tue Jun  5 09:45:42 2012        (r236602)
@@ -436,9 +436,8 @@ adaopen(struct disk *dp)
        softc = (struct ada_softc *)periph->softc;
        softc->flags |= ADA_FLAG_OPEN;
 
-       CAM_DEBUG(periph->path, CAM_DEBUG_TRACE,
-           ("adaopen: disk=%s%d (unit %d)\n", dp->d_name, dp->d_unit,
-            periph->unit_number));
+       CAM_DEBUG(periph->path, CAM_DEBUG_TRACE | CAM_DEBUG_PERIPH,
+           ("adaopen\n"));
 
        if ((softc->flags & ADA_FLAG_PACK_INVALID) != 0) {
                /* Invalidate our pack information. */
@@ -469,6 +468,10 @@ adaclose(struct disk *dp)
        }
 
        softc = (struct ada_softc *)periph->softc;
+
+       CAM_DEBUG(periph->path, CAM_DEBUG_TRACE | CAM_DEBUG_PERIPH,
+           ("adaclose\n"));
+
        /* We only sync the cache if the drive is capable of it. */
        if ((softc->flags & ADA_FLAG_CAN_FLUSHCACHE) != 0 &&
            (softc->flags & ADA_FLAG_PACK_INVALID) == 0) {
@@ -542,6 +545,8 @@ adastrategy(struct bio *bp)
 
        cam_periph_lock(periph);
 
+       CAM_DEBUG(periph->path, CAM_DEBUG_TRACE, ("adastrategy(%p)\n", bp));
+
        /*
         * If the device has been made invalid, error out
         */
@@ -1167,6 +1172,8 @@ adastart(struct cam_periph *periph, unio
        struct ada_softc *softc = (struct ada_softc *)periph->softc;
        struct ccb_ataio *ataio = &start_ccb->ataio;
 
+       CAM_DEBUG(periph->path, CAM_DEBUG_TRACE, ("adastart\n"));
+
        switch (softc->state) {
        case ADA_STATE_NORMAL:
        {
@@ -1175,7 +1182,7 @@ adastart(struct cam_periph *periph, unio
 
                /* Execute immediate CCB if waiting. */
                if (periph->immediate_priority <= periph->pinfo.priority) {
-                       CAM_DEBUG_PRINT(CAM_DEBUG_SUBTRACE,
+                       CAM_DEBUG(periph->path, CAM_DEBUG_SUBTRACE,
                                        ("queuing for immediate ccb\n"));
                        start_ccb->ccb_h.ccb_state = ADA_CCB_WAITING;
                        SLIST_INSERT_HEAD(&periph->ccb_list, &start_ccb->ccb_h,
@@ -1467,6 +1474,9 @@ adadone(struct cam_periph *periph, union
 
        softc = (struct ada_softc *)periph->softc;
        ataio = &done_ccb->ataio;
+
+       CAM_DEBUG(periph->path, CAM_DEBUG_TRACE, ("adadone\n"));
+
        switch (ataio->ccb_h.ccb_state & ADA_CCB_TYPE_MASK) {
        case ADA_CCB_BUFFER_IO:
        case ADA_CCB_TRIM:

Modified: head/sys/cam/ata/ata_pmp.c
==============================================================================
--- head/sys/cam/ata/ata_pmp.c  Tue Jun  5 06:43:45 2012        (r236601)
+++ head/sys/cam/ata/ata_pmp.c  Tue Jun  5 09:45:42 2012        (r236602)
@@ -429,7 +429,9 @@ pmpstart(struct cam_periph *periph, unio
 
        softc = (struct pmp_softc *)periph->softc;
        ataio = &start_ccb->ataio;
-       
+
+       CAM_DEBUG(periph->path, CAM_DEBUG_TRACE, ("pmpstart\n"));
+
        if (softc->restart) {
                softc->restart = 0;
                softc->state = min(softc->state, PMP_STATE_PRECONFIG);
@@ -560,7 +562,7 @@ pmpdone(struct cam_periph *periph, union
        softc = (struct pmp_softc *)periph->softc;
        ataio = &done_ccb->ataio;
 
-       CAM_DEBUG(done_ccb->ccb_h.path, CAM_DEBUG_TRACE, ("pmpdone\n"));
+       CAM_DEBUG(periph->path, CAM_DEBUG_TRACE, ("pmpdone\n"));
 
        priority = done_ccb->ccb_h.pinfo.priority;
 

Modified: head/sys/cam/scsi/scsi_cd.c
==============================================================================
--- head/sys/cam/scsi/scsi_cd.c Tue Jun  5 06:43:45 2012        (r236601)
+++ head/sys/cam/scsi/scsi_cd.c Tue Jun  5 09:45:42 2012        (r236602)
@@ -1014,6 +1014,9 @@ cdopen(struct disk *dp)
                return (error);
        }
 
+       CAM_DEBUG(periph->path, CAM_DEBUG_TRACE | CAM_DEBUG_PERIPH,
+           ("cdopen\n"));
+
        /*
         * Check for media, and set the appropriate flags.  We don't bail
         * if we don't have media, but then we don't allow anything but the
@@ -1051,6 +1054,9 @@ cdclose(struct disk *dp)
        cam_periph_lock(periph);
        cam_periph_hold(periph, PRIBIO);
 
+       CAM_DEBUG(periph->path, CAM_DEBUG_TRACE | CAM_DEBUG_PERIPH,
+           ("cdclose\n"));
+
        if ((softc->flags & CD_FLAG_DISC_REMOVABLE) != 0)
                cdprevent(periph, PR_ALLOW);
 
@@ -1395,7 +1401,8 @@ cdstrategy(struct bio *bp)
        }
 
        cam_periph_lock(periph);
-       CAM_DEBUG(periph->path, CAM_DEBUG_TRACE, ("entering cdstrategy\n"));
+       CAM_DEBUG(periph->path, CAM_DEBUG_TRACE,
+           ("cdstrategy(%p)\n", bp));
 
        softc = (struct cd_softc *)periph->softc;
 
@@ -1861,12 +1868,11 @@ cdioctl(struct disk *dp, u_long cmd, voi
                return(ENXIO);  
 
        cam_periph_lock(periph);
-       CAM_DEBUG(periph->path, CAM_DEBUG_TRACE, ("entering cdioctl\n"));
 
        softc = (struct cd_softc *)periph->softc;
 
-       CAM_DEBUG(periph->path, CAM_DEBUG_TRACE, 
-                 ("trying to do ioctl %#lx\n", cmd));
+       CAM_DEBUG(periph->path, CAM_DEBUG_TRACE,
+           ("cdioctl(%#lx)\n", cmd));
 
        if ((error = cam_periph_hold(periph, PRIBIO | PCATCH)) != 0) {
                cam_periph_unlock(periph);

Modified: head/sys/cam/scsi/scsi_da.c
==============================================================================
--- head/sys/cam/scsi/scsi_da.c Tue Jun  5 06:43:45 2012        (r236601)
+++ head/sys/cam/scsi/scsi_da.c Tue Jun  5 09:45:42 2012        (r236602)
@@ -942,9 +942,8 @@ daopen(struct disk *dp)
        softc = (struct da_softc *)periph->softc;
        softc->flags |= DA_FLAG_OPEN;
 
-       CAM_DEBUG(periph->path, CAM_DEBUG_TRACE,
-           ("daopen: disk=%s%d (unit %d)\n", dp->d_name, dp->d_unit,
-            unit));
+       CAM_DEBUG(periph->path, CAM_DEBUG_TRACE | CAM_DEBUG_PERIPH,
+           ("daopen\n"));
 
        if ((softc->flags & DA_FLAG_PACK_INVALID) != 0) {
                /* Invalidate our pack information. */
@@ -999,6 +998,9 @@ daclose(struct disk *dp)
 
        softc = (struct da_softc *)periph->softc;
 
+       CAM_DEBUG(periph->path, CAM_DEBUG_TRACE | CAM_DEBUG_PERIPH,
+           ("daclose\n"));
+
        if ((softc->quirks & DA_Q_NO_SYNC_CACHE) == 0
         && (softc->flags & DA_FLAG_PACK_INVALID) == 0) {
                union   ccb *ccb;
@@ -1108,7 +1110,9 @@ dastrategy(struct bio *bp)
                biofinish(bp, NULL, ENXIO);
                return;
        }
-       
+
+       CAM_DEBUG(periph->path, CAM_DEBUG_TRACE, ("dastrategy(%p)\n", bp));
+
        /*
         * Place it in the queue of disk activities for this disk
         */
@@ -1724,6 +1728,8 @@ dastart(struct cam_periph *periph, union
 
        softc = (struct da_softc *)periph->softc;
 
+       CAM_DEBUG(periph->path, CAM_DEBUG_TRACE, ("dastart\n"));
+
        switch (softc->state) {
        case DA_STATE_NORMAL:
        {
@@ -1732,7 +1738,7 @@ dastart(struct cam_periph *periph, union
 
                /* Execute immediate CCB if waiting. */
                if (periph->immediate_priority <= periph->pinfo.priority) {
-                       CAM_DEBUG_PRINT(CAM_DEBUG_SUBTRACE,
+                       CAM_DEBUG(periph->path, CAM_DEBUG_SUBTRACE,
                                        ("queuing for immediate ccb\n"));
                        start_ccb->ccb_h.ccb_state = DA_CCB_WAITING;
                        SLIST_INSERT_HEAD(&periph->ccb_list, &start_ccb->ccb_h,
@@ -2064,6 +2070,9 @@ dadone(struct cam_periph *periph, union 
 
        softc = (struct da_softc *)periph->softc;
        priority = done_ccb->ccb_h.pinfo.priority;
+
+       CAM_DEBUG(periph->path, CAM_DEBUG_TRACE, ("dadone\n"));
+
        csio = &done_ccb->csio;
        switch (csio->ccb_h.ccb_state & DA_CCB_TYPE_MASK) {
        case DA_CCB_BUFFER_IO:

Modified: head/sys/cam/scsi/scsi_pt.c
==============================================================================
--- head/sys/cam/scsi/scsi_pt.c Tue Jun  5 06:43:45 2012        (r236601)
+++ head/sys/cam/scsi/scsi_pt.c Tue Jun  5 09:45:42 2012        (r236602)
@@ -425,12 +425,14 @@ ptstart(struct cam_periph *periph, union
 
        softc = (struct pt_softc *)periph->softc;
 
+       CAM_DEBUG(periph->path, CAM_DEBUG_TRACE, ("ptstart\n"));
+
        /*
         * See if there is a buf with work for us to do..
         */
        bp = bioq_first(&softc->bio_queue);
        if (periph->immediate_priority <= periph->pinfo.priority) {
-               CAM_DEBUG_PRINT(CAM_DEBUG_SUBTRACE,
+               CAM_DEBUG(periph->path, CAM_DEBUG_SUBTRACE,
                                ("queuing for immediate ccb\n"));
                start_ccb->ccb_h.ccb_state = PT_CCB_WAITING;
                SLIST_INSERT_HEAD(&periph->ccb_list, &start_ccb->ccb_h,
@@ -483,6 +485,9 @@ ptdone(struct cam_periph *periph, union 
        struct ccb_scsiio *csio;
 
        softc = (struct pt_softc *)periph->softc;
+
+       CAM_DEBUG(periph->path, CAM_DEBUG_TRACE, ("ptdone\n"));
+
        csio = &done_ccb->csio;
        switch (csio->ccb_h.ccb_state) {
        case PT_CCB_BUFFER_IO:
_______________________________________________
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to