Another allocation/memory use made big mem friendly.

.... Ken

Index: ahci.c
===================================================================
RCS file: /cvs/src/sys/dev/pci/ahci.c,v
retrieving revision 1.172
diff -u -p -r1.172 ahci.c
--- ahci.c      28 Jan 2011 06:32:31 -0000      1.172
+++ ahci.c      3 Apr 2011 13:35:05 -0000
@@ -27,6 +27,7 @@
 #include <sys/timeout.h>
 #include <sys/queue.h>
 #include <sys/mutex.h>
+#include <sys/pool.h>
 
 #include <machine/bus.h>
 
@@ -391,8 +392,6 @@ struct ahci_port {
        u_int32_t               ap_err_saved_active;
        u_int32_t               ap_err_saved_active_cnt;
 
-       u_int8_t                ap_err_scratch[512];
-
 #ifdef AHCI_DEBUG
        char                    ap_name[16];
 #define PORTNAME(_ap)  ((_ap)->ap_name)
@@ -3098,9 +3097,10 @@ ahci_port_read_ncq_error(struct ahci_por
 {
        struct ahci_ccb                 *ccb;
        struct ahci_cmd_hdr             *cmd_slot;
-       u_int32_t                       cmd;
+       u_int8_t                        *ap_err_scratch = NULL;
        struct ata_fis_h2d              *fis;
        int                             rc = EIO;
+       u_int32_t                       cmd;
 
        DPRINTF(AHCI_D_VERBOSE, "%s: read log page\n", PORTNAME(ap));
 
@@ -3112,10 +3112,11 @@ ahci_port_read_ncq_error(struct ahci_por
        ahci_port_start(ap, 0);
 
        /* Prep error CCB for READ LOG EXT, page 10h, 1 sector. */
+       ap_err_scratch = dma_alloc(DEV_BSIZE, PR_WAITOK | PR_ZERO);
        ccb = ahci_get_err_ccb(ap);
        ccb->ccb_xa.flags = ATA_F_NOWAIT | ATA_F_READ | ATA_F_POLL;
-       ccb->ccb_xa.data = ap->ap_err_scratch;
-       ccb->ccb_xa.datalen = 512;
+       ccb->ccb_xa.data = ap_err_scratch;
+       ccb->ccb_xa.datalen = DEV_BSIZE;
        cmd_slot = ccb->ccb_cmd_hdr;
        bzero(ccb->ccb_cmd_table, sizeof(struct ahci_cmd_table));
 
@@ -3160,7 +3161,7 @@ err:
                struct ata_log_page_10h         *log;
                int                             err_slot;
 
-               log = (struct ata_log_page_10h *)ap->ap_err_scratch;
+               log = (struct ata_log_page_10h *)ap_err_scratch;
                if (ISSET(log->err_regs.type, ATA_LOG_10H_TYPE_NOTQUEUED)) {
                        /* Not queued bit was set - wasn't an NCQ error? */
                        printf("%s: read NCQ error page, but not an NCQ "
@@ -3181,6 +3182,9 @@ err:
 
        /* Restore saved CMD register state */
        ahci_pwrite(ap, AHCI_PREG_CMD, cmd);
+
+       if (ap_err_scratch)
+               dma_free(ap_err_scratch, DEV_BSIZE);
 
        return (rc);
 }

Reply via email to