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 _______________________________________________ Qemu-devel mailing list Qemu-devel@nongnu.org http://lists.nongnu.org/mailman/listinfo/qemu-devel