On Tue, 2012-05-08 at 10:02 -0400, Alan Stern wrote: > On Tue, 8 May 2012, Paul Bolle wrote: > > 1) These patches try to hide those errors by: > > - downgrading one error to a notice; and > > That's a reasonable thing to do, IMO. > > > - setting the NO_WP_DETECT quirk for these five devices. > > But that isn't. These quirks are intended for devices that crash when > they receive the command in question. Yes, these USB memory sticks don't crash. (They actually seem to work just fine, something that I perhaps should have emphasized in the commit descriptions.) > They aren't meant to suppress sending commands to devices that can > properly reject them. Even the sticks that hit "bad_sense" (in sd_read_cache_type(), which I forgot to mention in the comment descriptions)? Is that not as severe as it suggests? Of course, an easy way out would be to downgrade both the "Asking for cache data failed" and the "No Caching mode page present" errors to notices. But the SCSI people might disagree with that approach. Paul Bolle -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
