[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 05/14] pc: route all memory devices through t
From: |
David Hildenbrand |
Subject: |
Re: [Qemu-devel] [PATCH v4 05/14] pc: route all memory devices through the machine hotplug handler |
Date: |
Wed, 30 May 2018 16:31:04 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 |
On 30.05.2018 16:27, Paolo Bonzini wrote:
> On 30/05/2018 16:08, David Hildenbrand wrote:
>>>> if (object_dynamic_cast(OBJECT(dev), TYPE_PC_DIMM) ||
>>>> + object_dynamic_cast(OBJECT(dev), TYPE_MEMORY_DEVICE) ||
>>> you probably could drop TYPE_PC_DIMM above, it's redundant since DIMM
>>> can be cast to TYPE_MEMORY_DEVICE
>>>
>>> ditto for spapr
>>>
>> Yes, had the same in mind but left it for now this way (basically
>> because we do special handling for PC_DIMM, so anybody reading this code
>> is not directly confused)
>>
>
> Hmm, I think what Igor proposes is nicer. Can you send a v5 when he's
> done with review?
Sure!
>
> Paolo
>
--
Thanks,
David / dhildenb
- [Qemu-devel] [PATCH v4 02/14] memory-device: introduce separate config option, (continued)
[Qemu-devel] [PATCH v4 06/14] spapr: prepare for multi stage hotplug handlers, David Hildenbrand, 2018/05/17
[Qemu-devel] [PATCH v4 08/14] spapr: handle pc-dimm unplug via hotplug handler chain, David Hildenbrand, 2018/05/17
[Qemu-devel] [PATCH v4 07/14] spapr: route all memory devices through the machine hotplug handler, David Hildenbrand, 2018/05/17
[Qemu-devel] [PATCH v4 09/14] spapr: handle cpu core unplug via hotplug handler chain, David Hildenbrand, 2018/05/17
[Qemu-devel] [PATCH v4 10/14] memory-device: new functions to handle plug/unplug, David Hildenbrand, 2018/05/17
[Qemu-devel] [PATCH v4 11/14] pc-dimm: implement new memory device functions, David Hildenbrand, 2018/05/17
[Qemu-devel] [PATCH v4 13/14] memory-device: factor out unplug into hotplug handler, David Hildenbrand, 2018/05/17
[Qemu-devel] [PATCH v4 12/14] memory-device: factor out pre-plug into hotplug handler, David Hildenbrand, 2018/05/17
[Qemu-devel] [PATCH v4 14/14] memory-device: factor out plug into hotplug handler, David Hildenbrand, 2018/05/17