Bug 198429

Summary: configurable delay for killproc()
Product: [Fedora] Fedora Reporter: Joe Orton <jorton>
Component: initscriptsAssignee: Bill Nottingham <notting>
Status: CLOSED RAWHIDE QA Contact: Brock Organ <borgan>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: rvokal
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: 8.36-1 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2006-07-20 15:44:03 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
killproc patch
none
tab-fixed patch none

Description Joe Orton 2006-07-11 11:48:28 UTC 
Description of problem:
I'll attach a patch which makes the delay between doing a SIGTERM and a SIGKILL
in killproc() configurable.

This can be used to fix a race condition in httpd shutdown.  The httpd parent
will SIGTERM then SIGKILL children at shutdown in a similar way to how
killproc() will SIGTERM and then SIGKILL the httpd parent.  But these two race
and sometimes the parent will get SIGKILLed before it manages to kill all its
children.  This can leave orphans.

So making the killproc() delay configurable means we can ensure that httpd
always wins this race and cleans up its children before the parent gets cleaned up.

See also bug 184403.
Comment 1 Joe Orton 2006-07-11 11:48:28 UTC 
Created attachment 132229 [details]
killproc patch
Comment 2 Joe Orton 2006-07-11 11:51:53 UTC 
Created attachment 132230 [details]
tab-fixed patch

replacement patch which uses tabs for whitespace
Comment 3 Bill Nottingham 2006-07-20 15:44:03 UTC 
Added in CVS, will be in 8.36-1. Thanks!
Comment 4 Miloslav Trmač 2006-07-29 00:04:52 UTC 
*** Bug 187078 has been marked as a duplicate of this bug. ***
Comment 5 Joe Orton 2006-07-30 17:03:16 UTC 
*** Bug 187078 has been marked as a duplicate of this bug. ***