Bug 439898 - module load option to enable entropy generation from e1000,bnx2 network cards
Summary: module load option to enable entropy generation from e1000,bnx2 network cards
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: kernel
Version: 5.4
Hardware: All
OS: Linux
low
low
Target Milestone: rc
: 5.4
Assignee: Ivan Vecera
QA Contact: Red Hat Kernel QE team
URL:
Whiteboard:
Depends On:
Blocks: 450783 483784 502011 502021
TreeView+ depends on / blocked
 
Reported: 2008-03-31 22:02 UTC by Greg Marsden
Modified: 2018-10-19 19:36 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Enhancement
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-09-02 08:58:05 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
patch to provide module option for entropy generation (1.94 KB, patch)
2008-03-31 22:02 UTC, Greg Marsden
no flags Details | Diff
e1000 entropy enable module option patch (1.50 KB, patch)
2008-03-31 22:02 UTC, Greg Marsden
no flags Details | Diff
Proposed patch (1.24 KB, patch)
2008-12-19 10:23 UTC, Ivan Vecera
no flags Details | Diff
Final patch sent to review (2.20 KB, patch)
2008-12-19 11:24 UTC, Ivan Vecera
no flags Details | Diff


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHSA-2009:1243 0 normal SHIPPED_LIVE Important: Red Hat Enterprise Linux 5.4 kernel security and bug fix update 2009-09-01 08:53:34 UTC

Description Greg Marsden 2008-03-31 22:02:10 UTC
Description of problem:

Applications requesting random data from /dev/random either hang, or print
out the following information message:

  Not enough random bytes available.  Please do some other work to give
  the OS a chance to collect more entropy!

Analysis:

In OEL4 and RHEL4, the e1000 module was contributing entropy to the system
by passing the IRQF_SAMPLE_RANDOM during request_irq() calls.  

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


How reproducible:
On a diskless, keyboardless system, reading from /dev/random will hang forever
if system does not have hardware random support.



Additional info:
Patches attached to create add a load-time option for e1000 and bnx2 (default
does not change existing behavior)

Comment 1 Greg Marsden 2008-03-31 22:02:10 UTC
Created attachment 299775 [details]
patch to provide module option for entropy generation

Comment 2 Greg Marsden 2008-03-31 22:02:59 UTC
Created attachment 299776 [details]
e1000 entropy enable module option patch

Comment 3 Ivan Vecera 2008-06-13 18:56:32 UTC
This functionality was present in upstream but was removed by this commit:
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=c0bc8721b8d0380ec69fa97578c91201201b05a9
The reason for removal is not known for me... maybe the security (see bug
#439920). As for the proposed patch it would be necessary to push it upstream
but the question is if is it acceptable by upstream...

Comment 9 Ivan Vecera 2008-12-09 15:47:11 UTC
I prepared a testing kernel packages. Could you please test them? e1000 has entropy enabled (without using module param) and bnx2 has module parameter to enable entropy generation.

Comment 10 Ivan Vecera 2008-12-09 15:48:08 UTC
[addition to Comment #9]
The packages are available at:
http://people.redhat.com/ivecera/rhel-5-ivtest/

Comment 12 Martin Wilck 2008-12-15 11:14:49 UTC
But only e1000 and bnx2 are currently changed, correct? Any similar plans for igb and e1000e?

Comment 13 Ivan Vecera 2008-12-19 10:23:29 UTC
Created attachment 327424 [details]
Proposed patch

Comment 15 Ivan Vecera 2008-12-19 11:24:23 UTC
Created attachment 327433 [details]
Final patch sent to review

Comment 16 RHEL Program Management 2009-02-16 15:42:19 UTC
Updating PM score.

Comment 17 Don Zickus 2009-03-09 18:54:12 UTC
in kernel-2.6.18-134.el5
You can download this test kernel from http://people.redhat.com/dzickus/el5

Please do NOT transition this bugzilla state to VERIFIED until our QE team
has sent specific instructions indicating when to do so.  However feel free
to provide a comment indicating that this fix has been verified.

Comment 20 Chris Ward 2009-07-03 18:01:53 UTC
~~ Attention - RHEL 5.4 Beta Released! ~~

RHEL 5.4 Beta has been released! There should be a fix present in the Beta release that addresses this particular request. Please test and report back results here, at your earliest convenience. RHEL 5.4 General Availability release is just around the corner!

If you encounter any issues while testing Beta, please describe the issues you have encountered and set the bug into NEED_INFO. If you encounter new issues, please clone this bug to open a new issue and request it be reviewed for inclusion in RHEL 5.4 or a later update, if it is not of urgent severity.

Please do not flip the bug status to VERIFIED. Only post your verification results, and if available, update Verified field with the appropriate value.

Questions can be posted to this bug or your customer or partner representative.

Comment 21 Chris Ward 2009-07-10 19:04:11 UTC
~~ Attention Partners - RHEL 5.4 Snapshot 1 Released! ~~

RHEL 5.4 Snapshot 1 has been released on partners.redhat.com. If you have already reported your test results, you can safely ignore this request. Otherwise, please notice that there should be a fix available now that addresses this particular request. Please test and report back your results here, at your earliest convenience. The RHEL 5.4 exception freeze is quickly approaching.

If you encounter any issues while testing Beta, please describe the issues you have encountered and set the bug into NEED_INFO. If you encounter new issues, please clone this bug to open a new issue and request it be reviewed for inclusion in RHEL 5.4 or a later update, if it is not of urgent severity.

Do not flip the bug status to VERIFIED. Instead, please set your Partner ID in the Verified field above if you have successfully verified the resolution of this issue. 

Further questions can be directed to your Red Hat Partner Manager or other appropriate customer representative.

Comment 23 Chris Ward 2009-08-07 08:01:10 UTC
Oracle, 

Could you report back on the resolution of this issue in the RHEL 5.4 Beta bits available via RHN Beta Channel? Thanks.

Comment 24 Greg Marsden 2009-08-07 22:41:59 UTC
Thanks for including this patch.

Comment 26 Allen May 2009-08-14 18:32:38 UTC
Preliminary testing on a headless/keyboardless/mouseless system that was having trouble restarting services that use entropy is showing this working perfectly with the kernel-2.6.18-162.el5.i686.rpm build.

Anything I can do to help get this tested and verified for 5.4?

Allen

Comment 27 Martin Wilck 2009-08-24 08:57:15 UTC
It's looking good for bnx2, thanks. 

Side note: I had hoped for a more consistent treatment of the behavior across network drivers, but thanks anyway for adding the option to bnx2.

Comment 28 errata-xmlrpc 2009-09-02 08:58:05 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHSA-2009-1243.html


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