lkml.org 
[lkml]   [2012]   [Jul]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [ 38/48] SCSI & usb-storage: add try_rc_10_first flag
    On Mon, Jul 02, 2012 at 03:23:18PM -0700, Linus Torvalds wrote:
    > > Martin Petersen did extensive testing of devices when we
    > > changed it and doing RC16 first is hedged around by claiming support not
    > > only for SCSI_3 but also for SBC_2 in your INQUIRY data.
    >
    > Umm. That just smells like BS to me.
    >
    > The "extensive testing" part was clearly not true, and it seems that
    > SCSI people sometimes forget that the biggest user (by *far*) is the
    > USB storage layer.

    As James said, *at the time*, USB overrode the SBC_2 claims from USB
    devices and forced them to be SCSI_2. So no amount of testing we did
    would have uncovered this.

    > Also, your protection claim seems to be invalidated by the actual
    > code. Yes, it checks if the device claims to support protection. But
    > it *also* says "let's do that 16b command if "scsi_level >
    > SCSI_SPC_2". So your claim that it hedges around it by looking at the
    > inquiry data is pure crap. It's simply not true. Just look at the
    > code:

    I wrote this code ... James' memory is off. What happened is that T10
    in their infinite wisdom decided to put things like "supports TRIM" and
    "is actually a 4k block size but fakes 512 byte blocks" in the Read
    Capacity 16 results. So if we want to support those kinds of things
    (and I think we do), then we need to send Read Capacity 16 to devices.

    It's not about "enterprise features" at all, but about supporting the
    next generation of standard consumer drives. I'm tempted to say the
    USB Storage driver needs to go back to the way things were, because I
    don't see any other way to fix this.

    I have no idea what Windows is doing to support these features. That
    might be a fruitful course of investigation.

    --
    Matthew Wilcox Intel Open Source Technology Centre
    "Bill, look, we understand that you're interested in selling us this
    operating system, but compare it to ours. We can't possibly take such
    a retrograde step."


    \
     
     \ /
      Last update: 2012-07-03 03:21    [W:3.325 / U:0.120 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site