Re: [Qemu-devel] qemu/hw ide.c

[Jens Axboe]

On Mon, Feb 19 2007, Thiemo Seufer wrote:
> Jens Axboe wrote:
> > On Mon, Feb 19 2007, Thiemo Seufer wrote:
> > > CVSROOT:  /sources/qemu
> > > Module name:      qemu
> > > Changes by:       Thiemo Seufer <ths>     07/02/19 00:59:34
> > > 
> > > Modified files:
> > >   hw             : ide.c 
> > > 
> > > Log message:
> > >   Ignore special flags in nsector variable.
> > > 
> > > CVSWeb URLs:
> > > http://cvs.savannah.gnu.org/viewcvs/qemu/hw/ide.c?cvsroot=qemu&r1=1.53&r2=1.54
> > 
> > Why is nsector uint32_t to begin with?
> 
> Because nobody sent a patch to fix it, I figure.

Actually I seem to recall it's because it's being overloaded for
requests that are > 256 sectors. It would be a good cleanup to get rid
of that an turn nsector into a proper uint8_t.

-- 
Jens Axboe