From owner-freebsd-current  Mon Feb 11 19: 8: 0 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 6E4BC37B496
	for <current@freebsd.org>; Mon, 11 Feb 2002 18:17:11 -0800 (PST)
Received: from okeeffe.bestweb.net (okeefe.bestweb.net [209.94.100.110])
	by newman2.bestweb.net (Postfix) with ESMTP
	id AF3A5231EC; Mon, 11 Feb 2002 21:16:50 -0500 (EST)
Received: by okeeffe.bestweb.net (Postfix, from userid 0)
	id B41B09F2AC; Mon, 11 Feb 2002 21:11:55 -0500 (EST)
Date: Thu, 7 Feb 2002 19:17:53 -0800 (PST)
From: Julian Elischer <julian@elischer.org>
To: Terry Lambert <tlambert2@mindspring.com>
Cc: current@freebsd.org
Subject: Re: ucred for threads
Message-Id: <20020212021155.B41B09F2AC@okeeffe.bestweb.net>
Sender: owner-freebsd-current@FreeBSD.ORG
Precedence: bulk
List-ID: <freebsd-current.FreeBSD.ORG>
List-Archive: <http://docs.freebsd.org/mail/> (Web Archive)
List-Help: <mailto:majordomo@FreeBSD.ORG?subject=help> (List Instructions)
List-Subscribe: <mailto:majordomo@FreeBSD.ORG?subject=subscribe%20freebsd-current>
List-Unsubscribe: <mailto:majordomo@FreeBSD.ORG?subject=unsubscribe%20freebsd-current>
X-Loop: FreeBSD.ORG

Terry I don't have time to work on it now, but if you would like to take a
-current system
 and apply the threading KSE diffs from my web page, 
then maybe you can investigate whether I really ned the  if clause or
not..
I don't have time now to go back to it for a while


On Thu, 7 Feb 2002, Terry Lambert wrote:

> Julian Elischer wrote:
> > yes.. well in booting yuo can have a null ucred. ( I know,s
> > I've hit it), but in general you are correct.
> 
> [ ... ]
> 
> > > What is the default state of td->td_ucred?
> > 
> > on creation, NULL followed very rapidly with being set to
> > p->p_ucred. (via crhold)
> 
> Non-problem, then.
> 
> 
> > > If that's the case, then the code should be:
> > >
> > >         if (td->td_ucred != p->p_ucred) {
> > >                 PROC_LOCK(p);
> > >                 if (td->td_ucred) {
> > >                         crfree(td->td_ucred);
> > >                 }
> > 
> > without the if it crashes in boot sometimes.
> > (this may not be true right now but was during my testing of
> > the KSE kernel)
> 
> The place to fix this is by setting up a default reference to
> a root/boot ucred, I think, for use by the initial process
> template.
> 
> What are the consequences, if any, of me having removed the
> setting the thing to NULL, during boot?  I guess that it
> would leave the thread cred uninitialized.  Obviously, the
> problem with your crash is in the crhold( NULL).  8-).
> 
> It seems to me that the test would leave threads with NULL
> ucreds around as well, and just complicate things later.
> 
> Is there a reason you can't set up an initial ucred?
> 
> -- Terry
> 
> 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

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-current" in the body of the message