From owner-freebsd-current@FreeBSD.ORG Thu May 19 19:05:01 2005 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 898A716A4CE; Thu, 19 May 2005 19:05:01 +0000 (GMT) Received: from www.cryptography.com (li-22.members.linode.com [64.5.53.22]) by mx1.FreeBSD.org (Postfix) with ESMTP id DDE8843DCA; Thu, 19 May 2005 19:05:00 +0000 (GMT) (envelope-from nate@root.org) Received: from [10.0.0.34] (adsl-67-119-74-222.dsl.sntc01.pacbell.net [67.119.74.222]) by www.cryptography.com (8.12.8/8.12.8) with ESMTP id j4JJ4bLS028164 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NOT); Thu, 19 May 2005 12:04:59 -0700 Message-ID: <428CE32F.20307@root.org> Date: Thu, 19 May 2005 12:04:15 -0700 From: Nate Lawson User-Agent: Mozilla Thunderbird 1.0.2 (Windows/20050317) X-Accept-Language: en-us, en MIME-Version: 1.0 To: Fredrik Lindberg References: <428C1EA6.7020907@root.org> <20050519104207.GA1259@shapeshifter.se> In-Reply-To: <20050519104207.GA1259@shapeshifter.se> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit cc: Warner Losh cc: current@FreeBSD.org Subject: Re: Patch to make powerd(8) devd(8)-aware X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 19 May 2005 19:05:01 -0000 Fredrik Lindberg wrote: > On Wed, May 18, 2005 at 10:05:42PM -0700, Nate Lawson (nate@root.org) wrote: > >>I appreciate you working on a patch for this. However, I'd prefer the >>work on the devd side go into making devctl a clonable device or follow >>the bpf route and hook up /dev/devctl0,1,2,... If you start using the >>/var/run/devd.pipe, it's not available for other consumers. > > Yes, devctl should probably be able to have more readers, I won't argue with that. > However, devd allows any number of consumers to the devd.pipe. I've had > upto 4 programs reading devd.pipe simultaneously and they all got the same data. Ah, I wasn't aware of that. I thought there might be a race for who got each message. This is fine then, and perhaps we should retire devctl as a way to read messages since it is limited for that purpose and duplicates the pipe. We could keep it around for reverse signaling (sending messages to the kernel, which is needed eventually). >>Other comments: >>style(9) long sscanf line > > > I'm familiar with style(9), however I don't recall anything about sscanf, could you > please explain the correct behaviour in this case (except trying to reducing indent) I just think you should wrap the lines >80 cols. Pretty easy to do this by outdenting the string. Example: printf( "a really long string but less than 80 cols\n"); >>extra newlines inserted into existing code >>change read() in devd_read() to return on error instead of running >>sscanf -- this will allow the indent to be reduced. > > My mistakes, sorry. No problems, thanks for the work. With these changes, I'm fine with the patch being committed. I hope you can also fix the main loop soft error cases as well. -- Nate