[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL v2 07/11] target-lm32: move model features to LM3
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PULL v2 07/11] target-lm32: move model features to LM32CPU |
Date: |
Mon, 14 Oct 2013 19:16:26 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.0 |
Hi,
Am 14.10.2013 18:29, schrieb Michael Walle:
> This allows us to completely remove CPULM32State from DisasContext.
> Instead, copy the fields we need to DisasContext.
>
> Cc: Andreas Färber <address@hidden>
> Signed-off-by: Michael Walle <address@hidden>
Sorry for not getting to review this earlier.
The subject makes it sound to me as if a "features" field were being
moved from CPULM32State to LM32CPU, which would be great, as done or
proposed elsewhere.
However, instead it is adding a pointer field to LM32CPU to the
definition the class was originally created from IIUC. I would rather
access those fields through LM32CPUClass then (an existing pointer in
the object instance) - but since xtensa is still doing the same and this
not being functionally wrong, no objection to the pull, just a note that
we should revisit it.
Regards,
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
- [Qemu-devel] [PULL v2 00/11] target-lm32 updates, Michael Walle, 2013/10/14
- [Qemu-devel] [PULL v2 01/11] lm32_sys: increase test case name length limit, Michael Walle, 2013/10/14
- [Qemu-devel] [PULL v2 07/11] target-lm32: move model features to LM32CPU, Michael Walle, 2013/10/14
- Re: [Qemu-devel] [PULL v2 07/11] target-lm32: move model features to LM32CPU,
Andreas Färber <=
- [Qemu-devel] [PULL v2 03/11] milkymist-uart: use qemu_chr_fe_write_all() instead of qemu_chr_fe_write(), Michael Walle, 2013/10/14
- [Qemu-devel] [PULL v2 06/11] target-lm32: kill cpu_abort() calls, Michael Walle, 2013/10/14
- [Qemu-devel] [PULL v2 02/11] tests: lm32: new rule for single test cases, Michael Walle, 2013/10/14
- [Qemu-devel] [PULL v2 05/11] milkymist-vgafb: swap pixel data in source buffer, Michael Walle, 2013/10/14
- [Qemu-devel] [PULL v2 04/11] lm32_uart/lm32_juart: use qemu_chr_fe_write_all(), Michael Walle, 2013/10/14
- [Qemu-devel] [PULL v2 10/11] lm32_sys: dump cpu state if test case fails, Michael Walle, 2013/10/14