James Bottomley wrote:
On Wed, 2006-04-26 at 16:27 -0400, Mark Lord wrote:
From: Mark Lord <[email protected]>
..
When scsi_get_sense_info_fld() fails (returns 0), it does NOT update the
value of first_err_block. But sd_rw_intr() merrily continues to use that
variable regardless, possibly making incorrect decisions about retries and the like.
This patch removes the randomness there, by using the first sector of the
request (SCpnt->request->sector) in such cases, instead of first_err_block.
..
Thanks for finding the bug. Your solution is a bit, um, convoluted.
Heh, no, that's the original sd.c. I just tried to keep the patch
as tiny as possible to show what is needed. You are welcome to add
complexity to it, though. The surrounding code is, uhm, rather interesting.
What it should really be doing if we find no valid information field is
a break so we go out with the default good_sectors of zero (rather than
arriving at that value via a circuitous route).
Yes, but the difficulty there is that all of the convoluted logic
seems to still be wanted to set a correct "block_sectors" value,
needed as a parameter on the final call:
scsi_io_completion(SCpnt, good_bytes, block_sectors << 9);
How does this work?
...
Looks potentially buggy (still).
Cheers
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
