[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/7] savevm: Add VMSTATE_ helpers for target_phy
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH 4/7] savevm: Add VMSTATE_ helpers for target_phys_addr_t |
Date: |
Tue, 9 Oct 2012 09:03:21 +0100 |
On 9 October 2012 05:53, David Gibson <address@hidden> wrote:
> The savevm code contains VMSTATE_ helpers for a number of commonly used
> types, but not for target_phys_addr_t. This patch fixes that deficiency
> implementing VMSTATE_TPA helpers in terms of VMSTATE_UINT32 or
> VMSTATE_UINT64 helpers as appropriate.
(This comment is out of date...)
Traditionally we have deliberately not had any vmstate helpers
for target_phys_addr_t because the meaning of that type is "a
type wide enough to hold the widest address in use in the system".
vmstate fields, on the other hand, are for the state of a specific
device model, and a particular device's registers are always a
fixed width. Even if a pci card model is plugged into a system
with 64 bit PCI addresses, if the chip on the card itself only
has 32 bit wide registers controlling DMA those registers need
to be uint32_t. So if you think you need to put target_phys_addr_t
into a vmstate the chances are you don't. I think this logic
should still hold even now we have made target_phys_addr_t
64 bits for every platform.
What was the problem this patch is trying to fix?
-- PMM
- [Qemu-devel] [0/7] vmstate extensions, David Gibson, 2012/10/09
- [Qemu-devel] [PATCH 3/7] savevm: Add VMSTATE_FLOAT64 helpers, David Gibson, 2012/10/09
- [Qemu-devel] [PATCH 4/7] savevm: Add VMSTATE_ helpers for target_phys_addr_t, David Gibson, 2012/10/09
- Re: [Qemu-devel] [PATCH 4/7] savevm: Add VMSTATE_ helpers for target_phys_addr_t,
Peter Maydell <=
- Re: [Qemu-devel] [PATCH 4/7] savevm: Add VMSTATE_ helpers for target_phys_addr_t, David Gibson, 2012/10/09
- Re: [Qemu-devel] [PATCH 4/7] savevm: Add VMSTATE_ helpers for target_phys_addr_t, Peter Maydell, 2012/10/09
- Re: [Qemu-devel] [PATCH 4/7] savevm: Add VMSTATE_ helpers for target_phys_addr_t, David Gibson, 2012/10/09
- Re: [Qemu-devel] [PATCH 4/7] savevm: Add VMSTATE_ helpers for target_phys_addr_t, David Gibson, 2012/10/11
- Re: [Qemu-devel] [PATCH 4/7] savevm: Add VMSTATE_ helpers for target_phys_addr_t, Peter Maydell, 2012/10/11
- Re: [Qemu-devel] [PATCH 4/7] savevm: Add VMSTATE_ helpers for target_phys_addr_t, David Gibson, 2012/10/11
[Qemu-devel] [PATCH 1/7] savevm: Add VMSTATE_UINT64_EQUAL helpers, David Gibson, 2012/10/09
[Qemu-devel] [PATCH 2/7] savevm: Add VMSTATE_UINTTL_EQUAL helper, David Gibson, 2012/10/09
[Qemu-devel] [PATCH 5/7] savevm: Add VMSTATE_STRUCT_VARRAY_POINTER_UINT32, David Gibson, 2012/10/09
[Qemu-devel] [PATCH 7/7] savevm: Implement VMS_DIVIDE flag, David Gibson, 2012/10/09