svn commit: r187743 - head/etc

[Sam Leffler]

Author: sam
Date: Mon Jan 26 23:05:50 2009
New Revision: 187743
URL: http://svn.freebsd.org/changeset/base/187743

Log:
  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

Modified:
  head/etc/devd.conf

Modified: head/etc/devd.conf
==============================================================================
--- head/etc/devd.conf	Mon Jan 26 21:45:33 2009	(r187742)
+++ head/etc/devd.conf	Mon Jan 26 23:05:50 2009	(r187743)
@@ -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