Print this page
9702 HBA drivers don't need the redundant devfs_clean step
Reviewed by: Dan Fields <dan.fields@nexenta.com>
Reviewed by: Rick McNeal <rick.mcneal@nexenta.com>

@@ -8,20 +8,19 @@
  * source.  A copy of the CDDL is also available via the Internet at
  * http://www.illumos.org/license/CDDL.
  */
 
 /*
- * Copyright 2016 Nexenta Systems, Inc.
+ * Copyright 2018 Nexenta Systems, Inc.
  */
 
 #include <sys/atomic.h>
 #include <sys/cmn_err.h>
 #include <sys/conf.h>
 #include <sys/cpuvar.h>
 #include <sys/ddi.h>
 #include <sys/errno.h>
-#include <sys/fs/dv_node.h>
 #include <sys/kmem.h>
 #include <sys/kmem_impl.h>
 #include <sys/list.h>
 #include <sys/modctl.h>
 #include <sys/pci.h>

@@ -358,24 +357,12 @@
                         break;
         }
 
         if (devnode != NULL) {
                 if (inqrc != 0) {
-                        /* Target disappeared, drop devnode */
-                        if (i_ddi_devi_attached(devnode->pdip)) {
-                                char    *devname;
-                                /* Get full devname */
-                                devname = kmem_alloc(MAXPATHLEN, KM_SLEEP);
-                                (void) ddi_deviname(devnode->pdip, devname);
-                                /* Clean cache and name */
-                                (void) devfs_clean(devnode->parent, devname + 1,
-                                    DV_CLEAN_FORCE);
-                                kmem_free(devname, MAXPATHLEN);
-                        }
-
-                        (void) ndi_devi_offline(devnode->pdip, NDI_DEVI_REMOVE);
-
+                        (void) ndi_devi_offline(devnode->pdip,
+                            NDI_DEVFS_CLEAN | NDI_DEVI_REMOVE);
                         list_remove(&pvs->devnodes, devnode);
                         kmem_free(devnode, sizeof (*devnode));
                 } else if (childp != NULL) {
                         /* Target exists */
                         *childp = devnode->pdip;