From owner-p4-projects@FreeBSD.ORG  Thu Feb  1 14:02:23 2007
Return-Path: <owner-p4-projects@FreeBSD.ORG>
X-Original-To: p4-projects@freebsd.org
Delivered-To: p4-projects@freebsd.org
Received: by hub.freebsd.org (Postfix, from userid 32767)
	id 948BC16A40F; Thu,  1 Feb 2007 14:02:23 +0000 (UTC)
X-Original-To: perforce@freebsd.org
Delivered-To: perforce@freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52])
	by hub.freebsd.org (Postfix) with ESMTP id 3F0B316A402;
	Thu,  1 Feb 2007 14:02:23 +0000 (UTC)
	(envelope-from rwatson@FreeBSD.org)
Received: from cyrus.watson.org (cyrus.watson.org [209.31.154.42])
	by mx1.freebsd.org (Postfix) with ESMTP id 1098D13C4B2;
	Thu,  1 Feb 2007 14:02:22 +0000 (UTC)
	(envelope-from rwatson@FreeBSD.org)
Received: from fledge.watson.org (fledge.watson.org [209.31.154.41])
	by cyrus.watson.org (Postfix) with ESMTP id ACE254D015;
	Thu,  1 Feb 2007 08:33:52 -0500 (EST)
Date: Thu, 1 Feb 2007 13:33:51 +0000 (GMT)
From: Robert Watson <rwatson@FreeBSD.org>
X-X-Sender: robert@fledge.watson.org
To: Warner Losh <imp@FreeBSD.org>
In-Reply-To: <200702010631.l116VFBJ006811@repoman.freebsd.org>
Message-ID: <20070201133157.X12827@fledge.watson.org>
References: <200702010631.l116VFBJ006811@repoman.freebsd.org>
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed
Cc: Perforce Change Reviews <perforce@freebsd.org>
Subject: Re: PERFORCE change 113809 for review
X-BeenThere: p4-projects@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: p4 projects tree changes <p4-projects.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/p4-projects>,
	<mailto:p4-projects-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/p4-projects>
List-Post: <mailto:p4-projects@freebsd.org>
List-Help: <mailto:p4-projects-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/p4-projects>,
	<mailto:p4-projects-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Thu, 01 Feb 2007 14:02:23 -0000


On Thu, 1 Feb 2007, Warner Losh wrote:

> http://perforce.freebsd.org/chv.cgi?CH=113809
>
> Change 113809 by imp@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).

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";
>