Re: [Qemu-devel] [PATCH qemu] fdc: Exit if ISA controller does not support DMA

2018-03-15 Thread John Snow
On 03/05/2018 05:15 AM, Thomas Huth wrote: > On 08.12.2017 22:29, John Snow wrote: >> >> On 11/21/2017 09:48 PM, Alexey Kardashevskiy wrote: >>> On 07/11/17 11:58, John Snow wrote: On 10/26/2017 02:46 AM, Alexey Kardashevskiy wrote: > A "powernv" machine type defines an ISA bus

Re: [Qemu-devel] [PATCH qemu] fdc: Exit if ISA controller does not support DMA

2018-03-05 Thread Philippe Mathieu-Daudé
On 03/05/2018 07:29 AM, Philippe Mathieu-Daudé wrote: > On 10/26/2017 03:46 AM, Alexey Kardashevskiy wrote: >> A "powernv" machine type defines an ISA bus but it does not add any >> DMA >> controller to it so it is possible to hit assert(fdctrl->dma) by >> adding "-machine powernv -device isa-fdc".

Re: [Qemu-devel] [PATCH qemu] fdc: Exit if ISA controller does not support DMA

2018-03-05 Thread Philippe Mathieu-Daudé
On 10/26/2017 03:46 AM, Alexey Kardashevskiy wrote: > A "powernv" machine type defines an ISA bus but it does not add any > DMA > controller to it so it is possible to hit assert(fdctrl->dma) by > adding "-machine powernv -device isa-fdc". The same happens with the Alpha machine. On 12/13/2017 03

Re: [Qemu-devel] [PATCH qemu] fdc: Exit if ISA controller does not support DMA

2018-03-05 Thread Thomas Huth
On 08.12.2017 22:29, John Snow wrote: > > On 11/21/2017 09:48 PM, Alexey Kardashevskiy wrote: >> On 07/11/17 11:58, John Snow wrote: >>> >>> >>> On 10/26/2017 02:46 AM, Alexey Kardashevskiy wrote: A "powernv" machine type defines an ISA bus but it does not add any DMA controller to it so

Re: [Qemu-devel] [PATCH qemu] fdc: Exit if ISA controller does not support DMA

2017-12-12 Thread Hervé Poussineau
Le 08/12/2017 à 22:29, John Snow a écrit : [...] It looks like Herve was working on decoupling floppies from i8257, but perhaps didn't get all the way through -- I'm not actually clear on what work remains to be done here, maybe he can chime in if he's still interested in the project? Indeed,

Re: [Qemu-devel] [PATCH qemu] fdc: Exit if ISA controller does not support DMA

2017-12-08 Thread John Snow
On 11/21/2017 09:48 PM, Alexey Kardashevskiy wrote: > On 07/11/17 11:58, John Snow wrote: >> >> >> On 10/26/2017 02:46 AM, Alexey Kardashevskiy wrote: >>> A "powernv" machine type defines an ISA bus but it does not add any DMA >>> controller to it so it is possible to hit assert(fdctrl->dma) by >

Re: [Qemu-devel] [PATCH qemu] fdc: Exit if ISA controller does not support DMA

2017-11-21 Thread Alexey Kardashevskiy
On 07/11/17 11:58, John Snow wrote: > > > On 10/26/2017 02:46 AM, Alexey Kardashevskiy wrote: >> A "powernv" machine type defines an ISA bus but it does not add any DMA >> controller to it so it is possible to hit assert(fdctrl->dma) by >> adding "-machine powernv -device isa-fdc". >> >> This rep

Re: [Qemu-devel] [PATCH qemu] fdc: Exit if ISA controller does not support DMA

2017-11-06 Thread John Snow
On 10/26/2017 02:46 AM, Alexey Kardashevskiy wrote: > A "powernv" machine type defines an ISA bus but it does not add any DMA > controller to it so it is possible to hit assert(fdctrl->dma) by > adding "-machine powernv -device isa-fdc". > > This replaces assert() with an error message. > > Sig

[Qemu-devel] [PATCH qemu] fdc: Exit if ISA controller does not support DMA

2017-10-25 Thread Alexey Kardashevskiy
A "powernv" machine type defines an ISA bus but it does not add any DMA controller to it so it is possible to hit assert(fdctrl->dma) by adding "-machine powernv -device isa-fdc". This replaces assert() with an error message. Signed-off-by: Alexey Kardashevskiy --- Is it a must for ISA to have