cvs commit: src/sys/kern kern_proc.c
    Martin Blapp 
    mb at imp.ch
       
    Wed Sep 20 10:24:03 PDT 2006
    
    
  
>>                  mtx_init(&sess->s_mtx, "session", NULL, MTX_DEF);
>>                  PROC_LOCK(p);
>>                  p->p_flag &= ~P_CONTROLT;
>>                  PROC_UNLOCK(p);
>>                  PGRP_LOCK(pgrp);
>>                  sess->s_leader = p;
>>                  sess->s_sid = p->p_pid;
>>                  sess->s_count = 1;
>>                  sess->s_ttyvp = NULL;
>>                  sess->s_ttyp = NULL;
So we need GIANT too after the text 'else' ... What do you think ?
> Well, I'd rather use whatever lock we end up using for t_session instead
> of assuming it's going to be proctree_lock, so I'd like to leave t_session
> only under Giant for now until we really know what we are doing.
Ok. Should I back out tty.c rev. v. 1.258 or just going to work on the tty
lock directly and replace it with whatever lock we use in CURRENT ? I'll only
MFC the other part ...
Martin
    
    
More information about the cvs-all
mailing list