On Tue, Apr 21, 2009 at 01:52:54PM -0700, Dave Hansen wrote: > This is with current git as of this morning, which is at v2.6.30-rc2. > > I have a 1.5TB USB device which gets a bit angry when I plug it in. It > ends up with a scsi_disk->capacity of ffffffffaea87b30. I tracked it > down to the lba calculation in read_capacity_10(): > > lba = (buffer[0] << 24) | (buffer[1] << 16) | > (buffer[2] << 8) | buffer[3]; > > lba is getting all 0xf's in its high 32 bits. It seems odd that this > would happen since 'buffer' is an 'unsigned char', but that is > apparently what is going on. Nothing odd here; you get unsigned char promoted to int (since the entire range fits into the range of int), then calculations are carried within int and in the end you have (overflown into negative) int sign-extended into u64. Solution is correct, but perhaps use of get_unaligned_be32(buffer) would be better. -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
