--- drivers/cdrom/cdu31a.c.old Fri Feb 7 09:16:08 2003 +++ drivers/cdrom/cdu31a.c Fri Feb 7 09:42:53 2003 @@ -1341,7 +1341,7 @@ #endif } -/* read data from the drive. Note the nsect must be <= 4. */ +/* read data from the drive. Note the nblocks must be <= 4. */ static void read_data_block(char *buffer, unsigned int block, @@ -1364,7 +1364,7 @@ bytesleft = nblocks * 512; offset = 0; - /* If the data in the read-ahead does not match the block offset, + /* If the data in the read ahead does not match the block offset, then fix things up. */ if (((block % 4) * 512) != ((2048 - readahead_dataleft) % 2048)) { sony_next_block += block % 4; @@ -1384,9 +1384,9 @@ readahead_buffer + (2048 - readahead_dataleft), readahead_dataleft); - readahead_dataleft = 0; bytesleft -= readahead_dataleft; offset += readahead_dataleft; + readahead_dataleft = 0; } else { /* The readahead will fill the whole buffer, get the data and return. */ @@ -1533,7 +1533,7 @@ /* * The OS calls this to perform a read or write operation to the drive. - * Write obviously fail. Reads to a read ahead of sony_buffer_size + * Writes obviously fail. Reads to a read ahead of sony_buffer_size * bytes to help speed operations. This especially helps since the OS * uses 1024 byte blocks and the drive uses 2048 byte blocks. Since most * data access on a CD is done sequentially, this saves a lot of operations. @@ -1546,6 +1546,7 @@ unsigned int res_size; int num_retries; unsigned long flags; + char *buffer; #if DEBUG @@ -1616,6 +1617,7 @@ block = CURRENT->sector; nblock = CURRENT->nr_sectors; + buffer = CURRENT->buffer; if (!sony_toc_read) { printk("CDU31A: TOC not read\n"); @@ -1695,8 +1697,17 @@ } } - read_data_block(CURRENT->buffer, block, nblock, - res_reg, &res_size); + if (nblock >= 4) { + read_data_block(buffer, block, 4, + res_reg, &res_size); + nblock -= 4; + block += 4; + buffer += 4 * 512; + } else { + read_data_block(buffer, block, nblock, + res_reg, &res_size); + nblock = 0; + } if (res_reg[0] == 0x20) { if (num_retries > MAX_CDU31A_RETRIES) { end_request(0); @@ -1714,6 +1725,8 @@ translate_error(res_reg[1]), block, nblock); } + goto try_read_again; + } else if (nblock > 0) { goto try_read_again; } else { end_request(1);