Subject: Re: CVS commit: src/sys/dev/scsipi
To: Reinoud Zandijk <reinoud@NetBSD.org>
From: Charles M. Hannum <mycroft@MIT.EDU>
List: source-changes
Date: 09/11/2006 14:40:22
On Mon, Sep 11, 2006 at 05:57:07PM +0000, Reinoud Zandijk wrote:
> 
> Module Name:	src
> Committed By:	reinoud
> Date:		Mon Sep 11 17:57:07 UTC 2006
> 
> Modified Files:
> 	src/sys/dev/scsipi: scsipi_base.c
> 
> Log Message:
> Don't add one to the capacity returned of READ CAPACITY! it results in
> reporting a sd* disc that is one sector too big (!) Normally not much a
> problem in FFS since its clustered but other filingsystems *do* care.

*facepalm*

Could you please READ THE SPEC before committing??  READ CAPACITY is
supposed to return the highest LBA; to get the number of blocks, you
have to add one.

I realize that some broken USB devices return the wrong number, but
that's no excuse for breaking the interface for everyone else.