[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [Qemu-devel] [PATCH v9 0/6] migration/ppc: migrating DRC,
From: |
Laurent Vivier |
Subject: |
Re: [Qemu-ppc] [Qemu-devel] [PATCH v9 0/6] migration/ppc: migrating DRC, ccs_list and pending_events |
Date: |
Thu, 11 May 2017 20:38:27 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.0 |
Hi Daniel,
I'd like to test your patch series: is there a way to generate easily
the problem your series wants to fix?
Thanks,
Laurent
On 05/05/2017 22:47, Daniel Henrique Barboza wrote:
> v9:
> - patch 1 (*new*): added a qtail in sPAPRMachineState called
> pending_dimm_unplugs
> that stores the DIMM LMB state during the unplug process.
> - patch 2 (*new*): merged v8-patch1 and v8-patch2: removing detach_cb and
> detach_cb_opaque.
> - patch 3:
> * removed dk->vmsd entry. We're using vmstate_register instead
> * added 'awaiting_allocation' flag in the DRC migration
> - patch 4 (*new*): migrating spapr->pending_dimm_unplugs qtailq to allow
> for an ongoing PCDIMM unplug to continue after a migration.
>
> v8:
> - new patch added: 'removing spapr_drc_detach_cb opaques'. This new patch
> removes
> the need for the detach_cb_opaques inside the removal callback functions. See
> the commit message of the patch for more info.
>
> v7:
> - removed the first patch. DRC registration is now done by vmstate_register
> in patch 2.
> - added a new patch that changes spapr_dr_connector_new to receive as argument
> the detach_cb.
> - removed the callback logic of patch 2 since there is no need to restore the
> detach_cb on post-load due to the detach_cb on spapr_dr_connector_new change.
> - added word separators in the VMSD names of patch 3 and 4.
>
> v6: - Rebased with QEMU master after 6+ months.
> - Simplified the logic in patch 1.
> - Reworked patch 2: added CPU DRC migration, removed a function pointer
> from DRC
> class and minor improvements.
> - Added clarifications from the previous v5 discussions in the commit
> messages.
>
> v5: - Rebased to David's ppc-for-2.8.
>
> v4: - Introduce a way to set customized instance_id in SaveStateEntry. Use it
> to set instance_id for DRC using its unique index to address David
> Gibson's concern.
> - Rename VMS_CSTM to VMS_LINKED based on Paolo Bonzini's suggestions.
> - Clean up qjson stuff in put_qtailq.
> - Add trace for put_qtailq and get_qtailq based on David Gilbert's
> suggestion.
>
> - Based on David's ppc-for-2.7.
>
> v3: - Simplify overall design followng discussion with Paolo. No longer need
> metadata to migrate QTAILQ.
> - Extend VMStateInfo instead of adding similar fields to VMStateField.
> - Clean up macros in qemu/queue.h.
> (link: https://lists.nongnu.org/archive/html/qemu-devel/2016-05/msg05695.html)
>
> v2: - Introduce a general approach to migrate QTAILQ in qemu/queue.h.
> - Migrate signalled field in the DRC state.
> - Put the newly added migrating fields in subsections so that backward
> migration is not broken.
> - Set detach_cb field right after migration so that a migrated hot-unplug
> event could finish its course.
> (link: https://lists.nongnu.org/archive/html/qemu-devel/2016-05/msg04188.html)
>
> v1: - Inital version.
> (link: https://lists.nongnu.org/archive/html/qemu-devel/2016-04/msg02601.html)
>
>
> To make guest devices (PCI, CPU and memory) hotplug work together
> with guest migration, spapr drc state needs be transmitted in
> migration. This patch defines the VMStateDescription struct for
> spapr drc state to enable it.
>
> To fix the potential racing between hotplug events on guest and
> guest migration, ccs_list and pending_events of spapr state need be
> transmitted in migration. This patch set also takes care of it.
>
>
>
> Daniel Henrique Barboza (4):
> hw/ppc/spapr.c: adding pending_dimm_unplugs to sPAPRMachineState
> hw/ppc: removing drc->detach_cb and drc->detach_cb_opaque
> hw/ppc: migrating the DRC state of hotplugged devices
> hw/ppc/spapr.c: migrate pending_dimm_unplugs of spapr state
>
> Jianjun Duan (2):
> migration: spapr: migrate ccs_list in spapr state
> migration: spapr: migrate pending_events of spapr state
>
> hw/ppc/spapr.c | 158
> +++++++++++++++++++++++++++++++++++++++++---
> hw/ppc/spapr_drc.c | 97 ++++++++++++++++++++++-----
> hw/ppc/spapr_events.c | 24 ++++---
> hw/ppc/spapr_pci.c | 5 +-
> include/hw/pci-host/spapr.h | 3 +
> include/hw/ppc/spapr.h | 24 ++++++-
> include/hw/ppc/spapr_drc.h | 8 +--
> 7 files changed, 273 insertions(+), 46 deletions(-)
>
Re: [Qemu-ppc] [Qemu-devel] [PATCH v9 0/6] migration/ppc: migrating DRC, ccs_list and pending_events,
Laurent Vivier <=