2012/6/12, Sergei Shtylyov <sshtylyov@xxxxxxxxxx>: > Hello. > > On 10-06-2012 4:45, Namjae Jeon wrote: > >>>> Update information of Seagate Portable HDD and WD My Passport HDD in >>>> quirk list. > >>>> Signed-off-by: Namjae Jeon<namjae.jeon@xxxxxxxxxxx> >>>> Signed-off-by: Pankaj Kumar<pankaj.km@xxxxxxxxxxx> >>>> Signed-off-by: Amit Sahrawat<a.sahrawat@xxxxxxxxxxx> >>>> --- >>>> drivers/usb/storage/unusual_devs.h | 24 ++++++++++++++++++++++++ >>>> 1 file changed, 24 insertions(+) > >>>> diff --git a/drivers/usb/storage/unusual_devs.h >>>> b/drivers/usb/storage/unusual_devs.h >>>> index 1719886..64f0369 100644 >>>> --- a/drivers/usb/storage/unusual_devs.h >>>> +++ b/drivers/usb/storage/unusual_devs.h >>>> @@ -1267,6 +1267,18 @@ UNUSUAL_DEV( 0x0af0, 0xd357, 0x0000, 0x0000, >>>> USB_SC_DEVICE, USB_PR_DEVICE, NULL, >>>> 0 ), >>>> >>>> +/* Reported by Namjae Jeon<namjae.jeon@xxxxxxxxxxx> >>>> + * Even though write cache is enabled by default, >>>> + * SCSI driver assume that cache is not enabled >>>> + * which in turn might lead loss of data >>> ^ to the > >>>> + * since data still will be in cache. >>>> + * So this entry is needed for assuming that cache is enabled. >>>> + */ >>>> +UNUSUAL_DEV(0x0bc2, 0x2300, 0x0000, 0x9999, >>>> + "Seagate", >>>> + "Portable HDD", >>>> + USB_SC_DEVICE, USB_PR_DEVICE, NULL, US_FL_WRITE_CACHE), >>>> + >>>> /* Reported by Ben Efros<ben@xxxxxxxxxxxxx> */ >>>> UNUSUAL_DEV( 0x0bc2, 0x3010, 0x0000, 0x0000, >>>> "Seagate", >>>> @@ -1468,6 +1480,18 @@ UNUSUAL_DEV( 0x1058, 0x0704, 0x0000, 0x9999, >>>> USB_SC_DEVICE, USB_PR_DEVICE, NULL, >>>> US_FL_SANE_SENSE), >>>> >>>> +/* Reported by Namjae Jeon<namjae.jeon@xxxxxxxxxxx> >>>> + * Even though write cache is enabled by default, >>>> + * SCSI driver assume that cache is not enabled >>>> + * which in turn might lead loss of data >>> ^ to the >> to the ? Sorry, I don't understand what you want to change. >> like this ? "Even though write cache is enabled by default, SCSI >> driver assume that cache is not enabled and it in turn might lead the >> loss of data. " > > Lead to the loss of data. Okay, I changed it on the latest patch. Thanks a lot. > > WBR, Sergei > -- 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