PERFORCE change 113809 for review

[M. Warner Losh]

In message: <20070201133157.X12827 at fledge.watson.org>
            Robert Watson <rwatson at FreeBSD.org> writes:
: 
: On Thu, 1 Feb 2007, Warner Losh wrote:
: 
: > http://perforce.freebsd.org/chv.cgi?CH=113809
: >
: > Change 113809 by imp at imp_lighthouse on 2007/02/01 06:30:17
: >
: > 	Kinky new idea: run pccard_ether in the background.  This allows devd
: > 	to respond to things that are happening while pccard_ether is running.
: 
: Hmm.  Perhaps it would be better to serialize all events associated with a 
: particular device in a particular process or thread, so that events from ed0 
: and ed1 can run concurrently, but not two events for ed0?  I realize that's 
: quite a bit more complicated than slapping '&' in there, but it might provide 
: some useful consistency from the perspective of people writing scripts to 
: start and stop file systems, etc, on removable storage (for example).

I'm pretty sure that devd isn't the place for this.  Such cooridnation
are best done with the scripts that do the configuration.  How does
devd know to enforce this?  Only one fs at a time can run?  Only one
device + unit?  Only one device?  If there are issues, then things
like pccard_ether will need to get smarter about them.

I'm sure there will be issues with link UP/DOWN events.

Warner

: Robert N M Watson
: Computer Laboratory
: University of Cambridge
: 
: >
: > Affected files ...
: >
: > .. //depot/projects/arm/src/etc/devd.conf#2 edit
: >
: > Differences ...
: >
: > ==== //depot/projects/arm/src/etc/devd.conf#2 (text+ko) ====
: >
: > @@ -34,13 +34,13 @@
: > notify 0 {
: > 	match "system"		"IFNET";
: > 	match "type"		"ATTACH";
: > -	action "/etc/pccard_ether $subsystem start";
: > +	action "/etc/pccard_ether $subsystem start &";
: > };
: >
: > notify 0 {
: > 	match "system"		"IFNET";
: > 	match "type"		"DETACH";
: > -	action "/etc/pccard_ether $subsystem stop";
: > +	action "/etc/pccard_ether $subsystem stop &";
: > };
: >
: > #
: > @@ -63,11 +63,11 @@
: > #
: > detach 0 {
: > 	media-type "802.11";
: > -	action "/etc/pccard_ether $device-name stop";
: > +	action "/etc/pccard_ether $device-name stop &";
: > };
: > attach 0 {
: > 	media-type "802.11";
: > -	action "/etc/pccard_ether $device-name start";
: > +	action "/etc/pccard_ether $device-name start &";
: > };
: > notify 0 {
: > 	match "system"		"IFNET";
: >
: