Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 59388 - hdparm-5.5-r2 initscript reports current drive parameters when using more than one parameter for a drive
Summary: hdparm-5.5-r2 initscript reports current drive parameters when using more tha...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Core system (show other bugs)
Hardware: x86 Linux
: High minor (vote)
Assignee: Heinrich Wendel (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-08-04 05:30 UTC by Lars Geiger
Modified: 2004-08-04 07:02 UTC (History)
0 users

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


Attachments
Change for the initscript to fix the bug (hdparm.diff,424 bytes, patch)
2004-08-04 05:34 UTC, Lars Geiger
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Lars Geiger 2004-08-04 05:30:52 UTC
The hdparm initscripts calls 'hdparm -q $args $device' which causes hdparm to output a drive's current config if $args contains more than one argument (because -q just supresses the output of the one parameter immediately following the -q).

Reproducible: Always
Steps to Reproduce:
1. emerge "=hdparm-5.5-r2"
2. Edit /etc/conf.d/hdparm to contain something like 'disc0_args="-A1 -a64 -c3 -d1 -m16 -S120 -u1"'
3. '/etc/init.d/hdparm start'
Actual Results:  
The settings for disc0 were applied and reported:

 * Running hdparm on /dev/discs/disc0/disc...
 setting fs readahead to 64
 setting 32-bit IO_support flag to 3
 setting multcount to 16
 setting unmaskirq to 1 (on)
 setting using_dma to 1 (on)
 multcount    = 16 (on)
 IO_support   =  3 (32-bit w/sync)
 unmaskirq    =  1 (on)
 using_dma    =  1 (on)
 readahead    = 64 (on)


Expected Results:  
The settings should have been applied silently, just like they are when only a
single argument like 'disc0_args="-d1"' is present.
Comment 1 Lars Geiger 2004-08-04 05:34:19 UTC
Created attachment 36742 [details, diff]
Change for the initscript to fix the bug

Changed the hdparm call to not use -q (which only works for the first
parameter) and redirected output of hdparm to /dev/null.
Comment 2 Heinrich Wendel (RETIRED) gentoo-dev 2004-08-04 07:02:12 UTC
thx for pointing that out, fixed in -r3 :)