Bug 118049 - loader utility in anaconda does not parse some ethtool settings correctly
loader utility in anaconda does not parse some ethtool settings correctly
Status: CLOSED RAWHIDE
Product: Red Hat Enterprise Linux 3
Classification: Red Hat
Component: anaconda (Show other bugs)
3.0
All Linux
medium Severity medium
: ---
: ---
Assigned To: Jeremy Katz
Mike McLean
:
: 116618 (view as bug list)
Depends On:
Blocks: 116727
  Show dependency treegraph
 
Reported: 2004-03-11 09:14 EST by Neil Horman
Modified: 2007-11-30 17:07 EST (History)
2 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2004-05-06 17:38:08 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)
patch to correct speed parsing (760 bytes, patch)
2004-03-11 09:18 EST, Neil Horman
no flags Details | Diff

  None (edit)
Description Neil Horman 2004-03-11 09:14:04 EST
Description of problem:
when setting ethtool parameters in kickstart on on kernel command line
during install, making the first parameter "speed 10" fails, as loader
does not parse this correctly

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

How reproducible:
always

Steps to Reproduce:
1.insert 3AS-U1 cd into system
2.when boot prompt is presented enter the following line:
ethtool="speed 10 duplex full"
3.continue system boot
4.when prompted select NFS/network install
5.select a network interface to install over
6.select dhcp for ip assignment
7.switch to vt3 and examine logs
  
Actual results:
vt3 dsplays message indicating "Unknown speed 10 duplex half"

Expected results:
speed settings should be successfully driven to network interface

Additional info:
Comment 1 Neil Horman 2004-03-11 09:18:25 EST
Created attachment 98459 [details]
patch to correct speed parsing

This problem is being caused by an incorrect value in the strncmp line that
parses speed in the loader utility.  If speed is the first parameter in the
ethtool section of the kernel command line and is set to 10, then this strncmp
will fail becuase it compares 4 characters rather than 2, and the remaining
characters in the string don't match up with the "10" in the binaries string
table.	Interestingly if speed is the last paramter provided, the null
termiators match up and the strncmp fails, allowing for an easy in the field
workaround (just make sure speed is the last parameter).  This patch corrects
the strncmp value and allows speed and duplex to be specified in an arbitrary
order from the boot prompt.
Comment 2 Jeremy Katz 2004-05-06 17:38:08 EDT
Fixed in CVS for both HEAD and taroon-branch.  I got a little bit
carried away with my cutting and pasting.
Comment 3 Robert Perkins 2004-05-11 16:06:42 EDT
*** Bug 116618 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.