From owner-freebsd-current Mon Feb 11 19:35: 8 2002 Delivered-To: freebsd-current@freebsd.org Received: from newman2.bestweb.net (newman2.bestweb.net [209.94.102.67]) by hub.freebsd.org (Postfix) with ESMTP id 1A6B037B670; Mon, 11 Feb 2002 18:19:26 -0800 (PST) Received: from okeeffe.bestweb.net (okeefe.bestweb.net [209.94.100.110]) by newman2.bestweb.net (Postfix) with ESMTP id A40A72320E; Mon, 11 Feb 2002 21:18:19 -0500 (EST) Received: by okeeffe.bestweb.net (Postfix, from userid 0) id B89A39F413; Mon, 11 Feb 2002 21:12:52 -0500 (EST) 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 Message-Id: <20020212021252.B89A39F413@okeeffe.bestweb.net> 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 To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-current" in the body of the message