Bug 198429 - configurable delay for killproc()
Summary: configurable delay for killproc()
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: initscripts
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Bill Nottingham
QA Contact: Brock Organ
URL:
Whiteboard:
: 187078 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2006-07-11 11:48 UTC by Joe Orton
Modified: 2014-03-17 03:00 UTC (History)
1 user (show)

Fixed In Version: 8.36-1
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-07-20 15:44:03 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
killproc patch (1.01 KB, patch)
2006-07-11 11:48 UTC, Joe Orton
no flags Details | Diff
tab-fixed patch (1.00 KB, patch)
2006-07-11 11:51 UTC, Joe Orton
no flags Details | Diff

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. ***


Note You need to log in before you can comment on or make changes to this bug.