Re: [Qemu-devel] [PATCH] LSI53C895A: Do not update current_dma_len with dbc in TIA mode

[Justin Chevrier]

Ryan Harper wrote:
> Do you also have SCSI debug on in scsi-disk.c ?  I'd
> really like to see
> the scsi command that was generating the read with lengh
> 69632.
> 
> Actually, have you tried this since the 40 bit DMA patch
> was included?

I'm running head.

The command is a Read:

scsi-disk: Command: lun=0 tag=0x1003b data=0x28 0x00 0x00 0x5a 0x81 0x57 0x00 
0x00 0x88 0x00
scsi-disk: Read (sector 5931351, count 136)
scsi-disk: Read sector_count=136

The 69632 is coming from 136 * 512

> Yeah, I think for now, the right thing to do is revert the
> old patch
> until we figure how to handle this correctly for both test
> cases.

This reverts the relevant section of the original patch:

--- hw/lsi53c895a.c     (revision 5799)
+++ hw/lsi53c895a.c     (working copy)
@@ -920,7 +920,6 @@
             break;
         case PHASE_DI:
             s->waiting = 2;
-            s->current_dma_len = s->dbc;
             lsi_do_dma(s, 0);
             if (s->waiting)
                 s->waiting = 3;