[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/5] VMState cleanups
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH 0/5] VMState cleanups |
Date: |
Wed, 22 Feb 2012 11:26:12 +0000 |
On 22 February 2012 10:15, Igor Mitsyanko <address@hidden> wrote:
> This patchset cleans up and optimizes vmstate implementation.
>
> Patch 1 is a trivial bug fixing.
> Patches 2 and 3 replaces target_phys_addr_t in pxa implementation
> to uint32_t.
> Patch 4 moves VMSTATE_UINTTL from hw.h to vmstate.h. Explicit dependency
> on NEED_CPU_H is droped, I failed to understand why it was presented at all.
So if we apply patches 1-3 (which all look plausible) then the only
remaining user of VMSTATE_UINTTL is target-i386/machine.c as far as
I can see.
This leaves me wondering if we shouldn't just put it actually in
target-i386/machine.c as a convenience macro for that specific CPU
to avoid having to have more #ifdef TARGET_X86_64s. (I note that
the machine.c code is already pretty inconsistent, eg lstar and
cstar are defined as target_ulong and saved with VMSTATE_UINT64.)
Basically VMSTATE_UINTTL seems like a bit of a dangerous thing to
leave lying around as there aren't really very many use cases
for it...
-- PMM
- Re: [Qemu-devel] [PATCH 3/5] hw/pxa2xx_lcd.c: drop VMSTATE_UINTTL usage, (continued)
[Qemu-devel] [PATCH 1/5] target-alpha/machine.c: use VMSTATE_UINT64* instead of VMSTATE_UINTTL*, Igor Mitsyanko, 2012/02/22
[Qemu-devel] [PATCH 4/5] vmstate: refactor and move VMSTATE_UINTTL* macro, Igor Mitsyanko, 2012/02/22
[Qemu-devel] [PATCH 5/5] vmstate: introduce get_bufsize entry in VMStateField, Igor Mitsyanko, 2012/02/22
Re: [Qemu-devel] [PATCH 0/5] VMState cleanups,
Peter Maydell <=
- Re: [Qemu-devel] [PATCH 0/5] VMState cleanups, Mitsyanko Igor, 2012/02/22
- Re: [Qemu-devel] [PATCH 0/5] VMState cleanups, Andreas Färber, 2012/02/22
- Re: [Qemu-devel] [PATCH 0/5] VMState cleanups, Juan Quintela, 2012/02/22
- Re: [Qemu-devel] [PATCH 0/5] VMState cleanups, Andreas Färber, 2012/02/22
- Re: [Qemu-devel] [PATCH 0/5] VMState cleanups, Peter Maydell, 2012/02/22
- Re: [Qemu-devel] [PATCH 0/5] VMState cleanups, Andreas Färber, 2012/02/22
- Re: [Qemu-devel] [PATCH 0/5] VMState cleanups, Peter Maydell, 2012/02/22
- Re: [Qemu-devel] [PATCH 0/5] VMState cleanups, Alexander Graf, 2012/02/22
- Re: [Qemu-devel] [PATCH 0/5] VMState cleanups, Juan Quintela, 2012/02/23