cvs commit: src/sys/dev/usb umass.c

M. Warner Losh imp at
Wed Feb 28 10:31:35 UTC 2007

In message: <43468.1172657639 at>
            "Poul-Henning Kamp" <phk at> writes:
: In message <20070228.031000.1649769988.imp at>, "M. Warner Losh" writes
: :
: >: >  Create a new quirk READ_CAPACITY_OFFBY1 
: >: 
: >: A better idea would be to have scsi_da.c try to read the 
: >: last sector and chop it if it fails.
: >
: >Why is that a better idea?
: Because it will work automatically and not rely on us preventively
: adding the bogus devices to the kernel.
: Trivially testable quicks shouldn't have to be compiled into
: static lists.

Others have observed that the number of sectors on a disk should be
even.  Therefore if maxsectors returned is even, it implies an odd
number of sectors and should be adjusted.  That might be an even
easier path to follow...  I'm never sure when one can schedule
additional I/O inside of cam...


More information about the cvs-src mailing list