Bug 219758 - init.d/halt script unmount takes too long due to sleeps
Summary: init.d/halt script unmount takes too long due to sleeps
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Red Hat Enterprise Linux 4
Classification: Red Hat
Component: initscripts
Version: 4.4
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
: ---
Assignee: Bill Nottingham
QA Contact: Brock Organ
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2006-12-15 07:11 UTC by Tamas Kerecsen
Modified: 2014-03-17 03:04 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-12-15 16:32:04 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
A patch illustrating one way to make the halt sctipt run faster without compromising functionality (1.95 KB, patch)
2006-12-15 07:11 UTC, Tamas Kerecsen
no flags Details | Diff

Description Tamas Kerecsen 2006-12-15 07:11:56 UTC
Description of problem:
At two places, the init.d halt script has a 2 second sleep after unmounting
certain paths and before checking whether the action was successful. Under
typical conditions, the 2 second sleep is completely unnecessary. Collectively
these two sleeps slow down the shutdown process by almost 4 seconds.

Version-Release number of selected component (if applicable):
N/A

How reproducible:

Steps to Reproduce:
1. Execute /sbin/halt and measure time. 
2. Apply attached patch, run /sbin/halt again and measure time. 
3. Shutdown will be 3-4 seconds faster with patch.
  
Actual results:
See above

Expected results:
See above

Additional info:
On most systems even the 0.1 sec sleep I included is unnecessary, so feel free
to remove it.

The patch also comments out the "devremaining" variable assignments that look
like they have been left behind accidentally.

Comment 1 Tamas Kerecsen 2006-12-15 07:11:57 UTC
Created attachment 143738 [details]
A patch illustrating one way to make the halt sctipt run faster without compromising functionality

Comment 2 Bill Nottingham 2006-12-15 16:32:04 UTC
I've cloned this issue for the Fedora Core development tree; it can be tracked
there as bug 219816, as that's where we'll tackle it for future releases.

Given that this does not affect the actual functionality of the script, this is
something we're unlikely to tackle in a update for Red Hat Enterprise Linux 4 in
the future - we're concentrating more on bug fixes and regressions. We'll try
and get this fixed for future releases of Red Hat Enterprise Linux. Closing as
WONTFIX for RHEL4.


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