[PATCH] libata: fix translation for START STOP UNIT

[PATCH] libata: fix translation for START STOP UNIT

[Robert Hancock]

[-- Attachment #1: Type: text/plain, Size: 736 bytes --]

Applies to 2.6.20-rc6.

---

libata's SCSI translation for the SCSI START STOP UNIT command with the 
START bit clear (i.e. stopping the drive) appears to be incorrect. It 
sends an ATA STANDBY command with the time period set to 0, which the 
code comment says means "now", but the ATA standard says this means 
disable the standby timer, which effectively does nothing. Change this 
to issue a STANDBY IMMEDIATE command which will actually spin the drive 
down. The SAT (SCSI/ATA Translation) standard revision 9 concurs with 
this choice.

Signed-off-by: Robert Hancock <hancockr@shaw.ca>

-- 
Robert Hancock      Saskatoon, SK, Canada
To email, remove "nospam" from hancockr@nospamshaw.ca
Home Page: http://www.roberthancock.com/



[-- Attachment #2: libata-fix-start-stop-unit-translation.patch --]
[-- Type: text/plain, Size: 688 bytes --]

--- linux-2.6.20-rc6nv/drivers/ata/libata-scsi.c	2007-01-28 16:59:58.000000000 -0600
+++ linux-2.6.20-rc6nvedit/drivers/ata/libata-scsi.c	2007-01-28 17:30:12.000000000 -0600
@@ -983,11 +983,10 @@ static unsigned int ata_scsi_start_stop_
 		}
 
 		tf->command = ATA_CMD_VERIFY;	/* READ VERIFY */
-	} else {
-		tf->nsect = 0;	/* time period value (0 implies now) */
-		tf->command = ATA_CMD_STANDBY;
-		/* Consider: ATA STANDBY IMMEDIATE command */
-	}
+	} else
+		/* Issue ATA STANDBY IMMEDIATE command */
+		tf->command = ATA_CMD_STANDBYNOW1;
+
 	/*
 	 * Standby and Idle condition timers could be implemented but that
 	 * would require libata to implement the Power condition mode page

Re: [PATCH] libata: fix translation for START STOP UNIT

[Jeff Garzik]

Robert Hancock wrote:
> Applies to 2.6.20-rc6.
> 
> ---
> 
> libata's SCSI translation for the SCSI START STOP UNIT command with the 
> START bit clear (i.e. stopping the drive) appears to be incorrect. It 
> sends an ATA STANDBY command with the time period set to 0, which the 
> code comment says means "now", but the ATA standard says this means 
> disable the standby timer, which effectively does nothing. Change this 
> to issue a STANDBY IMMEDIATE command which will actually spin the drive 
> down. The SAT (SCSI/ATA Translation) standard revision 9 concurs with 
> this choice.
> 
> Signed-off-by: Robert Hancock <hancockr@shaw.ca>

FWIW, I am having to apply Andrew Morton's version of your patches, 
because your patch form isn't quite right.  Review 
Documentation/SubmittingPatches and http://linux.yyz.us/patch-format.html

Most notably:

* Your patch description follows the "---" terminator, which is 
precisely the opposite of what should happen.  In the correct form, any 
comments such as "Applies to 2.6.20-rc6" should follow the "---" 
terminator, which in turn follows the patch description and signoff.

* Include the patch inline rather than as an attachment.  Even a 
text/plain attachment is very difficult to review and quote in popular 
email programs.

	Jeff

Re: [PATCH] libata: fix translation for START STOP UNIT

[Robert Hancock]

Jeff Garzik wrote:
> * Include the patch inline rather than as an attachment.  Even a 
> text/plain attachment is very difficult to review and quote in popular 
> email programs.
> 
>     Jeff

I'd love to, but unfortunately nobody seems to have come up with a way 
of doing this in Thunderbird that keeps it from mangling whitespace 
without a ton of hassle. I was able to get it to cooperate once (sort 
of, anyway, I think it may have still damaged something on the first 
try), but it required mangling a bunch of settings that made using it 
for normal mail impossible.

The last time I looked, the main "how-to" page I found had an addendum 
that "I gave up on this, it's too hard, I just attach the patches". If 
anyone has gotten any new insight..

-- 
Robert Hancock      Saskatoon, SK, Canada
To email, remove "nospam" from hancockr@nospamshaw.ca
Home Page: http://www.roberthancock.com/

Re: [PATCH] libata: fix translation for START STOP UNIT

[Jeff Garzik]

Robert Hancock wrote:
> Jeff Garzik wrote:
>> * Include the patch inline rather than as an attachment.  Even a 
>> text/plain attachment is very difficult to review and quote in popular 
>> email programs.
>>
>>     Jeff
> 
> I'd love to, but unfortunately nobody seems to have come up with a way 
> of doing this in Thunderbird that keeps it from mangling whitespace 
> without a ton of hassle. I was able to get it to cooperate once (sort 
> of, anyway, I think it may have still damaged something on the first 
> try), but it required mangling a bunch of settings that made using it 
> for normal mail impossible.
> 
> The last time I looked, the main "how-to" page I found had an addendum 
> that "I gave up on this, it's too hard, I just attach the patches". If 
> anyone has gotten any new insight..

Just use "cat mail | sendmail -t" or git-send-email, Thunderbird will 
never get this stuff right.

I'm fighting Thunderbird right this second, in fact, because it randomly 
decided to stop supporting drag-n-drop to sub-folders in Fedora Core 6 :(

	Jeff

Re: [PATCH] libata: fix translation for START STOP UNIT

[Martin J. Bligh]

Jeff Garzik wrote:
> Robert Hancock wrote:
>> Jeff Garzik wrote:
>>> * Include the patch inline rather than as an attachment.  Even a 
>>> text/plain attachment is very difficult to review and quote in 
>>> popular email programs.
>>>
>>>     Jeff
>>
>> I'd love to, but unfortunately nobody seems to have come up with a way 
>> of doing this in Thunderbird that keeps it from mangling whitespace 
>> without a ton of hassle. I was able to get it to cooperate once (sort 
>> of, anyway, I think it may have still damaged something on the first 
>> try), but it required mangling a bunch of settings that made using it 
>> for normal mail impossible.
>>
>> The last time I looked, the main "how-to" page I found had an addendum 
>> that "I gave up on this, it's too hard, I just attach the patches". If 
>> anyone has gotten any new insight..
> 
> Just use "cat mail | sendmail -t" or git-send-email, Thunderbird will 
> never get this stuff right.
> 
> I'm fighting Thunderbird right this second, in fact, because it randomly 
> decided to stop supporting drag-n-drop to sub-folders in Fedora Core 6 :(

http://mbligh.org/linuxdocs/Email/Clients/Thunderbird

Describes how to fix this.

M.

Re: [PATCH] libata: fix translation for START STOP UNIT

[Mark Lord]

Robert Hancock wrote:
> Jeff Garzik wrote:
>> * Include the patch inline rather than as an attachment. 

..
> 
> I'd love to, but unfortunately nobody seems to have come up with a way 
> of doing this in Thunderbird that keeps it from mangling whitespace 

Yup, major nuisance.  I have to fire up Kmail whenever I'm posting patches,
and then go back to Thunderbird again afterwards. 

Cheers

Re: [PATCH] libata: fix translation for START STOP UNIT

[Tejun Heo]

Mark Lord wrote:
> Robert Hancock wrote:
>> I'd love to, but unfortunately nobody seems to have come up with a way
>> of doing this in Thunderbird that keeps it from mangling whitespace 
> 
> Yup, major nuisance.  I have to fire up Kmail whenever I'm posting patches,
> and then go back to Thunderbird again afterwards.

A much better solution with thunderbird is using external editor
extension.

http://globs.org/articles.php?lng=en&pg=2

Install the extension.  Configure it with your favorite editor.
Thunderbird won't do a thing about your message if you turn off
linewrap.

	 For example, this line starts with a tab and a space and (double tab)		is very long and I'm writing in emacs fired up from thunderbird. (double space)  

Cheers.

-- 
tejun

Re: [PATCH] libata: fix translation for START STOP UNIT

[Mark Lord]

Tejun Heo wrote:
> 
> A much better solution with thunderbird is using external editor
> extension.
> 
> http://globs.org/articles.php?lng=en&pg=2

Brilliant!  Installed, and now tested (the max_cmd_len patch posting).

-ml