[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v9 8/9] hw/m68k: add a dummy SWIM floppy control
From: |
Laurent Vivier |
Subject: |
Re: [Qemu-devel] [PATCH v9 8/9] hw/m68k: add a dummy SWIM floppy controller |
Date: |
Mon, 9 Sep 2019 18:34:22 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.8.0 |
Le 09/09/2019 à 18:18, Dr. David Alan Gilbert a écrit :
> * Peter Maydell (address@hidden) wrote:
>> On Mon, 9 Sep 2019 at 17:05, Laurent Vivier <address@hidden> wrote:
>>>
>>> Co-developed-by: Mark Cave-Ayland <address@hidden>
>>> Signed-off-by: Mark Cave-Ayland <address@hidden>
>>> Signed-off-by: Laurent Vivier <address@hidden>
>>> Reviewed-by: Hervé Poussineau <address@hidden>
>>
>>> +static void sysbus_swim_class_init(ObjectClass *oc, void *data)
>>> +{
>>> + DeviceClass *dc = DEVICE_CLASS(oc);
>>> +
>>> + dc->realize = sysbus_swim_realize;
>>> +}
>>
>> Missing reset and VMState for migration. These should be
>> baseline requirements for adding new device models to the
>> tree, because in an ideal world every device would support
>> both -- we should be gradually fixing the existing devices
>> which are missing these, and not letting new devices in,
>> so the situation gets gradually better, not worse.
>
> I'm happy to see things marked unmigratable instead for obscure
> devices; I don't see the need to worry about migration for stuff
> that's not going to be migrated.
>
I think in this case it's better to mark it unmigratable because the
structure will change in the future to really manage floppy and it
avoids to introduces a new version of the migration data stream.
Thanks,
Laurent
- [Qemu-devel] [PATCH v9 0/9] hw/m68k: add Apple Machintosh Quadra 800 machine, Laurent Vivier, 2019/09/09
- [Qemu-devel] [PATCH v9 4/9] hw/m68k: implement ADB bus support for via, Laurent Vivier, 2019/09/09
- [Qemu-devel] [PATCH v9 7/9] hw/m68k: add Nubus support for macfb video card, Laurent Vivier, 2019/09/09
- [Qemu-devel] [PATCH v9 2/9] dp8393x: manage big endian bus, Laurent Vivier, 2019/09/09
- [Qemu-devel] [PATCH v9 1/9] esp: add pseudo-DMA as used by Macintosh, Laurent Vivier, 2019/09/09
- [Qemu-devel] [PATCH v9 8/9] hw/m68k: add a dummy SWIM floppy controller, Laurent Vivier, 2019/09/09
- Re: [Qemu-devel] [PATCH v9 8/9] hw/m68k: add a dummy SWIM floppy controller, Aleksandar Markovic, 2019/09/09
- [Qemu-devel] [PATCH v9 5/9] hw/m68k: add macfb video card, Laurent Vivier, 2019/09/09
- [Qemu-devel] [PATCH v9 6/9] hw/m68k: add Nubus support, Laurent Vivier, 2019/09/09
- [Qemu-devel] [PATCH v9 9/9] hw/m68k: define Macintosh Quadra 800, Laurent Vivier, 2019/09/09
- [Qemu-devel] [PATCH v9 3/9] hw/m68k: add via support, Laurent Vivier, 2019/09/09
- Re: [Qemu-devel] [PATCH v9 0/9] hw/m68k: add Apple Machintosh Quadra 800 machine, no-reply, 2019/09/09
- Re: [Qemu-devel] [PATCH v9 0/9] hw/m68k: add Apple Machintosh Quadra 800 machine, no-reply, 2019/09/09