Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 126249 - sys-apps/hotplug: /etc/hotplug/scsi.agent needs longer timeout on type attribute
Summary: sys-apps/hotplug: /etc/hotplug/scsi.agent needs longer timeout on type attribute
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Core system (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Greg Kroah-Hartman (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-03-14 19:00 UTC by Robin Johnson
Modified: 2006-03-23 20:52 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Increases timeout to 60 seconds. (scsi.agent-timeout.patch,667 bytes, patch)
2006-03-14 19:05 UTC, Robin Johnson
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Robin Johnson archtester Gentoo Infrastructure gentoo-dev Security 2006-03-14 19:00:42 UTC
On some fibre-channel hardware (my case is a Sun StorEdge A5000 with a Qlogic2100 controller), there is a very long delay between the kernel informing udev that the device is coming online, and the device actually being ready to probe.

The ioerr_cnt check is only part of the problem, as the drive has not yet spun up then even.

In hotplug's scsi.agent, there is a check for the type attribute with a 10 second timeout. It needs to be greatly extended to pick up my hardware.

If not extended, scsi.agent times out, and then scsi_id fails to get data, followed by path_id not running because ID_TYPE is not set - a wonderful cascading failure.

I've produced a patch for this (attached), and I'll apply it in the tree if there is no response by the end of the week.

I did 100 passes of testing, and found that it took an average of 30 seconds to come online, with a worst case of 45 seconds. To have some margin, I've set the timeout 30% higher, 60 seconds.

This change does not cause delays for other hardware, as their type attribute exists long before the existing 10 second timeout anyway.
Comment 1 Robin Johnson archtester Gentoo Infrastructure gentoo-dev Security 2006-03-14 19:05:55 UTC
Created attachment 82182 [details, diff]
Increases timeout to 60 seconds.
Comment 2 Robin Johnson archtester Gentoo Infrastructure gentoo-dev Security 2006-03-23 20:52:57 UTC
commited now after ACK by gregkh.