Peter Wemm wrote:
> Matthew Dillon wrote:
> > :> - mtx_lock(&Giant);
> > :> - td->td_retval[0] = p->p_ucred->cr_ruid;
> > :> + s = mtx_lock_giant(kern_giant_ucred);
> > :> + td->td_retval[0] = td->td_ucred->cr_ruid;
> > :> #if defined(COMPAT_43) || defined(COMPAT_SUNOS)
> > :> - td->td_retval[1] = p->p_ucred->cr_uid;
> > :> + td->td_retval[1] = td->td_ucred->cr_uid;
> > :> #endif
> > :> - mtx_unlock(&Giant);
> > :> + mtx_unlock_giant(s);
> > :> return (0);
> > :
> > :What a waste.. John has already done all this stuff already (using
> > :td_ucred instead of p_ucred) over the entire tree.
> > :
> > :Cheers,
> > :-Peter
> >
> > He didn't instrument Giant, and if you actually believe that one
> > massive commit is going to be more stable then the piecemeal safe-mode
> > commits I am making then you are smoking something. Or are you
> > expecting John to commit his patchset piecemeal as well and test
> > inbetween? If that is so, then he just wasted a whole lot time
> > managing all this junk in P4 because, frankly, it only took me a few
> > minutes to instrument the easier system calls. I spend far more
> > time testing.
>
> So, John's last few months of work is junk then, is it?
I looked up john's version of this:
@ -228,14 +228,11 @@
struct thread *td;
struct getuid_args *uap;
{
- struct proc *p = td->td_proc;
- mtx_lock(&Giant);
- td->td_retval[0] = p->p_ucred->cr_ruid;
+ td->td_retval[0] = td->td_ucred->cr_ruid;
#if defined(COMPAT_43) || defined(COMPAT_SUNOS)
- td->td_retval[1] = p->p_ucred->cr_uid;
+ td->td_retval[1] = td->td_ucred->cr_uid;
#endif
- mtx_unlock(&Giant);
return (0);
}
That has exactly the same change that you made, with regards to
p_ucred/td_ucred.
Regarding the instrumentation of Giant for *trivial* stuff like this: I'm
one of the people you called "bozos" that disagrees with you about the
usefulness of bloating the source with this stuff that only needs to be
removed again later.
Cheers,
-Peter
--
Peter Wemm - [EMAIL PROTECTED]; [EMAIL PROTECTED]; [EMAIL PROTECTED]
"All of this is for nothing if we don't go to the stars" - JMS/B5
To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message
