Re: Loud pop coming from hard drive on reboot

2007-04-17 Thread emisca
2007/4/17, Chuck Ebbert [EMAIL PROTECTED]: Jan Engelhardt wrote: On Apr 15 2007 12:53, Henrique de Moraes Holschuh wrote: On Sat, 14 Apr 2007, Pavel Machek wrote: How common are notebooks that cut power to disks during reboot? Assuming it also does this when running Windows, I'd report it as

Re: Loud pop coming from hard drive on reboot

2007-04-17 Thread Fabio Comolli
Hi. On 4/17/07, Chuck Ebbert [EMAIL PROTECTED] wrote: snip (2) Linux (alone) gives a very muted pop on shutdown. This could be from bad interaction with the shutdown command, or some other reason (drive not given enough time to shut down?) The noise is not very loud, maybe the head

Re: [PATCH 01/12] libata: allocate ap separately from shost

2007-04-17 Thread Jeff Garzik
Tejun Heo wrote: Don't embed ap inside shost. Allocate it separately and point it back from shosts's hostdata. This makes port allocation more flexible and allows regular ATA and SAS share host alloc/init paths. Signed-off-by: Tejun Heo [EMAIL PROTECTED] --- drivers/ata/libata-core.c | 19

Re: [PATCH] libata: fix native mode disabled port handling

2007-04-17 Thread Jeff Garzik
Tejun Heo wrote: Disabled port handling in ata_pci_init_native_mode() is slightly broken in that it may end up using the wrong port_info. This patch updates it such that disables ports are made dummy as done in the legacy and other cases. While at it, fix indentation in

Re: [PATCH 1/2] ahci: implement ata_save/restore_initial_config()

2007-04-17 Thread Jeff Garzik
Tejun Heo wrote: There are several registers which describe how the controller is configured. These registers are sometimes implemented as r/w registers which are configured by firmware and get cleared on controller reset or after suspend/resume cycle. ahci saved and restored those values

Re: [PATCH 4/4] libata: implement libata.spindown_compat

2007-04-17 Thread James Bottomley
On Tue, 2007-04-17 at 11:20 -0400, Jeff Garzik wrote: then, unless there are dependencies in libata-dev, how about Tejun pushes patches 3-4 through scsi-misc as well? Sure ... as long as there are no dependencies. James - To unsubscribe from this list: send the line unsubscribe linux-ide in

Re: [PATCH] pci-quirks: disable MSI on RS400-200 and RS480

2007-04-17 Thread Jeff Garzik
Tejun Heo wrote: MSI doesn't work on RS400-200 and RS480 requiring pci=nomsi kernel boot parameter for ahci to work. This patch disables MSI on those chips. http://thread.gmane.org/gmane.linux.ide/17820 http://thread.gmane.org/gmane.linux.ide/17516

Re: [PATCH 4/4] libata: implement libata.spindown_compat

2007-04-17 Thread Jeff Garzik
James Bottomley wrote: On Tue, 2007-04-17 at 10:45 -0400, Jeff Garzik wrote: ACK patches 3-4 for upstream. I'm dropping them, and requesting resend when patches 1-2 are applied (which I also ACK). Or, alternately, I could apply all four patches with the other subsystem owner's assent, or

Re: [PATCH 4/4] libata: implement libata.spindown_compat

2007-04-17 Thread James Bottomley
On Tue, 2007-04-17 at 10:45 -0400, Jeff Garzik wrote: ACK patches 3-4 for upstream. I'm dropping them, and requesting resend when patches 1-2 are applied (which I also ACK). Or, alternately, I could apply all four patches with the other subsystem owner's assent, or the other subsystem

Re: [PATCH] libata: HPA support

2007-04-17 Thread Jeff Garzik
Alan Cox wrote: Signed-off-by: Alan Cox [EMAIL PROTECTED] Add support for ignoring the BIOS HPA result (off by default) and setting the disk to the full available size unless already frozen. Tested with various platforms/disks and confirmed to work with the Macintosh (which broke earlier) and

Re: 2.6.21-rc6-mm1 ATA HPT37x regression

2007-04-17 Thread John Stoffel
Sergei == Sergei Shtylyov [EMAIL PROTECTED] writes: Sergei Hello. Sergei John Stoffel wrote: I was just testing out 2.6.21-rc6-mm1 to test some Cyclades patches and I noticed that my HPT302 (rev1) controller with a pair of 120gb WD disks are not longer detected and I get the following in the

Re: [PATCH] pci-quirks: disable MSI on RS400-200 and RS480

2007-04-17 Thread Greg KH
On Tue, Apr 17, 2007 at 10:57:28AM -0400, Jeff Garzik wrote: Tejun Heo wrote: MSI doesn't work on RS400-200 and RS480 requiring pci=nomsi kernel boot parameter for ahci to work. This patch disables MSI on those chips. http://thread.gmane.org/gmane.linux.ide/17820

Re: [PATCH] pata: cable methods

2007-04-17 Thread Jeff Garzik
Alan Cox wrote: Versus upstream as requested Last of the trivial switches to cable_detect methods. Signed-off-by: Alan Cox [EMAIL PROTECTED] applied - To unsubscribe from this list: send the line unsubscribe linux-ide in the body of a message to [EMAIL PROTECTED] More majordomo info at

Re: [PATCH libata#upstream] sata_promise: fix error decode regression

2007-04-17 Thread Jeff Garzik
Mikael Pettersson wrote: Promise ATA ports should always be reset by pdc_reset_port() when errors are detected, but the recent error reason decoding update to sata_promise replaced that reset with a freeze. This patch changes the error detection to do a reset again. This makes the error

Re: [PATCH] libata: improve AC_ERR_DEV handling for -post_internal_cmd

2007-04-17 Thread Jeff Garzik
Tejun Heo wrote: -post_internal_cmd is simplified EH for internal commands. Its primary mission is to stop the controller such that no rogue memory access or other activities occur after the internal command is released. It may provide error diagnostics by setting qc-err_mask but this hasn't

Re: [PATCH] libata: kill type mismatch compile warning

2007-04-17 Thread Jeff Garzik
Tejun Heo wrote: kill the following compile warning. drivers/ata/libata-core.c:1786: warning: comparison of distinct pointer types lacks a cast Signed-off-by: Tejun Heo [EMAIL PROTECTED] applied - To unsubscribe from this list: send the line unsubscribe linux-ide in the body of a message

Re: exception Emask 0x0 SAct 0x0 SErr 0x0 action 0x2 frozen on SATA hd

2007-04-17 Thread Tejun Heo
Rodrigo Severo wrote: Now with kernel 2.6.21-rc6, addon board reattached and cables changed I still get messages like the following every 10 minutes or so under high load: Apr 16 13:03:27 [kernel] ata1.00: exception Emask 0x0 SAct 0x0 SErr 0x0 action 0x2 frozen Apr 16 13:03:27 [kernel]

Re: [PATCH pata-2.6] cmd64x: procfs code fixes/cleanups (take 2)

2007-04-17 Thread Alan Cox
CMD64x is an established name, while PCI-xxx is something I've never heard of (and it sounds awfully generic). Specs are available from http://gkernel.sourceforge.net/specs/ -- you can see the names on the first pages. End users don't read the specs, they read the box, and they seem

Re: 2.6.21-rc6-mm1 ATA HPT37x regression

2007-04-17 Thread Sergei Shtylyov
Hello. John Stoffel wrote: I was just testing out 2.6.21-rc6-mm1 to test some Cyclades patches and I noticed that my HPT302 (rev1) controller with a pair of 120gb WD disks are not longer detected and I get the following in the dmesg logs: [ 148.121490] hpt37x: DPLL did not stabilize.

Re: [PATCH] ahci: add PCI ID for new VIA chip

2007-04-17 Thread Jeff Garzik
Tejun Heo wrote: Add PCI ID for new VIA chip. Original patch is from Maarten Vanraes. Signed-off-by: Tejun Heo [EMAIL PROTECTED] Cc: Maarten Vanraes [EMAIL PROTECTED] diff --git a/drivers/ata/ahci.c b/drivers/ata/ahci.c index faa7588..3675964 100644 --- a/drivers/ata/ahci.c +++

What's in libata-dev.git?

2007-04-17 Thread Jeff Garzik
I just checked in some useful improvements, so I thought it would be a good time to review what's queued for 2.6.21 in git://git.kernel.org:/pub/scm/linux/kernel/git/jgarzik/libata-dev.git Here is a quick tour of the branches currently available: ALL Meta-branch. Conduit for

Re: 2.6.21-rc6-mm1 ATA HPT37x regression

2007-04-17 Thread Sergei Shtylyov
John Stoffel wrote: I was just testing out 2.6.21-rc6-mm1 to test some Cyclades patches and I noticed that my HPT302 (rev1) controller with a pair of 120gb WD disks are not longer detected and I get the following in the dmesg logs: [ 148.121490] hpt37x: DPLL did not stabilize. Where

Re: [PATCH pata-2.6] cmd64x: procfs code fixes/cleanups (take 2)

2007-04-17 Thread Sergei Shtylyov
Hello. Alan Cox wrote: CMD64x is an established name, while PCI-xxx is something I've never heard of (and it sounds awfully generic). Specs are available from http://gkernel.sourceforge.net/specs/ -- you can see the names on the first pages. End users don't read the specs, they read

Re: [PATCH] pata_hpt3xx: Updates to match work done by Sergei to old IDE

2007-04-17 Thread Jeff Garzik
Alan Cox wrote: Signed-off-by: Alan Cox [EMAIL PROTECTED] Correct timing lists Use DPLL when appropriate Fix HPT372 UDMA 100 limit Add HPT371N support Reset the correct channel only not both when doing the pre_reset processing Pull DPLL clock from I/O not PCI space Correct DPLL failure error

Re: [PATCH] pata_hpt3xx: Updates to match work done by Sergei to old IDE

2007-04-17 Thread Alan Cox
Pull DPLL clock from I/O not PCI space Correct DPLL failure error handling Looks OK to me, but... should I still apply this, given the recent regression reports that hit LKML less than 48 hours ago? Only one report so far but its certainly a good idea to hold off and get to the bottom

Re: [PATCH] pata_winbond

2007-04-17 Thread Jeff Garzik
Alan Cox wrote: Not a PCI device so doesn't need PCI includes Signed-off-by: Alan Cox [EMAIL PROTECTED] applied - To unsubscribe from this list: send the line unsubscribe linux-ide in the body of a message to [EMAIL PROTECTED] More majordomo info at

Re: [PATCH] pata_pdc2027x: Updates

2007-04-17 Thread Jeff Garzik
Alan Cox wrote: Signed-off-by: Alan Cox [EMAIL PROTECTED] Correct missing modefilter (crash if BAR4 unassigned) Use Cable Detect method Wrap -set_mode instead ready for -post_set_mode removal Maxtor errata as per Jeff Garzik report Remove duplicated private udma_mask hacking applied - To

Re: What's in libata-dev.git?

2007-04-17 Thread Jeff Garzik
Jeff Garzik wrote: I just checked in some useful improvements, so I thought it would be a good time to review what's queued for 2.6.21 in git://git.kernel.org:/pub/scm/linux/kernel/git/jgarzik/libata-dev.git Er. Make that queued for 2.6.22. Jeff - To unsubscribe from this list:

Re: [PATCH 4/4] libata: implement libata.spindown_compat

2007-04-17 Thread Tejun Heo
James Bottomley wrote: On Tue, 2007-04-17 at 11:20 -0400, Jeff Garzik wrote: then, unless there are dependencies in libata-dev, how about Tejun pushes patches 3-4 through scsi-misc as well? Sure ... as long as there are no dependencies. James, are you going to rebase scsi-misc-2.6 before

Re: 2.6.21-rc6-mm1 ATA HPT37x regression

2007-04-17 Thread Alan Cox
Ok, so do I need to do anything special with the next -mm release and the next version? Well, let Alan decide that (2Alan: and I said that HPT code is bogus :-). Try drivers/ide/pci/hpt366 - if that works grab a dmesg and let me know. It means that Sergei's DPLL sync code seems to work

Re: exception Emask 0x0 SAct 0x0 SErr 0x0 action 0x2 frozen on SATA hd

2007-04-17 Thread Rodrigo Severo
On 4/17/07, Tejun Heo [EMAIL PROTECTED] wrote: Certainly not. Can you post the result of 'smartctl -d ata -a /dev/sdX'? Here is smartctl -d ata -a /dev/sda for the same HD that presented the previous log messages:

Re: [PATCH 4/4] libata: implement libata.spindown_compat

2007-04-17 Thread James Bottomley
On Wed, 2007-04-18 at 02:09 +0900, Tejun Heo wrote: James, are you going to rebase scsi-misc-2.6 before sending to Linus when 2.6.22-rc1 window opens? If not, Jeff can pull it into libata-dev and patch 3 and 4 can go in there. The difference in libata between scsi-misc-2.6 and libata-dev is

Re: exception Emask 0x0 SAct 0x0 SErr 0x0 action 0x2 frozen on SATA hd

2007-04-17 Thread Tejun Heo
Rodrigo Severo wrote: SMART Error Log Version: 1 No Errors Logged It seemed like a transmission problem so I was hoping the drive's SMART log recorded something. Apparently not. :-( This is the SATA board running now. Everything else is exactly the same. I got no syslog messages after

Re: exception Emask 0x0 SAct 0x0 SErr 0x0 action 0x2 frozen on SATA hd

2007-04-17 Thread Rodrigo Severo
On 4/17/07, Tejun Heo [EMAIL PROTECTED] wrote: Rodrigo Severo wrote: Please let me know if there is anything else I can do to help debug this issue. I can put the ULi M5281/M5283 back for tests. ULi SATA is a fairly standard controller and the driver doesn't do much other than feeding IO

Re: [PATCH pata-2.6] cmd64x: procfs code fixes/cleanups (take 2)

2007-04-17 Thread Mark Lord
Sergei Shtylyov wrote: Mark Lord wrote: - correct the chipset names (from CMDxxx to PCI-xxx) Please explain why this rename is a correction. Because the chips are officially named PCI064[036] and PCI-64[89]. We normally name things with a combination of brand and chip number. Lots of

Re: [PATCH pata-2.6] cmd64x: procfs code fixes/cleanups (take 2)

2007-04-17 Thread Sergei Shtylyov
Mark Lord wrote: - correct the chipset names (from CMDxxx to PCI-xxx) Please explain why this rename is a correction. Because the chips are officially named PCI064[036] and PCI-64[89]. We normally name things with a combination of brand and chip number. Lots of companies could have

Re: exception Emask 0x0 SAct 0x0 SErr 0x0 action 0x2 frozen on SATA hd

2007-04-17 Thread Tejun Heo
Rodrigo Severo wrote: I have another ULi M5281/M5283 at home. I can bring it to work and try to reproduce this issue. Interested? Yeap, that will tell us whether the specific card is busted or not. BTW lspci mentions a M5281 but the chipset is marked M5283. Is this ok? Yeap, that's okay. --

Re: [PATCH] libata: HPA support

2007-04-17 Thread Dave Jones
On Wed, Apr 11, 2007 at 12:23:13AM +0100, Alan Cox wrote: Signed-off-by: Alan Cox [EMAIL PROTECTED] Add support for ignoring the BIOS HPA result (off by default) and setting the disk to the full available size unless already frozen. Tested with various platforms/disks and confirmed

Re: exception Emask 0x0 SAct 0x0 SErr 0x0 action 0x2 frozen on SATA hd

2007-04-17 Thread Rodrigo Severo
On 4/17/07, Tejun Heo [EMAIL PROTECTED] wrote: Rodrigo Severo wrote: I have another ULi M5281/M5283 at home. I can bring it to work and try to reproduce this issue. Interested? Yeap, that will tell us whether the specific card is busted or not. I expect to have news tomorrow. Thanks again

Re: [PATCH] libata: HPA support

2007-04-17 Thread Jeff Garzik
Dave Jones wrote: On Wed, Apr 11, 2007 at 12:23:13AM +0100, Alan Cox wrote: Signed-off-by: Alan Cox [EMAIL PROTECTED] Add support for ignoring the BIOS HPA result (off by default) and setting the disk to the full available size unless already frozen. Tested with various

Re: 2.6.21-rc6-mm1 ATA HPT37x regression

2007-04-17 Thread John Stoffel
Ok, so do I need to do anything special with the next -mm release and the next version? Well, let Alan decide that (2Alan: and I said that HPT code is bogus :-). Alan Try drivers/ide/pci/hpt366 - if that works grab a dmesg and let Alan me know. It means that Sergei's DPLL sync code seems

Re: [PATCH] libata: HPA support

2007-04-17 Thread Dave Jones
On Tue, Apr 17, 2007 at 03:15:23PM -0400, Jeff Garzik wrote: Dave Jones wrote: On Wed, Apr 11, 2007 at 12:23:13AM +0100, Alan Cox wrote: Signed-off-by: Alan Cox [EMAIL PROTECTED] Add support for ignoring the BIOS HPA result (off by default) and setting the disk to the

Re: 2.6.21-rc6-mm1 ATA HPT37x regression

2007-04-17 Thread Sergei Shtylyov
Hello. John Stoffel wrote: Ok, so do I need to do anything special with the next -mm release and the next version? Well, let Alan decide that (2Alan: and I said that HPT code is bogus :-). Alan Try drivers/ide/pci/hpt366 - if that works grab a dmesg and let Alan me know. It means that

Re: 2.6.21-rc6-mm1 ATA HPT37x regression

2007-04-17 Thread Mark Lord
Sergei Shtylyov wrote: HPT chips are surely not a good example of how to do things, more like an example how *not* to do. :-) That explains a lot! :) (guess who's first ATA work was on the Highpoint driver.. .. no need to post the answer here, though) Cheers - To unsubscribe from this