Bug List: (This bug is not in your last search results)   Show last search results      Search page      Enter new bug
Bug#: 16848
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: Martin Schlemmer (RETIRED) <azarah@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Arno <gentoo@onra.org>
Add CC:
CC:
Remove selected CCs
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
hdparm /etc/init.d/hdparm -- ATARAID changes. text/plain Justin Jones 2003-03-10 02:42 0000 2.12 KB Details
hdparm hdparm rc-script -- grep ide (changes proposed by drobbins) text/plain Justin Jones 2003-03-30 21:41 0000 1.74 KB Details
hdparm hdparm rc-script v2 -- changes proposed by drobbins text/plain Justin Jones 2003-04-08 01:46 0000 1.87 KB Details
hdparm hdparm rc-script v3.. more changes asked for by drobbins :> text/plain Justin Jones 2003-04-11 08:44 0000 2.21 KB Details
hdparm v4.. Sorry, v3 had a small error =\ text/plain Justin Jones 2003-04-11 23:20 0000 2.19 KB Details
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 16848 depends on: Show dependency tree
Bug 16848 blocks:
Votes: 0    Show votes for this bug    Vote for this bug

Additional Comments: (this is where you put emerge --info)


Not eligible to see or edit group visibility for this bug.






View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug

Description:   Opened: 2003-03-04 14:08 0000 
hdparm 5.3-r2
The /etc/init.d/hdparm should skip ataraid discs, as these are RAID arrays.
I.E. on my machine, hdparm init scripts tries to apply settings on these discs:
ls -l /dev/discs/
total 0
...  disc0 -> ../ide/host2/bus0/target0/lun0
...  disc1 -> ../ide/host2/bus1/target0/lun0
...  disc2 -> ../ataraid/disc0
hdparm init script fails on disc2.
I guess it could be easy to add a grep for ataraid in the function checkconfig().

Thanks,
-- Arno.


Reproducible: Always
Steps to Reproduce:
1.
2.
3.

------- Comment #1 From Seth Chandler 2003-03-07 11:49:06 0000 ------- 
nagash you wanna pick this one up and run with it?

------- Comment #2 From Justin Jones 2003-03-09 19:22:56 0000 ------- 
Yeh sure. Sorry for my tardiness. I kept asking you on IRC. :>

------- Comment #3 From Justin Jones 2003-03-10 02:42:46 0000 ------- 
Created an attachment (id=9193) [details]
/etc/init.d/hdparm -- ATARAID changes.

Works fine, maybe some aesthetic issues seth, I'll leave that for you to
decide.
(It says "Skipping cdrom1..." if $all_args is specified OR $cdrom1_args is
specified.)

I thought it might be a good idea to let the user know that hdparm isn't being
run on that drive, otherwise people might assume it is (when using $all_args)

------- Comment #4 From Daniel Robbins (RETIRED) 2003-03-27 13:15:02 0000 ------- 
OK, this is in 5.3-r2

------- Comment #5 From Daniel Robbins (RETIRED) 2003-03-27 13:17:09 0000 ------- 
oops. OK, why doesn't the hdparm script just scan for "ide" in the /dev/discs/
directory rather than scanning for "scsi" and *not* handling those?

------- Comment #6 From Justin Jones 2003-03-30 03:29:52 0000 ------- 
No reason I guess, besides I just didnt think of it :)

------- Comment #7 From Justin Jones 2003-03-30 03:31:47 0000 ------- 
I'll change it and upload tomorrow.

------- Comment #8 From Justin Jones 2003-03-30 21:41:00 0000 ------- 
Created an attachment (id=10051) [details]
hdparm rc-script -- grep ide (changes proposed by drobbins)

All done, works fine on my machine.

------- Comment #9 From Justin Jones 2003-04-08 01:46:15 0000 ------- 
Created an attachment (id=10362) [details]
hdparm rc-script v2 -- changes proposed by drobbins

Another version, this one uses an ewarn to inform the user that hdparm will not
be a run on a device (when it is found not to be an IDE device)

Eg.
v1 Output..
 * Starting hdparm...
 * Running hdparm on disc0...					[ ok ]
 * Running hdparm on disc1...					[ ok ]

v2 Output..
 * Starting hdparm...
 * Skipping cdrom0, hdparm only supports IDE devices!
 * Running hdparm on disc0...					[ ok ]
 * Running hdparm on disc1...					[ ok ]


No other changes from v1.

------- Comment #10 From Justin Jones 2003-04-11 08:44:05 0000 ------- 
Created an attachment (id=10500) [details]
hdparm rc-script v3.. more changes asked for by drobbins :>

IMHO this one looks like an ugly hack :>

I'm not a bash genius, so I'm sure theres many easier ways to do it, but ah
well, here it is.

------- Comment #11 From Justin Jones 2003-04-11 23:20:57 0000 ------- 
Created an attachment (id=10534) [details]
v4.. Sorry, v3 had a small error =\

------- Comment #12 From Heinrich Wendel (RETIRED) 2004-04-06 04:42:49 0000 ------- 
can you take a look at bug 37755 if that fixes your problem

------- Comment #13 From Heinrich Wendel (RETIRED) 2004-08-01 07:43:13 0000 ------- 
fixed with bug #37755
Bug List: (This bug is not in your last search results)   Show last search results      Search page      Enter new bug