Re: [PATCH] SCSI: make use of the residue value

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 2008-02-27 at 15:25 -0500, Alan Stern wrote:
> On 20 Feb 2008, Alan Stern wrote:
> 
> > This patch (as1036) causes the SCSI midlayer to take into account the
> > residue value provided by some low-level drivers.  There's at least
> > one situation (USB mass storage with the Bulk-only transport) where
> > the specification states that it is permissible for a device to
> > indicate some of the data was not transferred correctly merely by
> > setting the residue value, without issuing a Check Condition.
> 
> After a week, there hasn't been any feedback on this patch.  Has it
> been accepted?  Is there anything wrong with it?  Is it still on a
> "to-look-at" queue?

OK, I ran it through its paces, but it fails in testing.  A very fun
failure, actually, some disks fail to appear with udev.

The reason is they return a residue from the VPD inquiry.  What your
patch actually causes is the block layer to resubmit the command with
the residue and triggers an overrun error (because the length in the
command is now much longer than the data buffer).

The bottom line is that this patch won't work with variable length
commands like inquiry that always return a residue.

James




--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux