On Mon, 2011-10-17 at 11:34 -0400, Zdenek Pavlas wrote:
> >  Where do we get zombies of extended lifetime (roll a d20 ;) from?
> 
> a mysterious, incurable virus has come..

 :)

> >  How good/compat. is that info. from stat vs. taking the line from
> > status?
> 
> Not sure, I'd prefer status too, but utils.get_process_info()
> uses 'stat' to get the process status.

 Fair enough then.

> >  Also, at least put this behind a "locker_alive" API call or
> >  something.
> 
> Is an option.. I'd then merge the 'kill(PID, 0)' into that.

 Sure, sounds great.

> What do you think of commit e177294f5094 ?  I think we should
> revert that.  doLock() already handles this.

 That patch drops the timeout down a lot when we can't lock, and we
can't see the lock owner (presumably because it's exiting/exited).
 I guess this is triggered by zombies? If so that problem should go away
if we ignore zombies, right?
 Changing the 0.1 to 0.5 or just always doing a 2 second timeout doesn't
seem bad.

_______________________________________________
Yum-devel mailing list
Yum-devel@lists.baseurl.org
http://lists.baseurl.org/mailman/listinfo/yum-devel

Reply via email to