Messages in this thread Patch in this message |  | | | Date | Thu, 27 May 2004 19:49:20 +0400 | | From | Ivan Kokshaysky <> | | Subject | [patch 2.6] don't put IDE disks in standby mode on halt on Alpha |
| |
Spinning the disks down across a 'halt' on Alpha is even worse than doing that on reboot on i386 (assuming the boot device is IDE disk). Typically, the sequence to boot another kernel is: # halt kernel shuts down, firmware re-initializes, then on firmware prompt we type something like >>> boot -file new_kernel_image.gz
Unfortunately, the firmware does not expect the IDE drive to be in standby mode and reports 'bootstrap failure' on the first and all subsequent boot attempts until the drive spins up, which is extremely annoying and confuses users a lot.
This patch allows architectures override the default behavior (don't spin the disks down on reboot only) in asm/ide.h.
Ivan.
--- 2.6/drivers/ide/ide-disk.c Fri May 21 21:06:12 2004 +++ linux/drivers/ide/ide-disk.c Fri May 21 21:48:09 2004 @@ -1723,7 +1723,7 @@ static void ide_device_shutdown(struct d { ide_drive_t *drive = container_of(dev, ide_drive_t, gendev); - if (system_state == SYSTEM_RESTART) { + if (ide_shutdown_omit_standby(system_state)) { ide_cacheflush_p(drive); return; } --- 2.6/include/linux/ide.h Fri May 21 21:47:13 2004 +++ linux/include/linux/ide.h Fri May 21 21:48:09 2004 @@ -1709,4 +1709,8 @@ static inline int ata_can_queue(ide_driv extern struct bus_type ide_bus_type; +#ifndef ide_shutdown_omit_standby +#define ide_shutdown_omit_standby(sys_state) (sys_state == SYSTEM_RESTART) +#endif + #endif /* _IDE_H */ --- 2.6/include/asm-alpha/ide.h Fri May 21 21:47:54 2004 +++ linux/include/asm-alpha/ide.h Fri May 21 21:48:34 2004 @@ -52,6 +52,8 @@ static inline unsigned long ide_default_ #define ide_init_default_irq(base) ide_default_irq(base) #endif +#define ide_shutdown_omit_standby(sys_state) (sys_state != SYSTEM_POWER_OFF) + #include <asm-generic/ide_iops.h> #endif /* __KERNEL__ */ - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
|  |