Re: [Qemu-devel] [PATCH v2] scsi: esp: remove handling of SATN/STOP

2016-06-17 Thread Paolo Bonzini
On 17/06/2016 15:41, Mark Cave-Ayland wrote: >> > >> > Ok, I'll either fix migration (adding a subsection) or bump the version >> > number. > Hi Paolo, > > Just to clarify that these hangs aren't migration related, they occur > simply trying to boot the above images from ISOs with the patch

Re: [Qemu-devel] [PATCH v2] scsi: esp: remove handling of SATN/STOP

2016-06-17 Thread Mark Cave-Ayland
On 17/06/16 14:35, Paolo Bonzini wrote: > On 17/06/2016 15:13, Mark Cave-Ayland wrote: >> Unforunately this causes regressions on a few of my SPARC32 images: >> NextStep, Solaris 1.1.2, NetBSD and Debian Etch all hang whilst >> enumerating the SCSI bus with this patch applied. > > Ok, I'll

Re: [Qemu-devel] [PATCH v2] scsi: esp: remove handling of SATN/STOP

2016-06-17 Thread Paolo Bonzini
On 17/06/2016 15:13, Mark Cave-Ayland wrote: > Unforunately this causes regressions on a few of my SPARC32 images: > NextStep, Solaris 1.1.2, NetBSD and Debian Etch all hang whilst > enumerating the SCSI bus with this patch applied. Ok, I'll either fix migration (adding a subsection) or bump

Re: [Qemu-devel] [PATCH v2] scsi: esp: remove handling of SATN/STOP

2016-06-17 Thread Mark Cave-Ayland
On 17/06/16 10:55, Paolo Bonzini wrote: > The implementation of SATN/STOP is completely busted. The idea > would be that the next DMA read is for a SCSI message and after > that the adapter would transition to the command phase. > > The recent fix to SATN/STOP broke migration, which is one more

Re: [Qemu-devel] [PATCH v2] scsi: esp: remove handling of SATN/STOP

2016-06-17 Thread Amit Shah
On (Fri) 17 Jun 2016 [11:55:48], Paolo Bonzini wrote: > The implementation of SATN/STOP is completely busted. The idea > would be that the next DMA read is for a SCSI message and after > that the adapter would transition to the command phase. > > The recent fix to SATN/STOP broke migration,

Re: [Qemu-devel] [PATCH v2] scsi: esp: remove handling of SATN/STOP

2016-06-17 Thread P J P
+-- On Fri, 17 Jun 2016, Paolo Bonzini wrote --+ | The implementation of SATN/STOP is completely busted. The idea | would be that the next DMA read is for a SCSI message and after | that the adapter would transition to the command phase. | | The recent fix to SATN/STOP broke migration, which is

[Qemu-devel] [PATCH v2] scsi: esp: remove handling of SATN/STOP

2016-06-17 Thread Paolo Bonzini
The implementation of SATN/STOP is completely busted. The idea would be that the next DMA read is for a SCSI message and after that the adapter would transition to the command phase. The recent fix to SATN/STOP broke migration, which is one more reason to drop SATN/STOP handling completely. It