Remove DETACH event handling; this is race prone and does nothing useful.

Leave a comment for the next person that thinks they need to be helpful.

Reviewed by:	imp, jhb
MFC after:	2 weeks

1 files changed, 5 insertions, 6 deletions

diff --git a/etc/devd.conf b/etc/devd.conf
index 8dbf34b..f4f04ad 100644
--- a/etc/devd.conf
+++ b/etc/devd.conf
@@ -31,18 +31,17 @@ options {
 # Configure the interface on attach.  Due to a historical accident, this
 # script is called pccard_ether.
 #
+# NB: DETACH events are ignored; the kernel should handle all cleanup
+#     (routes, arp cache) if you need to do something beware of races
+#     against immediate create of a device w/ the same name; e.g.
+#	ifconfig bridge0 destroy; ifconfig bridge0 create
+#
 notify 0 {
 	match "system"		"IFNET";
 	match "type"		"ATTACH";
 	action "/etc/pccard_ether $subsystem start";
 };
 
-notify 0 {
-	match "system"		"IFNET";
-	match "type"		"DETACH";
-	action "/etc/pccard_ether $subsystem stop";
-};
-
 #
 # Try to start dhclient on Ethernet like interfaces when the link comes
 # up.  Only devices that are configured to support DHCP will actually