[Qemu-devel] XP install cores with SCSI LSI 53C895A disks - follow up

[Qemu-devel] XP install cores with SCSI LSI 53C895A disks - follow up

[Gerhard Wiesinger]

Hello,

Clean XP install cores with SCSI LSI 53C89A disk when copying files. 
Isn't on the same file, so looks like a timing problem. Reproduceable. 
Driver used is sym_hi. Details are below.

See also: 
https://lists.gnu.org/archive/html/qemu-devel/2012-03/msg00523.html

Looks like problem is from Paolo's commit: 
2f0772c5b4818d4b2078be9dace0036d1030faee
qemu-system-x86_64: hw/lsi53c895a.c:351: lsi_soft_reset: Assertion 
`((&s->queue)->tqh_first == ((void *)0))' failed.

So SCSI queue isn't empty (was an assumption and asserted), so 
qdev_reset_all(&s->dev.qdev); might not work or some other timing 
related issues.

Any ideas to solve?
Reproduceable?

I'm using BIOS from SEABIOS.

Thank you.

Ciao,
Gerhard

Image created with:
qemu-img create -f qcow2 XP-TEST.qcow2 10G

qemu-kvm: 4d9367b76f71c6d938cf8201392abe4bfb1136cb
/root/download/qemu/git/qemu-kvm/x86_64-softmmu/qemu-system-x86_64
-device nec-usb-xhci,id=usb0
-drive file=VM-XP-TEST/XP-TEST.qcow2,media=disk,if=scsi,bus=0,unit=0
-drive if=ide,index=3,media=cdrom,file=ISO/XP.iso
-boot order=dac,menu=on
-m 2048 -k de
-vga vmware
-vnc :0 -bios /root/download/seabios/git/seabios/out/bios.bin
-chardev stdio,id=seabios -device isa-debugcon,iobase=0x402,chardev=seabios
-device rtl8139,mac=00:02:44:92:87:6a,vlan=0,romfile= -net 
tap,ifname=tap0,script=no,downscript=no,vlan=0

Re: [Qemu-devel] XP install cores with SCSI LSI 53C895A disks - follow up

[Paolo Bonzini]

Il 04/11/2012 11:20, Gerhard Wiesinger ha scritto:
> Hello,
> 
> Clean XP install cores with SCSI LSI 53C89A disk when copying files.
> Isn't on the same file, so looks like a timing problem. Reproduceable.
> Driver used is sym_hi. Details are below.

Hmm, why is XP resetting the HBA while copying files?  Can you get
logging output when compiling with hw/lsi53c895a.c with DEBUG_LSI defined?

> See also:
> https://lists.gnu.org/archive/html/qemu-devel/2012-03/msg00523.html
> 
> Looks like problem is from Paolo's commit:
> 2f0772c5b4818d4b2078be9dace0036d1030faee
> qemu-system-x86_64: hw/lsi53c895a.c:351: lsi_soft_reset: Assertion
> `((&s->queue)->tqh_first == ((void *)0))' failed.
> 
> So SCSI queue isn't empty (was an assumption and asserted), so
> qdev_reset_all(&s->dev.qdev); might not work or some other timing
> related issues.

Even before that commit, it probably would result in a use-after-free or
similar problem.

Thanks!

Paolo