From owner-freebsd-current Mon Feb 11 17:31: 8 2002 Delivered-To: freebsd-current@freebsd.org Received: from mail11.speakeasy.net (mail11.speakeasy.net [216.254.0.211]) by hub.freebsd.org (Postfix) with ESMTP id 4672337B419 for ; Mon, 11 Feb 2002 17:31:06 -0800 (PST) Received: (qmail 22227 invoked from network); 12 Feb 2002 01:31:04 -0000 Received: from unknown (HELO laptop.baldwin.cx) ([65.91.152.142]) (envelope-sender ) by mail11.speakeasy.net (qmail-ldap-1.03) with SMTP for ; 12 Feb 2002 01:31:04 -0000 Message-ID: X-Mailer: XFMail 1.4.0 on FreeBSD X-Priority: 3 (Normal) Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 8bit MIME-Version: 1.0 In-Reply-To: <3C68400E.7F3FF1CE@elischer.org> Date: Mon, 11 Feb 2002 20:30:54 -0500 (EST) From: John Baldwin To: Julian Elischer Subject: RE: ucred holding patch, BDE version Cc: current@freebsd.org, bde@freebsd.org Sender: owner-freebsd-current@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On 11-Feb-02 Julian Elischer wrote: > here is the BDE version ready to commit. > Extended to other architectures. > > Bruce, John, comments? > > As I was adding a prototype to ucred.h I stripped the __Ps of the others in > that > section > (in the spirit of "change it when editing it anyhow" Hmm, acquire_ucred (don't really like that name, maybe thread_updatecred(td) which can use td_proc to get the proc) probably should be declared in sys/proc.h. Well, maybe not, sys/ucred.h is probably fine. But it's implementation should then be in kern_prot.c along with all the other ucred related functions. :) Also, please make the comment above the function into a complete sentence and capitalize appropriately, etc. as per style(9) just to be pedantic. I guess removing __P() as you go is ok if that spirit is what the -arch thread is desired. Personally I thought it should be the other way around just like we don't mix whitespace commits with code commits to avoid obfuscating function changes with style changes. IMO, just commit to ucred.h blowing away __P() first, then commit your functional changes with the rest. -- John Baldwin <>< http://www.FreeBSD.org/~jhb/ "Power Users Use the Power to Serve!" - http://www.FreeBSD.org/ To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-current" in the body of the message