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

[M. Warner Losh]

In message: <43468.1172657639 at critter.freebsd.dk>
            "Poul-Henning Kamp" <phk at phk.freebsd.dk> writes:
: In message <20070228.031000.1649769988.imp at bsdimp.com>, "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...

Warner