Re: Read errors on Flash Drive Transcend TS1GJF2A

[Alan Stern <stern@xxxxxxxxxxxxxxxxxxx>]

On Fri, 16 Nov 2007, RTE wrote:

> On Fri, 12 Oct 2007 00:18:11 +0400, Alan Stern <stern@xxxxxxxxxxxxxxxxxxx> wrote:
> > I'd like you to try some more experiments
> 
> sorry for the long period of silence - not enough free time :(
> 
> 2.6.17-git21 whith patch
> after copying 700 MB file from flash to HDD
> errors in the copied file are not present

Okay, that's good.  Now here's a comparable patch for 2.6.17-git22 or 
later.  Let's see what happens when the copied data does contain 
errors.

Alan Stern


--- 2.6.17/drivers/scsi/sd.c1	2007-10-11 15:53:20.000000000 -0400
+++ 2.6.17/drivers/scsi/sd.c	2007-11-16 14:27:24.000000000 -0500
@@ -904,6 +904,8 @@ static void sd_rw_intr(struct scsi_cmnd 
 		sense_valid = scsi_command_normalize_sense(SCpnt, &sshdr);
 		if (sense_valid)
 			sense_deferred = scsi_sense_is_deferred(&sshdr);
+printk(KERN_INFO "driver_byte %x, sense key %x\n", driver_byte(result),
+sshdr.sense_key);
 	}
 #ifdef CONFIG_SCSI_LOGGING
 	SCSI_LOG_HLCOMPLETE(1, printk("sd_rw_intr: %s: res=0x%x\n", 
@@ -954,6 +956,9 @@ static void sd_rw_intr(struct scsi_cmnd 
 		 * the resolution of the device's medium.
 		 */
 		good_bytes = (bad_lba - start_lba)*SCpnt->device->sector_size;
+printk(KERN_INFO "error_sector %d, req_sector %d, resid %d, underflow %d\n",
+(int) bad_lba, (int) start_lba, (int) SCpnt->resid,
+(int) SCpnt->underflow);
 		break;
 	case RECOVERED_ERROR:
 	case NO_SENSE:
@@ -979,6 +984,9 @@ static void sd_rw_intr(struct scsi_cmnd 
 		break;
 	}
  out:
+if (good_bytes != xfer_size)
+printk(KERN_INFO "Adjusted good_bytes from %d to %d\n",
+xfer_size, good_bytes);
 	scsi_io_completion(SCpnt, good_bytes);
 }
 


-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Linux-usb-users@xxxxxxxxxxxxxxxxxxxxx
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-users