Bingo! I switched from ata_piix.c to piix_ide.c and the "pop" disappeared. I must say that the "pop" also disappeared after suspending to disk using suspend2 (obviously without executing halt -n -h -p) . In both cases it was present with the previous setup. This is with a pure PATA setup with no SATA ports and the disk is: hda: TOSHIBA MK8025GAS, ATA DISK drive Many thanks! regards, Fabio On 4/19/07, Chuck Ebbert <cebbert@xxxxxxxxxx> wrote:
Stephen Clark wrote: > Mark Lord wrote: > >> Mark Lord wrote: >> >> >>> With the patch applied, I don't see *any* new activity in those >>> S.M.A.R.T. >>> attributes over multiple hibernates (Linux "suspend-to-disk"). >>> >> >> Scratch that -- operator failure. ;) >> The patch makes no difference over hibernates in the SMART logs. >> >> It's still logging extra Power-Off_Retract_Count pegs, >> which it DID NOT USED TO DO not so long ago. >> >> Now I'll poke at the shutdown again. >> >> Meanwhile, Stephen: Could you try without this line in the patched file: >> >> case ATA_16: >> + dev->needs_sync_cache = 1; >> >> Ie. comment out that last "dev->needs_sync_cache" line. >> >> > No joy - even with that line commented out I get a click and my > Power-Off_Retract_Count has incremented. > I found a solution of sorts on Fedora 6: Edit /etc/rc0.d/S01halt, changing the third-from-last line so it reads: HALTARGS="-d -h -n" Then use the halt(8) command to shut down and turn off the power manually. No click and the "retract" counter does not increment. (Even this procedure incremented the counter until I changed the script.) Seems safe, using the IDE drivers, since they sync the cache anyway. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html