Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/sys/dev/ic fix ahci_detach() to count the ports propertly (s...



details:   https://anonhg.NetBSD.org/src/rev/331146fb6dbd
branches:  trunk
changeset: 994358:331146fb6dbd
user:      jdolecek <jdolecek%NetBSD.org@localhost>
date:      Fri Nov 02 21:27:30 2018 +0000

description:
fix ahci_detach() to count the ports propertly (same as attach), to avoid
triggerring panic when disks are plugged to non-adjacent ports - this should
fix panic reported by Masanobu SAITOH on current-users

also fix different miscounting of ports in ahci_setup_ports() and
ahci_reprobe_drives()

diffstat:

 sys/dev/ic/ahcisata_core.c |  13 ++++++++-----
 1 files changed, 8 insertions(+), 5 deletions(-)

diffs (65 lines):

diff -r 074892c73d64 -r 331146fb6dbd sys/dev/ic/ahcisata_core.c
--- a/sys/dev/ic/ahcisata_core.c        Fri Nov 02 19:57:32 2018 +0000
+++ b/sys/dev/ic/ahcisata_core.c        Fri Nov 02 21:27:30 2018 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: ahcisata_core.c,v 1.65 2018/10/24 19:38:00 jdolecek Exp $      */
+/*     $NetBSD: ahcisata_core.c,v 1.66 2018/11/02 21:27:30 jdolecek Exp $      */
 
 /*
  * Copyright (c) 2006 Manuel Bouyer.
@@ -26,7 +26,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: ahcisata_core.c,v 1.65 2018/10/24 19:38:00 jdolecek Exp $");
+__KERNEL_RCSID(0, "$NetBSD: ahcisata_core.c,v 1.66 2018/11/02 21:27:30 jdolecek Exp $");
 
 #include <sys/types.h>
 #include <sys/malloc.h>
@@ -184,6 +184,7 @@
                        break;
                }
                ahci_setup_port(sc, i);
+               port++;
        }
 }
 
@@ -206,6 +207,7 @@
                chp = &achp->ata_channel;
 
                ahci_probe_drive(chp);
+               port++;
        }
 }
 
@@ -502,19 +504,19 @@
        struct ahci_channel *achp;
        struct ata_channel *chp;
        struct scsipi_adapter *adapt;
-       int i, j;
+       int i, j, port;
        int error;
 
        atac = &sc->sc_atac;
        adapt = &atac->atac_atapi_adapter._generic;
 
-       for (i = 0; i < AHCI_MAX_PORTS; i++) {
+       for (i = 0, port = 0; i < AHCI_MAX_PORTS; i++) {
                achp = &sc->sc_channels[i];
                chp = &achp->ata_channel;
 
                if ((sc->sc_ahci_ports & (1U << i)) == 0)
                        continue;
-               if (i >= sc->sc_atac.atac_nchannels) {
+               if (port >= sc->sc_atac.atac_nchannels) {
                        aprint_error("%s: more ports than announced\n",
                            AHCINAME(sc));
                        break;
@@ -541,6 +543,7 @@
                    achp->ahcic_cmd_tbl_nseg);
 
                ata_channel_detach(chp);
+               port++;
        }
 
        bus_dmamap_unload(sc->sc_dmat, sc->sc_cmd_hdrd);



Home | Main Index | Thread Index | Old Index