2012/2/16 James Bottomley <James.Bottomley@xxxxxxxxxxxxxxxxxxxxx>: > On Thu, 2012-02-16 at 12:22 +0530, Amit Sahrawat wrote: > >> diff --git a/drivers/scsi/sd.c b/drivers/scsi/sd.c >> index c691fb5..3ca507a 100644 >> --- a/drivers/scsi/sd.c >> +++ b/drivers/scsi/sd.c >> @@ -50,6 +50,11 @@ >> #include <linux/string_helpers.h> >> #include <linux/async.h> >> #include <linux/slab.h> >> + >> +#ifdef CONFIG_ATA >> +#include <linux/libata.h> >> +#endif >> + >> #include <linux/pm_runtime.h> >> #include <asm/uaccess.h> >> #include <asm/unaligned.h> >> @@ -2129,7 +2134,11 @@ sd_read_cache_type(struct scsi_disk *sdkp, unsigned char *buffer) >> if (modepage == 0x3F) { >> sd_printk(KERN_ERR, sdkp, "No Caching mode page " >> "present\n"); >> +#ifdef CONFIG_ATA >> + goto WCE_USING_ATA; >> +#else >> goto defaults; >> +#endif >> } else if ((buffer[offset] & 0x3f) != modepage) { >> sd_printk(KERN_ERR, sdkp, "Got wrong page\n"); >> goto defaults; >> @@ -2149,6 +2158,15 @@ sd_read_cache_type(struct scsi_disk *sdkp, unsigned char *buffer) >> "Uses READ/WRITE(6), disabling FUA\n"); >> sdkp->DPOFUA = 0; >> } >> +#ifdef CONFIG_ATA >> +WCE_USING_ATA: >> + if (!sdp->removable && !sdkp->WCE) { >> + sd_printk(KERN_NOTICE, sdkp, "Try to check write cache" >> + " enable/disable using ATA command\n"); >> + sdkp->WCE = ata_get_cachestatus(sdp); >> + } >> +#endif >> + > > I already said before, this is a non starter: we can't randomly send > SCSI encapsulated ATA commands to disks from sd. What we can do is give > you a WCE variable you can change on the fly in sysfs via whatever > mechanism is most appropriate (but this would have to be a mechanism > external to sd.c). Oops.. it is mistake. we will make patch v2 included setting via sysfs. Thanks for your reply. > > Alternatively, you can co-opt the USB quirks handling to set it for you. > > James > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-ide" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html