[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 0/2] VT-d migration support
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [Qemu-devel] [PATCH v3 0/2] VT-d migration support |
Date: |
Fri, 6 Jan 2017 13:27:38 +0000 |
User-agent: |
Mutt/1.7.1 (2016-10-04) |
* Peter Xu (address@hidden) wrote:
> This series enables VT-d IOMMU migration.
>
> v3:
> - save/load for csr [Jason]
>
> v2:
> - remove SaveStateEntry.priority, let priority store only in one
> place, which is VMStateDescription. Meanwhile, provide another
> helper to fetch the priority.
> - add enum MigrationPriority to unify the ordering, rather than use
> magic numbers like 100 everywhere [Dave]
> - fix commit log where proper
>
> First patch is an enhancement to current migration framework to allow
> prioritized save state entry. This is required to let VT-d migrate
> properly.
>
> The second patch enables the migration for the vIOMMU.
>
> Please check commit message for more information.
>
> Test done:
>
> Smoke test is done with parameter:
>
> $qemu -M q35,kernel-irqchip=off -enable-kvm \
> -device intel-iommu,intremap=on \
> -netdev user,id=net0 \
> -device e1000,netdev=net0 \
> -m 512M -monitor stdio \
> $param /var/lib/libvirt/images/vm1.qcow2
>
> Please review, thanks.
>
> (P.S. I found that split irqchip cannot work well with migration. Is
> this an known issue?)
How did it fail?
Dave
> Peter Xu (2):
> migration: allow to prioritize save state entries
> intel_iommu: allow migration
>
> hw/i386/intel_iommu.c | 22 +++++++++++++++++++++-
> include/migration/vmstate.h | 7 +++++++
> migration/savevm.c | 34 ++++++++++++++++++++++++++++++----
> 3 files changed, 58 insertions(+), 5 deletions(-)
>
> --
> 2.7.4
>
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK