linux-ide.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Mark Lord <liml@rtr.ca>
To: matthieu castet <castet.matthieu@free.fr>
Cc: Tejun Heo <tj@kernel.org>, David Greaves <david@dgreaves.com>,
	linux-ide@vger.kernel.org,
	Smartmontools Mailing List
	<smartmontools-support@lists.sourceforge.net>,
	linux-kernel@vger.kernel.org
Subject: Re: [smartmontools-support] SMART to USB/sata dock in 2.6.27-rc5 doesn't work
Date: Thu, 23 Oct 2008 16:36:33 -0400	[thread overview]
Message-ID: <4900E051.9040608@rtr.ca> (raw)
In-Reply-To: <4900DCE4.3020901@rtr.ca>

Mark Lord wrote:
> matthieu castet wrote:
>> Mark Lord wrote:
>>> matthieu castet wrote:
>>>> Hi,
>>>>
>>>> Tejun Heo wrote:
>>>>> David Greaves wrote:
>>>>>> Error Counter logging not supported
>>>>>> Device does not support Self Test logging
>>>>>
>>>>> The USB bridge should support SAT (SCSI-ATA translation) and many
>>>>> don't.  Nothing much can be done about it.  The hardware just don't 
>>>>> know
>>>>> how to do it.
>>>>>
>>>> That true that the hardware should support SAT, but even if it 
>>>> support it, linux need some patches to support SAT over USB. See 
>>>> http://thread.gmane.org/gmane.linux.scsi/43688 and the patch 
>>>> mentioned in the thread.
>>> ...
>>>
>>> Ahh.. Great!  This mostly fixes smartctl operation on at least two
>>> of my USB drive enclosures here:
>>>
>>>    1. a recent rev.  2.5" NexStar-3 SATA enclosure.
>>>    2. a 250GB 2.5" WD "Passport" external USB drive/enclosure.
>>>
>>> With the patch, this command line works for me:
>>>
>>>    smartctl -a -dsat -T permissive /dev/sdb
>>> And hdparm just works regardless, with or without the patch.
>>>
>>>
>> It should work without permissive, permissive was need without the patch.
>>
>> You said hdparm works without the patch. I wonder how it work. IRRC it 
>> asks check condition stuff for every command. I am interested to see 
>> the hdparm --verbose output without the patch.
> ..
> 
> Mmm... actually, the "-dsat" flag seems to make smartctl mostly work
> with and without the patch.  This is on 2.6.25.6 on my notebook here.
> Some month I'll update the kernel and fix VMware and wireless again
> for a newer kernel, but for now that's what it runs.
> 
> Here's hdparm --verbose, sans patch:
> 
> hdparm --verbose -I /dev/sdb
> 
> /dev/sdb:
> outgoing cdb:  85 08 2e 00 00 00 00 00 00 00 00 00 00 40 ec 00
> SG_IO: ATA_16 status=0x2, host_status=0x0, driver_status=0x8
> SG_IO: sb[] = {72 00 00 00 00 00 00 0e}
> SG_IO: desc[] = {09 0c .. }
>      ATA_16 tf->status=0x00 tf->error=0x00
> 
> ATA device, with non-removable media
>        Model Number:       Hitachi HTS541612J9SA00
>        Serial Number:      SB2D11E4GESWAB
>        Firmware Revision:  SBDOC74P

..

Here it is again, with the entire returned sense data shown:

hdparm --verbose -I /dev/sdb

/dev/sdb:
outgoing cdb:  85 08 2e 00 00 00 00 00 00 00 00 00 00 40 ec 00
SG_IO: ATA_16 status=0x2, host_status=0x0, driver_status=0x8
SG_IO: sb[] = {72 00 00 00 00 00 00 0e}
SG_IO: desc[] = {09 0c 00 00 50 01 40 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00}
      ATA_16 tf->status=0x00 tf->error=0x00

ATA device, with non-removable media
        Model Number:       Hitachi HTS541612J9SA00
        Serial Number:      SB2D11E4GESWAB
        Firmware Revision:  SBDOC74P
...

I guess it "works" by accident, since the 00 device status
doesn't have the ATA error bit set.  But I suppose this also
means that a failed ATA command would not be noticed either..
At least not without the patch.

Cheers

  reply	other threads:[~2008-10-23 20:36 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-10-09 21:56 SMART to USB/sata dock in 2.6.27-rc5 doesn't work David Greaves
2008-10-10  8:09 ` SMART to USB/sata dock in 2.6.27-rc5 doesn't work : hardware limitation David Greaves
2008-10-14  2:18 ` SMART to USB/sata dock in 2.6.27-rc5 doesn't work Tejun Heo
2008-10-15 19:16   ` [smartmontools-support] " matthieu castet
2008-10-23 16:04     ` Mark Lord
2008-10-23 18:58       ` matthieu castet
2008-10-23 20:21         ` Mark Lord
2008-10-23 20:36           ` Mark Lord [this message]
2008-10-24 18:54             ` matthieu castet
2008-10-25 12:46               ` Mark Lord
2008-10-23 16:01   ` Mark Lord

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4900E051.9040608@rtr.ca \
    --to=liml@rtr.ca \
    --cc=castet.matthieu@free.fr \
    --cc=david@dgreaves.com \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=smartmontools-support@lists.sourceforge.net \
    --cc=tj@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).