Very good question, and I think the answer is yes, which means that the "if" could just as well be deleted because chunksamples is always > 0.
In that case, it looks good to me.
In other news, SVN r6449 should have all of the changes necessary for the SGI Indy driver (ip225015) to actually boot the Integrated Diagnostic Environment off of a CD (and promptly fail the diagnostics), or to boot sash (the SGI Stand-Alone Shell) and fx (the disc-partitioning tool), partition a blank HDD image, write out a valid boot sector, then boot up the IRIX 6.5 installer, at which point the installer bombs out. But still, progress.
