Date: Sat, 12 Apr 2003 14:39:29 -0700 From: "Cliff L. Biffle" <cbiffle@safety.net> To: freebsd-mobile@freebsd.org Subject: [PATCH] ATA disk spin-down for -current Message-ID: <200304121439.29781.cbiffle@safety.net>
next in thread | raw e-mail | index | archive | help
Allo all. I've ported the 4.x patch that enabled ATA disk spin-down to the 5.x kernel. (It looks like it'd also work on recent 4.x.) Since I got the original patch out of this list's archives, I thought I'd send it back. Basically this just updates the ata_command calls to the new LBA format rather than the older C/H/S. I'm relatively unfamiliar with the ATA subsystem, and I -think- putting the suspend call after ATA_UNLOCK_CH is correct. If anyone knows better, let me know. Anyway, I've been using this on my laptop with great success. Suspend time is set (in seconds) in hw.ata.suspend; this must be done at boot in loader.conf, as it's read-only at runtime. Comments? Corrections? -Cliff L. Biffle --- src/sys/dev/ata/ata-disk.c.orig Fri Apr 11 12:50:37 2003 +++ src/sys/dev/ata/ata-disk.c Fri Apr 11 13:05:54 2003 @@ -70,9 +70,11 @@ static int ata_dma = 1; static int ata_wc = 1; static int ata_tags = 0; +static int ata_suspend = 0; TUNABLE_INT("hw.ata.ata_dma", &ata_dma); TUNABLE_INT("hw.ata.wc", &ata_wc); TUNABLE_INT("hw.ata.tags", &ata_tags); +TUNABLE_INT("hw.ata.suspend", &ata_suspend); static MALLOC_DEFINE(M_AD, "AD driver", "ATA disk driver"); /* sysctl vars */ @@ -83,6 +85,8 @@ "ATA disk write caching"); SYSCTL_INT(_hw_ata, OID_AUTO, tags, CTLFLAG_RD, &ata_tags, 0, "ATA disk tagged queuing support"); +SYSCTL_INT(_hw_ata, OID_AUTO, suspend, CTLFLAG_RD, &ata_suspend, 0, + "ATA disk suspend timer"); void ad_attach(struct ata_device *atadev) @@ -185,6 +189,12 @@ #endif ATA_UNLOCK_CH(atadev->channel); + if(ata_suspend > 0) { + /* attempt suspend mode. The drive uses increments of ten seconds */ + if(ata_command(atadev, 0xe2, 0, ata_suspend/10, 0, ATA_WAIT_INTR)) + ata_prtdev(atadev, "suspend mode failed."); + } + adp->disk.d_open = adopen; adp->disk.d_close = adclose; adp->disk.d_strategy = adstrategy; @@ -856,6 +866,11 @@ ata_command(atadev, ATA_C_SET_MULTI, 0, adp->transfersize / DEV_BSIZE, 0, ATA_WAIT_READY); atadev->setmode(atadev, adp->device->mode); + if(ata_suspend > 0) { + /* attempt suspend mode. The drive uses increments of ten seconds */ + if(ata_command(atadev, 0xe2, 0, ata_suspend/10, 0, ATA_WAIT_READY)) + ata_prtdev(atadev, "suspend mode failed."); + } } void
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200304121439.29781.cbiffle>