From e3033defcc77bf5f695b2625c29e1efe1cf8e5bd Mon Sep 17 00:00:00 2001 From: Sam Leffler Date: Mon, 26 Jan 2009 23:05:50 +0000 Subject: [PATCH] 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 --- etc/devd.conf | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/etc/devd.conf b/etc/devd.conf index 8dbf34b605c..f4f04ad5588 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