uas: Add a quirk for rejecting ATA_12 and ATA_16 commands
And set this quirk for the Seagate Expansion Desk (0bc2:2312), as that one seems to hang upon receiving an ATA_12 or ATA_16 command. https://bugzilla.kernel.org/show_bug.cgi?id=79511 https://bbs.archlinux.org/viewtopic.php?id=183190 While at it also add missing documentation for the u value for usb-storage quirks. Cc: stable@vger.kernel.org # 3.16, 3.17 Signed-off-by: Hans de Goede <hdegoede@redhat.com> -- Changes in v2: Add documentation for new t and u usb-storage.quirks flags Changes in v3: Fix typo in documentation Changes in v4: Also apply the quirk to (0bc2:3312) Changes in v5: Rebased on 3.17-rc5, drop u documentation, already upstream Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:

committed by
Greg Kroah-Hartman

parent
ab945eff83
commit
593078525c
@@ -28,6 +28,7 @@
|
||||
#include <scsi/scsi_tcq.h>
|
||||
|
||||
#include "uas-detect.h"
|
||||
#include "scsiglue.h"
|
||||
|
||||
/*
|
||||
* The r00-r01c specs define this version of the SENSE IU data structure.
|
||||
@@ -49,6 +50,7 @@ struct uas_dev_info {
|
||||
struct usb_anchor cmd_urbs;
|
||||
struct usb_anchor sense_urbs;
|
||||
struct usb_anchor data_urbs;
|
||||
unsigned long flags;
|
||||
int qdepth, resetting;
|
||||
struct response_iu response;
|
||||
unsigned cmd_pipe, status_pipe, data_in_pipe, data_out_pipe;
|
||||
@@ -714,6 +716,15 @@ static int uas_queuecommand_lck(struct scsi_cmnd *cmnd,
|
||||
|
||||
BUILD_BUG_ON(sizeof(struct uas_cmd_info) > sizeof(struct scsi_pointer));
|
||||
|
||||
if ((devinfo->flags & US_FL_NO_ATA_1X) &&
|
||||
(cmnd->cmnd[0] == ATA_12 || cmnd->cmnd[0] == ATA_16)) {
|
||||
memcpy(cmnd->sense_buffer, usb_stor_sense_invalidCDB,
|
||||
sizeof(usb_stor_sense_invalidCDB));
|
||||
cmnd->result = SAM_STAT_CHECK_CONDITION;
|
||||
cmnd->scsi_done(cmnd);
|
||||
return 0;
|
||||
}
|
||||
|
||||
spin_lock_irqsave(&devinfo->lock, flags);
|
||||
|
||||
if (devinfo->resetting) {
|
||||
@@ -1080,6 +1091,8 @@ static int uas_probe(struct usb_interface *intf, const struct usb_device_id *id)
|
||||
devinfo->resetting = 0;
|
||||
devinfo->running_task = 0;
|
||||
devinfo->shutdown = 0;
|
||||
devinfo->flags = id->driver_info;
|
||||
usb_stor_adjust_quirks(udev, &devinfo->flags);
|
||||
init_usb_anchor(&devinfo->cmd_urbs);
|
||||
init_usb_anchor(&devinfo->sense_urbs);
|
||||
init_usb_anchor(&devinfo->data_urbs);
|
||||
|
Reference in New Issue
Block a user