LKML Archive on
help / color / mirror / Atom feed
From: Sergey Dolgov <>
To: Matthew Dharm <>
Cc: Alan Stern <>,
	Oliver Neukum <>,,
Subject: Re: usb-storage, error reading the last 8 sectors, regression in 2.6.25-rc7
Date: Tue, 1 Apr 2008 20:48:46 +0400 (MSD)	[thread overview]
Message-ID: <alpine.DEB.1.10.0804012040300.3959@slke.laptop> (raw)
In-Reply-To: <>

[-- Attachment #1: Type: TEXT/PLAIN, Size: 3072 bytes --]

On Tue, 1 Apr 2008, Matthew Dharm wrote:

> On Tue, Apr 01, 2008 at 10:42:51AM -0400, Alan Stern wrote:
>> On Tue, 1 Apr 2008, Matthew Dharm wrote:
>>> On Tue, Apr 01, 2008 at 10:28:52AM -0400, Alan Stern wrote:
>>>> On Tue, 1 Apr 2008, Oliver Neukum wrote:
>>>>> Am Dienstag, 1. April 2008 03:58:31 schrieb Alan Stern:
>>>>>> Nevertheless, it's clear that the problem has nothing to do with the
>>>>>> USB stack.  The real source of the problem lies in the device itself,
>>>>>> for reporting a bogus error when in fact nothing went wrong.  That may
>>>>>> also explain why you don't always see the problem -- sometimes the
>>>>>> device works the way it ought to.
>>>>> Reminds me of the devices that can read the last sector but only if it is read
>>>>> by itself. Do you reckon this device may have the "opposite" quirk?
>>>> Could be something like that.
>>> Didn't I see some SCSI patches go by to implement exactly this change?
>>> That is, only read the last sector by itself?
>> You are getting the two problems mixed up.  The older problem, which
>> the SCSI patche addressed, was that the device would fail when
>> accessing the last sector unless the transfer was 1 sector long.
>> This problem is different.  When performing an 8-sector read that
>> includes the last sector, the device succeeds.  When performing a
>> 7-sector read starting from the same place (so not including the last
>> sector), the device fails.
> I thought the patch I saw unconditionally re-wrote any access that included
> the last sector into two accesses -- everything but the last sector, and
> the last sector.
> In other words, the patch attempted to avoid problems on devices that
> couldn't access the last sector unless the transfer was 1 sector long by
> ONLY accessing the last sector in a single transfer.
> If I'm remembering correctly, that would explain the behavior change which
> lead to the exposure of the bad behavior of this new device.  This new
> device worked with the old code, but not with the new code.
> Basically, by avoiding a common error condition in device firmware, we've
> found a device that has exactly the opposite bug.
> Presuming someone can find the patch in the archive, reverting it would
> produce a good test case; it should restore this device to a working state.
> Maybe we need some auto-detect logic here; try the new way, if it fails,
> revert to the old behavior.  That's probably the safe order, as a lot of
> the devices with the more 'classic' bug just die completely, whereas this
> one appears to be recoverable.

First of all, thank you guys for your help!

I have reverted commit 23c3e290fb9ce38cabc2822b47583fc8702411bf ([SCSI] 
usbstorage: use last_sector_bug flag universally) and my card reader seems 
to be working properly.

I'll need to give it some more testing to be 100 % shure though (so far 
I've just booted once).

As for what should be done to handle both kinds of hardware peculiarities, 
I'm not the one to advise, but looks like sometheng needs to be done.


  parent reply	other threads:[~2008-04-01 16:49 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-03-30  7:49 Sergey Dolgov
2008-03-30 18:59 ` Alan Stern
2008-03-30 22:32   ` Sergey Dolgov
2008-04-01  1:18     ` Sergey Dolgov
2008-04-01  1:58       ` Alan Stern
2008-04-01  7:38         ` Oliver Neukum
2008-04-01 14:28           ` Alan Stern
2008-04-01 14:34             ` Matthew Dharm
2008-04-01 14:42               ` Alan Stern
2008-04-01 15:26                 ` Boaz Harrosh
2008-04-01 15:53                 ` Matthew Dharm
2008-04-01 16:31                   ` Boaz Harrosh
2008-04-01 20:24                     ` Hans de Goede
2008-04-01 21:01                       ` Alan Stern
2008-04-02  7:01                         ` Hans de Goede
2008-04-02 14:15                           ` Alan Stern
2008-04-01 16:48                   ` Sergey Dolgov [this message]
2008-04-03 22:49 2.6.25-rc8-git2: Reported regressions from 2.6.24 Rafael J. Wysocki
2008-04-03 23:22 ` usb-storage, error reading the last 8 sectors, regression in 2.6.25-rc7 Rafael J. Wysocki

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:

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

  git send-email \
    --in-reply-to=alpine.DEB.1.10.0804012040300.3959@slke.laptop \ \ \ \ \ \ \
    --subject='Re: usb-storage, error reading the last 8 sectors, regression in 2.6.25-rc7' \

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

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).